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

move ember-deep-tracked dep to showcase #1950

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Conversation

Dhaulagiri
Copy link
Contributor

📌 Summary

AFAICT that is only used in the showcase but was moved to a component dep in a7206c8

Copy link

vercel bot commented Feb 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Feb 14, 2024 3:13pm
hds-website ✅ Ready (Inspect) Visit Preview Feb 14, 2024 3:13pm

Copy link
Member

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, I stumbled upon it at rebase, but for some reason I thought it was used inside the Table component

@Dhaulagiri Dhaulagiri merged commit b23092d into main Feb 14, 2024
4 of 6 checks passed
@Dhaulagiri Dhaulagiri deleted the br-deep-tracking-value branch February 14, 2024 15:08
@alex-ju
Copy link
Member

alex-ju commented Feb 14, 2024

Also hard to spot in the diff – thanks for catching that!

@Dhaulagiri
Copy link
Contributor Author

Indeed, I obviously didn't notice it when I had reviewed it 👀 . Just something I stumbled across poking around in the codebase this morning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants