Skip to content

Conversation

@Technici4n
Copy link
Member

@Technici4n Technici4n commented Apr 25, 2025

Easy enough actually. What I am not sure about is what the CLI args should look like for this feature.

This will be useful for neoforged/ModDevGradle#236.

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

public AccessTransformerValidation validation = AccessTransformerValidation.LOG;

// TODO: as is, either this or the normal option is required
@CommandLine.Option(names = "--access-transformer-no-validation")
Copy link
Contributor

Choose a reason for hiding this comment

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

--access-transformer-validation-exclude ??

@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label May 2, 2025
@neoforged-automation
Copy link
Contributor

@Technici4n, this pull request has conflicts, please resolve them for this PR to move forward.

@shartte
Copy link
Contributor

shartte commented May 2, 2025

This is now fixed via #50 and filtering the problem report by path in NFRT

@shartte shartte closed this May 2, 2025
@Technici4n Technici4n deleted the validate-subset-of-ats branch May 2, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase This Pull Request needs to be rebased before being merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants