Skip to content

/ Slim-Http

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename Decorators And Match Original Slim-Http Namespace #73

Merged
merged 4 commits into from Nov 26, 2018
Merged

Rename Decorators And Match Original Slim-Http Namespace #73

merged 4 commits into from Nov 26, 2018

Conversation

@l0gicgate
Copy link
Member

l0gicgate commented Nov 26, 2018

As per our Slack discussion. Removing the Decorator keyword and renaming decorated objects to match original Slim-Http namespacing.
l0gicgate added 4 commits Nov 26, 2018
@l0gicgate
rename decorators to match anterior Slim-Http objects and namespace
da2abf9
@l0gicgate
fix README
94155b6
@l0gicgate
rename base Test class to TestCase for cohesion with Slim repo
2e05ab0
@l0gicgate
rename test cases for decorators
Loading status checks…
468ccb5
@coveralls
Copy link

coveralls commented Nov 26, 2018

Coverage Status

Coverage remained the same at 99.27% when pulling 468ccb5 on l0gicgate:Rename-Decorators into efdc70a on slimphp:master.
@akrabat akrabat merged commit 468ccb5 into slimphp:master Nov 26, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 99.27%
Details
akrabat added a commit that referenced this pull request Nov 26, 2018
@akrabat
Merge branch 'Rename-Decorators'
Verified
This commit was signed with a verified signature.
akrabat Rob Allen
GPG key ID: 0A6B29B5B3F4BC23 Learn about signing commits
Loading status checks…
9ada50c 
Closes #73
@akrabat akrabat added this to the 0.6 milestone Nov 26, 2018
@l0gicgate l0gicgate deleted the l0gicgate:Rename-Decorators branch May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@l0gicgate @coveralls @akrabat
You can’t perform that action at this time.