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

Update MoppyControlGUI dependencies & add temporary IntelliJ files to .gitignore #217

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

AmonBune
Copy link
Contributor

Updates all dependencies in the MoppyControlGUI and MoppyLib to their latest version.
I also included the names of the temporary files which were generated by IntelliJ in the .gitignore, so they won't be uploaded anymore.

@Sammy1Am
Copy link
Owner

Oo, thanks! What version of Java have you tested this on? (There were some issues with 23 (or maybe 22?), curious to see if the dependencies were the issue)

@AmonBune
Copy link
Contributor Author

I used Java 21. Initially I wanted to make 22 work since it's the latest stable version but I couldn't figure out what was the cause of this issue: #210
Maybe you have an idea? It seems to be something quite low level inside GraalVM 🤷

@AmonBune AmonBune changed the title Update MoppyControlGUI dependencies & add temoporary IntelliJ files to .gitignore Update MoppyControlGUI dependencies & add temporary IntelliJ files to .gitignore Jul 18, 2024
@Sammy1Am
Copy link
Owner

Yeah, I suspect they just made a weird breaking change somewhere... I don't think GraalVM is probably super heavily used. :P

Appreciate the update to the dependencies though! Hopefully I'll get time to fix the weirdness soon.

@Sammy1Am Sammy1Am merged commit 96d811b into Sammy1Am:master Jul 18, 2024
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