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

The guarded namespace can use when the mixin is invoked #3725

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lumburr
Copy link
Contributor

@lumburr lumburr commented May 7, 2022

What: fix #3407

Why:
If a namespace has a guard, mixins defined by it are used only if the guard condition returns true.The behavior of the variables should also be consistent.
image

How: In matchcondition we should let the condition evaluate.

Checklist:

  • Documentation
  • Added/updated unit tests
  • Code complete

@lumburr lumburr changed the title Fix 3407 Fix guarded namespaced no pass when true May 9, 2022
@lumburr lumburr changed the title Fix guarded namespaced no pass when true The guarded namespace can use when the mixin is invoked May 9, 2022
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.

Guarded namespace unavailable when calling mixin with ruleset
1 participant