-
Notifications
You must be signed in to change notification settings - Fork 50
chore(QTDI-1765): remove out-dated plugin #1078
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
chore(QTDI-1765): remove out-dated plugin #1078
Conversation
* replace its functionality with a community supported plugins
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes the outdated common-maven-plugin
from the project and replaces its functionality with community-supported alternatives. The PR also includes a version bump for the spotless maven plugin from 2.44.4 to 2.46.1.
- Removal of
common-maven-plugin
dependency and configuration from all POM files - Replacement of plugin functionality with standard Maven plugins (
maven-antrun-plugin
andmaven-resources-plugin
) - Version upgrade of
spotless-maven-plugin
to 2.46.1
Reviewed Changes
Copilot reviewed 7 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
pom.xml | Removes common-maven-plugin property and configurations, updates spotless version, removes POM sorting configuration |
documentation/pom.xml | Replaces common-maven-plugin with maven-antrun-plugin and maven-resources-plugin for file filtering |
component-tools-webapp/pom.xml | Adds maven-antrun-plugin and maven-resources-plugin to replace common-maven-plugin functionality |
component-starter-server/pom.xml | Adds maven-antrun-plugin and maven-resources-plugin executions for frontend file processing |
component-studio/pom.xml | Removes common-maven-plugin configuration |
talend-component-maven-plugin/src/it/dependency/pom.xml | Removes common-maven-plugin configuration |
component-runtime-testing/component-runtime-junit/pom.xml | Minor comment formatting change |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Did not find what was the purpose of the rootlocation value?!?
Requirements
Why this PR is needed?
What does this PR adds (design/code thoughts)?