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

OptimizedWalker should skip conditions if the nested node is skipped #995

Conversation

romain-grecourt
Copy link
Contributor

  • Fix OptimizedWalker to skip condition if the nested node is skipped
  • Overload ArchetypeV2.generate to expose a variant that accepts externalValues, externalDefaults and directorySupplier This allows to avoid creating a new instance of the engine for every project generation

Fixes #994

@romain-grecourt romain-grecourt added this to the 4.0.1 milestone Oct 23, 2023
@romain-grecourt romain-grecourt self-assigned this Oct 23, 2023
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Oct 23, 2023
- Overload ArchetypeV2.generate to expose a variant that accepts externalValues, externalDefaults and directorySupplier
  This allows to avoid creating a new instance of the engine for every project generation

Fixes helidon-io#994
@romain-grecourt romain-grecourt force-pushed the fix-optimized-walker-condition branch from ea631ae to c546864 Compare October 23, 2023 23:17
@romain-grecourt romain-grecourt merged commit 6cc827a into helidon-io:master Oct 23, 2023
21 checks passed
@romain-grecourt romain-grecourt deleted the fix-optimized-walker-condition branch October 23, 2023 23:27
@romain-grecourt romain-grecourt modified the milestones: 4.0.1, 4.0.2 Oct 23, 2023
@romain-grecourt romain-grecourt modified the milestones: 4.0.2, 4.0.1 Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OptimizedWalker should skip conditions if the nested node is skipped
2 participants