-
Notifications
You must be signed in to change notification settings - Fork 46
Gradle cleanup, interface definition cleanup #390
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
Conversation
mkosmul
commented
Oct 16, 2024
- Using Gradle toolchain for easier compilation on various machines (auto-downloading needed JDK)
- Updated dependencies and version catalog made more readable
- Removed unneccessary public and abstract modifiers from interface methods
…auto-downloading needed JDK) * Updated dependencies and version catalog made more readable * Removed unneccessary public and abstract modifiers from interface methods
Ooouch1
left a comment
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.
Good updates. Leave the "public" in comments remaining, or remove all license comments from the code.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
I know putting a license comment in every code is an old way and annoying, though.
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.
Sorry, my oversight. I'll fix this in a moment.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
| * Copyright (C) 2005-2009 Jun Mitani http://mitani.cs.tsukuba.ac.jp/ | ||
| This program is free software: you can redistribute it and/or modify | ||
| it under the terms of the GNU General Public License as published by |
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 "public" in comments should not be removed.
Ooouch1
left a comment
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.
Cool. You can merge this PR if allowed. (I'm not sure the project setting allows you to merge...)
Unfortunately, I don't have permission to merge. |