Skip to content
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

Java update from 21.0.4 to 21.0.5 including different depenndencies #7221

Merged
merged 5 commits into from
Nov 16, 2024

Conversation

Vest
Copy link
Contributor

@Vest Vest commented Nov 11, 2024

@karianna, I have prepared this PR with a small update. Can you please review it?
Please merge the changes, and don't rebase them.

p.s. I made a typo in one of the commit messages, I have updated Java to 21.0.5, and not to 21.0.4.

Vest and others added 3 commits November 9, 2024 12:42
Upgraded CheckStyle from 10.18.1 to 10.20.1
Upgraded pmd from 7.6.0 to 7.7.0
Corrected pmd error in ClassLevelTableModel.java
Updated ruleset.xml, renamed deprecated rules.
@karianna
Copy link
Contributor

@Vest - I can Squash and merge but not pure merge (unless I override at the org). LMK what you prefer.

build.gradle Outdated Show resolved Hide resolved
@Vest
Copy link
Contributor Author

Vest commented Nov 12, 2024

I prefer squash, because I don’t like scattered commits. People can always find how I came to this commit, if they read this PR.

@Vest Vest requested a review from karianna November 13, 2024 11:47
@karianna karianna merged commit d30055b into PCGen:master Nov 16, 2024
2 checks passed
@Vest Vest deleted the update_deps branch November 16, 2024 21:57
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