-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix(styles): fix the auro capitalize function #151
fix(styles): fix the auro capitalize function #151
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. My onlny nit-pick is to append the issue ID to the commit.
fix(styles): fix the auro capitalize function #148
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything works great! Thanks @marnel-mangrubang and @chethanayasa
I tested this with a local version of the Auro docsite and all green lights!
@marnel-mangrubang and @chethanayasa please make sure to agree to the CLA. It's part of the OSS agreement with this project. |
21edb1f
to
cdfaac4
Compare
🎉 This PR is included in version 4.3.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Alaska Airlines Pull Request
This PR is to address the AuroDocSite failing to build with the use of version 4.3.0 of WCSS. Due to the addition of numbers in the
$auro-spacing-options
default value, theauro_capitalize
function inside of utilityFunctions would throw an error because its always expecting the argument type to be string. This PR is to unblock another story on our board relating to the AuroDocSite repo.Fixes: #148
Summary:
auro_capitalize
function to only capitalize if string type. If number type, return string version of that number without capitalizing.auro_map-deep-get
to always be string so that it can find the value of the matching string key from the collection of tokens.spacing.spec
to account for number values.Type of change:
Please delete options that are not relevant.
Checklist:
By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team