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

updated generation of AbstractLanguageServerTest #3333

Closed
wants to merge 1 commit into from

Conversation

cdietrich
Copy link
Contributor

No description provided.

@cdietrich
Copy link
Contributor Author

@LorenzoBettini @szarnekow is this file dirty for you too?

@LorenzoBettini
Copy link
Contributor

@cdietrich That happens when you use Java 21, I guess because of the different hierarchy structure.

Unfortunately, we should use Java 21 to run Eclipse and most of the UI tests. However, we target Java 17, so such dirty files should be reverted instead of committed. At least, that's what I always do.

I guess it's also safe to commit them; in the end, it's up to us.

@cdietrich
Copy link
Contributor Author

i see, so i will try on different pc with older tp....

@LorenzoBettini
Copy link
Contributor

i see, so i will try on different pc with older tp....

no need to: it only gets regenerated in clean builds; if you revert it, it will stay the same

@cdietrich cdietrich closed this Feb 3, 2025
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