-
Notifications
You must be signed in to change notification settings - Fork 71
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
chore: remove unnecessary explicit public
access modifiers
#363
Conversation
Thank you, 🤖 Clarity Release Bot |
The `public` access modifier was inconsistently used. This change makes the code more concise and consistent.
813f28d
to
8cc76e5
Compare
🎉 This PR is included in version 13.9.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
https://sass-lang.com/documentation/at-rules/import#partials This fix prevents warnings when building. part of #363
This can cause the rules to be included more than once in the output when a partial is imported more than once. This happens when using the dark theme. This fix can reduce bundle sizes. part of #363
The above commit references were typos. |
https://sass-lang.com/documentation/at-rules/import#partials This fix prevents warnings when building. part of #363
This can cause the rules to be included more than once in the output when a partial is imported more than once. This happens when using the dark theme. This fix can reduce bundle sizes. part of #363
Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed PRs after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary. |
The
public
access modifier was inconsistently used. This change makes the code more concise and consistent.PR Checklist
PR Type
Code style update
Does this PR introduce a breaking change?
No