Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Migrate from Plexus to Sisu Guice #341

Merged
merged 4 commits into from
Nov 19, 2024
Merged

Migrate from Plexus to Sisu Guice #341

merged 4 commits into from
Nov 19, 2024

Conversation

elharo
Copy link
Contributor

@elharo elharo commented Nov 18, 2024

No description provided.

@elharo elharo marked this pull request as ready for review November 18, 2024 12:15
@elharo elharo requested a review from michael-o November 19, 2024 02:06
@michael-o michael-o requested review from cstamas, gnodet and slawekjaranowski and removed request for michael-o November 19, 2024 09:04
pom.xml Outdated
Comment on lines 492 to 499
<executions>
<execution>
<id>generate-index</id>
<goals>
<goal>main-index</goal>
</goals>
</execution>
</executions>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a definition in parent pom ....
https://github.com/apache/maven-parent/blob/master/pom.xml#L1014-L1027

with additional we execute plugin twice:

[INFO] --- sisu:0.9.0.M3:main-index (index-project) @ maven-javadoc-plugin ---
...
[INFO] --- sisu:0.9.0.M3:main-index (generate-index) @ maven-javadoc-plugin ---

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch. removed.

@elharo elharo merged commit 69c1ba7 into master Nov 19, 2024
113 checks passed
@elharo elharo deleted the guice branch November 19, 2024 12:31
rreich added a commit to CoreMedia/maven-javadoc-plugin that referenced this pull request Feb 17, 2025
…a-update-3.12.0

* commit 'dc085593a39b845805ccbd04078c782bc1859275': (162 commits)
  [MJAVADOC-824] remove output-configuration.apt.vm (apache#349)
  Specify locale for case conversion (apache#345)
  Replace two deprecated methods (apache#347)
  Replace some deprecated FileUtils methods (apache#340)
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release maven-javadoc-plugin-3.11.2
  use github for scm
  Bump com.thoughtworks.qdox:qdox from 2.1.0 to 2.2.0
  [MJAVADOC-823] legacyMode keeps using module-info.java (-sourcedirectory still use as well as java files input) (apache#343)
  Bump commons-io:commons-io from 2.17.0 to 2.18.0
  Migrate from Plexus to Sisu Guice (apache#341)
  Remove usages of deprecated ReaderFactory class (apache#339)
  [MJAVADOC-814] handle parameters such packages with multi lines (apache#337)
  refactor: Replace Plexus AbstractLogEnabled with SLF4J (apache#338)
  [MJAVADOC-822] skippedModules should be more scalable and support regex (apache#336)
  Bump org.codehaus.mojo:l10n-maven-plugin from 1.0.0 to 1.1.0
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release maven-javadoc-plugin-3.11.1
  [MJAVADOC-821] Align toolchain discovery code with Maven Compiler Plugin
  [MJAVADOC-820] [REGRESSION] MJAVADOC-787 was merged incompletely
  ...
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants