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.

Sign up for GitHub

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

Jump to bottom

Javadoc for AvajeJavalinPlugin and tidy javalin ControllerWriter only #342

Merged

SentryMan merged 1 commit into master from feature/tidy-n-javadoc

Nov 1, 2023

Contributor

rob-bygrave commented Nov 1, 2023 •

edited

Loading

Notes / thoughts:

AvajeJavalinPlugin could almost be called AvajeJavalinRoute
Oh, I don't like that the Javalin Plugin is an abstract type - yuk !!!


          Javadoc for AvajeJavalinPlugin and tidy javalin ControllerWriter only

d8be3d4

SentryMan enabled auto-merge

November 1, 2023 22:48

SentryMan approved these changes

View reviewed changes

SentryMan merged commit cc2c233 into master

3 checks passed

SentryMan deleted the feature/tidy-n-javadoc branch

November 1, 2023 22:48

Contributor Author

rob-bygrave commented Nov 1, 2023

FYI: I've added a comment to the Javalin PR that changed Plugin into an abstract type, will see what the reply there is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet