-
Notifications
You must be signed in to change notification settings - Fork 42
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
feat(ember-flight-icons): convert to typescript & publish types #2170
feat(ember-flight-icons): convert to typescript & publish types #2170
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
f343a26
to
808a7a3
Compare
You must have Developer access to commit code to HashiCorp on Vercel. If you contact an administrator and receive Developer access, commit again to see your changes. Learn more: https://vercel.com/docs/accounts/team-members-and-roles/access-roles#team-level-roles |
808a7a3
to
e016af7
Compare
e016af7
to
a306543
Compare
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.
Left a couple of comments
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.
I see this change also in https://github.com/hashicorp/design-system/pull/2142/files#diff-e31a2dfa439cd2cc8aa9b1e01d10b79db320fd27009fdb3b129ea1bae67aa866 so this means that this branch should be rebased on #2142 no?
a306543
to
35aa878
Compare
Ensures type safety without the need for inline type assertions
35aa878
to
abd1462
Compare
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.
I added an extra commit to ensure type safety without the need for inline type assertions, and rebased again, hope you don't mind. Couldn't find the Vercel deploy culprit (local builds are fine), so I'd say let's merge it in and see if it persists.
6d4fa88
into
feat/embe-flight-icons-bluprint-update
📌 Summary
The main goal of this change is to make sure types are created and published for the Ember Flight Icon component
🔗 External links
Jira ticket: https://hashicorp.atlassian.net/browse/HDS-3315
Previously created PR that was massed up: #2126
👀 Component checklist
💬 Please consider using conventional comments when reviewing this PR.