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

(main) Use latest maven-plugin-tools #717

Merged

Conversation

abelsromero
Copy link
Member

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Documentation
  • Refactor
  • Build improvement
  • Other (please describe)

What is the goal of this pull request?

Maven Plugin Tools is a set of tools (plugin, dependencies) to generate metadata required for the Maven plugin. As such, we can run the latest and use Dependabot to automate the management without issues.
We still have CI tests to validate backward compatibility.

Are there any alternative ways to implement this?

Keep using a pinned older version, however that means we cannot use Dependabot. So we depend on manual action to update from time to time.
Since we have CI to test multiple Maven versions I don't think it makes sense to keep this "use older version" policy.
There will be other similar PRs like this for other dependencies.

Are there any implications of this pull request? Anything a user must know?

no

Is it related to an existing issue?

  • Yes
  • No

Finally, please add a corresponding entry to CHANGELOG.adoc

Maven Plugin Tools is a set of tools (plugin, dependencies)
to generate metadata required for the Maven plugin.
As such, we can run the latest and use Dependabot to automate
the management without issues.
We still have CI tests to validate backward compatibility
@abelsromero abelsromero merged commit 8130ba9 into asciidoctor:main Jan 1, 2024
19 checks passed
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PR asciidoctor#717, to ease project maintenance
we can automate Doxia dependencies and use CI
to validate backward compatibility.
@abelsromero abelsromero mentioned this pull request Jan 1, 2024
8 tasks
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PR asciidoctor#717, to ease project maintenance
we can automate Doxia dependencies and use CI
to validate backward compatibility.
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PR asciidoctor#717, to ease project maintenance
we can automate Doxia dependencies and use CI
to validate backward compatibility.
abelsromero added a commit that referenced this pull request Jan 1, 2024
* Similar to PR #717, to ease project maintenance
we can automate Doxia dependencies and use CI
to validate backward compatibility.
* Move Doxia properties to parent pom
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PRs asciidoctor#717 and asciidoctor#719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PRs asciidoctor#717 and asciidoctor#719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Remove maven exclusion from Dependabot.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
* Fix mising 'url' in 'asciidoctor-parser-doxia-module' module.
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PRs asciidoctor#717 and asciidoctor#719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Remove maven exclusion from Dependabot.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
* Fix mising 'url' in 'asciidoctor-parser-doxia-module' module.
@abelsromero abelsromero mentioned this pull request Jan 1, 2024
8 tasks
abelsromero added a commit to abelsromero/asciidoctor-maven-plugin that referenced this pull request Jan 1, 2024
Similar to PRs asciidoctor#717 and asciidoctor#719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Remove maven exclusion from Dependabot.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
* Fix mising 'url' in 'asciidoctor-parser-doxia-module' module.
abelsromero added a commit that referenced this pull request Jan 1, 2024
Similar to PRs #717 and #719, to ease project maintenance
we can automate Maven (core and transitives) dependencies
and use CI to validate backward compatibility.
* Remove maven exclusion from Dependabot.
* Deleted direct references to dependencies that are provided
by maven-core: ideally we'd like to refer them if we use them,
but ther lack of a BOM make it un-practical.
* Rename build properties for ease of use.
* Fix mising 'url' in 'asciidoctor-parser-doxia-module' module.
@abelsromero abelsromero deleted the use-latest-maven-plugin-tools branch February 11, 2024 17:45
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.

1 participant