Skip to content

Remove outdated jdk11 profile - #1174

Merged
snuyanzin merged 1 commit into
datafaker-net:mainfrom
snuyanzin:jdk11
Apr 25, 2024
Merged

snuyanzin merged 1 commit into
datafaker-net:mainfrom
snuyanzin:jdk11

Conversation

@snuyanzin

Copy link
Copy Markdown
Collaborator

No description provided.

@what-the-diff

what-the-diff Bot commented Apr 25, 2024

Copy link
Copy Markdown

PR Summary

  • Removal of <profile> block for jdk11+
    This PR includes removing a particular section in the project's configuration file which was meant for handling specific settings when the project was being run with JDK version 11 and above. This change implies that specific settings for different JDK versions are no longer necessary, possibly indicating enhancements in compatibility across different versions of the JDK (Java Development Kit).

@kingthorin

Copy link
Copy Markdown
Collaborator

I have nothing against this change, I don't know maven well enough. Should we just be changing this to 17+?

@snuyanzin

Copy link
Copy Markdown
Collaborator Author

it looks like anyway it became useless since all the tests are passing without these flags
Thus I do not see a reason to keep with jdk17 or 21

@snuyanzin
snuyanzin merged commit d5561a7 into datafaker-net:main Apr 25, 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