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

Cyclic error combines mixin path and var paths #2820

Open
idoros opened this issue Feb 5, 2023 · 0 comments
Open

Cyclic error combines mixin path and var paths #2820

idoros opened this issue Feb 5, 2023 · 0 comments
Labels
bug Unexpected behavior or exception core Processing and transforming logic

Comments

@idoros
Copy link
Collaborator

idoros commented Feb 5, 2023

In the transform process the passedThough context param is used to track both mixins and build vars for cyclic loops.
The issue causes unintentional diagnostic messages when getting to a cyclic var through a mixin.

playground example - the issue is with the cyclic message contain the mixin tracking:

→ mix from /src/app.st.css 
↪ /src/app.st.css: b 
↪ /src/app.st.css: a 
↻ /src/app.st.css: b

Maybe knowing that the error came through a mixin is not that bad, but this wasn't intended to look like this and the formatted arrows don't look good, maybe indent the cyclic vars section.

@idoros idoros added bug Unexpected behavior or exception core Processing and transforming logic labels Feb 5, 2023
@idoros idoros changed the title Cyclic error combines mixin path var paths Cyclic error combines mixin path and var paths Feb 5, 2023
@idoros idoros mentioned this issue Feb 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected behavior or exception core Processing and transforming logic
Projects
None yet
Development

No branches or pull requests

1 participant