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

Refine AspectJ plugin build phase config #3284

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Dec 29, 2023

  1. Move AspectJ Maven executions to default phases

    Document the pros and cons in the POM with this comment:
    
    Attention: aspectj-maven-plugin MUST be declared AFTER
    maven-compiler-plugin, if they are both supposed to run in the same
    default phases 'compile' and 'test-compile', but execution order is to
    be guaranteed. Then, you have the convenience of being able to run e.g.
    'mvn [clean] compile' instead of 'mvn [clean] process-classes'. For a
    slightly less convenient, but less refactoring-error-prone solution,
    see the commented-out phases below.
    kriegaex committed Dec 29, 2023
    Configuration menu
    Copy the full SHA
    6e28a9f View commit details
    Browse the repository at this point in the history

Commits on Dec 30, 2023

  1. Configuration menu
    Copy the full SHA
    4ac9cb4 View commit details
    Browse the repository at this point in the history