Skip to content

Conversation

@adamretter
Copy link

@adamretter adamretter commented Aug 29, 2025

Allows specific files (or patterns of files) to be included and excluded, e.g.:

<configuration>
    <compliance>21</compliance>
    <noClasspath>true</noClasspath>
    <includes>
        <include>org/exist/security/UnixStylePermission.java</include>
    </includes>
    <processors>
        <processor>xyz.elemental.build.tools.spoon.processors.PermissionRequiredProcessor</processor>
    </processors>
</configuration>

@monperrus
Copy link
Contributor

thanks for the PR @adamretter

CI is broken, trying to fix it in INRIA/spoon#6421

@monperrus
Copy link
Contributor

CI is fixed, could you rebase? we might also need a test case. Thanks!

@adamretter adamretter force-pushed the feature/includes-excludes-sources branch from 4937302 to afc89ac Compare September 1, 2025 17:11
@adamretter
Copy link
Author

@monperrus Thank you for doing that. I have now rebased this PR. I am not sure how I would write a test case for this behaviour. Do you have an example please?

@monperrus
Copy link
Contributor

thanks for rebasing.

a test could be:

  • project to analyze
  • a folder with two processors, one crashing systematically

the test would check that:
without exclude it fails
with excluding the crashing processor it works

@adamretter
Copy link
Author

@monperrus Okay... thanks, but how do I set up such a thing within the structure of this project - is there an example/skeleton I could copy?

@monperrus
Copy link
Contributor

@monperrus
Copy link
Contributor

@adamretter would you document the new option in the README before merge? Thanks.

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.

2 participants