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

Fix version maven source plugin #117

Merged
merged 23 commits into from
Jan 13, 2025

Conversation

EyalDelarea
Copy link

@EyalDelarea EyalDelarea commented Jan 12, 2025

This PR sets the version of the maven-source-plugin to 3.2.1.

The decision to pin the version is due to issues introduced in versions 3.3+, where the plugin generates source files at the parent level. This behavior causes build conflicts that could not be resolved with the current configuration.

As part of troubleshooting, I attempted switching the goal from jar to jar-no-fork based on recommendations (e.g., StackOverflow post, but this did not resolve the issue in our build setup.

If you are reviewing this in the future, you may try updating the plugin version to see if the conflicts have been addressed in newer releases and if the build passes successfully.

@EyalDelarea EyalDelarea added the ignore for release Automatically generated release notes label Jan 12, 2025
@EyalDelarea EyalDelarea added the safe to test Approve running integration tests on a pull request label Jan 12, 2025
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Jan 12, 2025
@EyalDelarea EyalDelarea merged commit a2cb2e8 into jfrog:main Jan 13, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore for release Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants