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

Maven plugin doesn't handle extra sources #1846

Closed
tisonkun opened this issue Oct 5, 2023 · 2 comments · Fixed by #1847
Closed

Maven plugin doesn't handle extra sources #1846

tisonkun opened this issue Oct 5, 2023 · 2 comments · Fixed by #1847

Comments

@tisonkun
Copy link
Contributor

tisonkun commented Oct 5, 2023

Testable repo - korandoru/hawkeye#99

hawkeye-gradle-plugin/src/functionalTest is an extra source directory, but the maven plugin cannot identify it or apply on it:

[DEBUG] Spotless will not format an up-to-date file: /Users/tison/Brittani/hawkeye/hawkeye-gradle-plugin/pom.xml
[INFO] Spotless.Format is keeping 1 files clean - 0 were changed to be clean, 0 were already clean, 1 were skipped because caching determined they were already clean
[DEBUG] Spotless will not format an up-to-date file: /Users/tison/Brittani/hawkeye/hawkeye-gradle-plugin/src/test/java/io/korandoru/hawkeye/gradle/plugin/HawkeyeGradlePluginTest.java
[DEBUG] Spotless will not format an up-to-date file: /Users/tison/Brittani/hawkeye/hawkeye-gradle-plugin/src/main/java/io/korandoru/hawkeye/gradle/plugin/HawkeyeGradlePlugin.java
[INFO] Spotless.Java is keeping 2 files clean - 0 were changed to be clean, 0 were already clean, 2 were skipped because caching determined they were already clean

Note that only files under src/main and src/test are processed.

@tisonkun
Copy link
Contributor Author

tisonkun commented Oct 5, 2023

If I use:

    <build>
        <sourceDirectory>src/functionalTest/java</sourceDirectory>
    </build>

It can help. But this will miss the src/main/java directory.

@basil
Copy link

basil commented Nov 27, 2023

Causes #1914.

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 a pull request may close this issue.

2 participants