Skip to content

Conversation

@mkosmul
Copy link
Contributor

@mkosmul 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
Copy link
Contributor

@Ooouch1 Ooouch1 left a 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
Copy link
Contributor

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.

Copy link
Contributor Author

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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.

Copy link
Contributor

@Ooouch1 Ooouch1 left a 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...)

@mkosmul
Copy link
Contributor Author

mkosmul commented Oct 16, 2024

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.

@Ooouch1 Ooouch1 merged commit 620218d into oripa:master Oct 17, 2024
@mkosmul mkosmul deleted the gradle-and-interface-cleanup branch October 17, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants