-
Notifications
You must be signed in to change notification settings - Fork 41
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): update blueprint #2142
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
50521c2
to
d4f149f
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've had a quick look, but my knowledge about these settings is very limited, so I'll defer to @alex-ju to do a in depth review.
d4f149f
to
fda4e3f
Compare
484f775
to
0c2f6b5
Compare
0c2f6b5
to
a26d1a7
Compare
ffc27f7
to
5f7dc0f
Compare
5f7dc0f
to
4eb5765
Compare
With an eye toward some consistency across the monorepo I excised most of the blueprint-related changes here and got those in #2166. We should be able to rebase this one to get it down to just the TS-related changes and more cleanly review and release that. |
4eb5765
to
be8fd7e
Compare
be8fd7e
to
a7a46ff
Compare
a7a46ff
to
c268fb2
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 think is OK, but please wait an approval from @alex-ju or @Dhaulagiri before merging
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.
Looking good overall! It took a bit more than expected as I wanted to make sure I test with atlas
as well
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 |
Ensures type safety without the need for inline type assertions
6d4fa88
to
8cde778
Compare
📌 Summary
Preparatory PR for #2170
(context: https://hashicorp.slack.com/archives/C06NMSYDZPA/p1717440951790419?thread_ts=1717099148.439879&cid=C06NMSYDZPA)
👀 Component checklist
💬 Please consider using conventional comments when reviewing this PR.