Skip to content

Implement setting of the toolchain on all tests#292

Merged
marchermans merged 4 commits into
NG_7.1from
fix/iis-ats-from-dependencies
Dec 6, 2025
Merged

Implement setting of the toolchain on all tests#292
marchermans merged 4 commits into
NG_7.1from
fix/iis-ats-from-dependencies

Conversation

@marchermans
Copy link
Copy Markdown
Member

Fix handling of ATs and IIS when they are loaded from dependencies.

The internal APIs where not properly updated to running in Binary Patch mode, where they eagerly resolved some configurations and file collections. This fixes this.

Additionally the expose(Dependency) method has been deprecated in favor of the DepenencyCollector based approach, a warning will be issued, and a link to the correct usage is added to the ReadMe.MD

Removal of stubs

Interface injected stubs where not properly removed from recompiled code. This is now fixed.

Closes: #291

@marchermans marchermans enabled auto-merge (squash) December 6, 2025 18:49
@neoforged-pr-publishing
Copy link
Copy Markdown

  • Publish PR to GitHub Packages

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 6, 2025

TestsPassed ✅SkippedFailed
JUnit Test Report534 ran534 passed0 skipped0 failed

@marchermans marchermans merged commit 0d854d3 into NG_7.1 Dec 6, 2025
100 checks passed
@marchermans marchermans deleted the fix/iis-ats-from-dependencies branch December 6, 2025 20:28
@neoforged-releases
Copy link
Copy Markdown

🚀 This PR has been released as NeoGradle version 7.1.8.

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.

Using interfaceInjections.expose for a dependency in "implementation" causes a ResolutionException

1 participant