Skip to content

Conversation

zamoore
Copy link
Contributor

@zamoore zamoore commented Oct 7, 2025

๐Ÿ“Œ Summary

If merged, this PR adds ember-basic-dropdown as a peer dependency.

๐Ÿ› ๏ธ Detailed description

This change addresses a runtime error for consumers of Hds::Form::SuperSelect. The component relies on ember-basic-dropdown, but since it was only a devDependency in HDS, applications that didn't have ember-basic-dropdown installed themselves would encounter a crash. Moving it to a peer dependency ensures that users are notified that they need to install it in order to use HDS.

๐Ÿ”— External links

Jira ticket: HDS-5273


๐Ÿ‘€ Component checklist

๐Ÿ’ฌ Please consider using conventional comments when reviewing this PR.

๐Ÿ“‹ PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

Copy link

vercel bot commented Oct 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
hds-showcase Ready Ready Preview Oct 10, 2025 3:48pm
hds-website Ready Ready Preview Oct 10, 2025 3:48pm

@zamoore zamoore changed the base branch from main to main-5.0.0 October 7, 2025 19:06
@zamoore zamoore added release-candidate Publishes release candidates to npm and removed packages/flight-icons labels Oct 7, 2025
Copy link
Contributor

github-actions bot commented Oct 7, 2025

๐Ÿ“ฆ RC Packages Published

Latest commit: d140998

Published 2 packages

@hashicorp/design-system-components@5.0.0-rc-20251007215008

yarn up -C @hashicorp/design-system-components@rc

@hashicorp/flight-icons@4.0.0-rc-20251007215008

yarn up -C @hashicorp/flight-icons@rc

@zamoore zamoore force-pushed the zamoore/hds-5273/ember-basic-dropdown-peer-dep branch from 6f64b2b to ed140af Compare October 7, 2025 20:03
@zamoore zamoore marked this pull request as ready for review October 7, 2025 20:13
@zamoore zamoore requested a review from a team as a code owner October 7, 2025 20:13
@zamoore zamoore requested a review from a team as a code owner October 7, 2025 21:48
@zamoore zamoore force-pushed the zamoore/hds-5273/ember-basic-dropdown-peer-dep branch from ed140af to d140998 Compare October 7, 2025 21:49
@zamoore zamoore removed the release-candidate Publishes release candidates to npm label Oct 8, 2025
@zamoore zamoore force-pushed the zamoore/hds-5273/ember-basic-dropdown-peer-dep branch from d140998 to ae0578b Compare October 9, 2025 19:45
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.

Left two comments for your consideration

zamoore and others added 2 commits October 10, 2025 11:40
Co-authored-by: Alex <alex-ju@users.noreply.github.com>
Co-authored-by: Alex <alex-ju@users.noreply.github.com>
@zamoore zamoore merged commit 9a42841 into main-5.0.0 Oct 10, 2025
3 of 5 checks passed
@zamoore zamoore deleted the zamoore/hds-5273/ember-basic-dropdown-peer-dep branch October 10, 2025 15:41
zamoore added a commit that referenced this pull request Oct 10, 2025
Co-authored-by: Alex <alex-ju@users.noreply.github.com>
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