-
Notifications
You must be signed in to change notification settings - Fork 2
Refactored Router to be a non-static class #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changed files are looking good now, but please do a commit with the updated examples as well. Currently, you are calling non-static methods statically in your examples.
Update the README too.
Examples are getting nuked and there's gonna be more changes which is why the README hasn't been updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright then, sounds good. Approved.
Why?
In attempts to make all of the libraries in TNAPF comply with PSR and be usable in any circumstance. It only makes sense to convert Router into a non-static class so it can be used with PSR-15. Which will be implemented soon