Skip to content

Conversation

@siddharthkp
Copy link
Member

@siddharthkp siddharthkp commented Nov 11, 2024

diff from PR that had to be reverted:

- if (!open) return <>{anchor}</>
+ if (!open) return <LiveRegion>{anchor}</LiveRegion>

The reason I had to revert it is because i added one commit after running integration tests that was faulty 🤦

@siddharthkp siddharthkp self-assigned this Nov 11, 2024
@changeset-bot
Copy link

changeset-bot bot commented Nov 11, 2024

🦋 Changeset detected

Latest commit: 080e0f6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the staff Author is a staff member label Nov 11, 2024
@github-actions
Copy link
Contributor

👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks!

@github-actions github-actions bot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Nov 11, 2024
@github-actions github-actions bot temporarily deployed to storybook-preview-5262 November 11, 2024 16:29 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2024

size-limit report 📦

Path Size
packages/react/dist/browser.esm.js 98.59 KB (+0.02% 🔺)
packages/react/dist/browser.umd.js 99.02 KB (+0.16% 🔺)

@primer-integration
Copy link

primer-integration bot commented Nov 11, 2024

🟢 golden-jobs completed with status success.

@github-actions github-actions bot added integration-tests: failing Changes in this PR cause breaking changes in gh/gh and removed integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm labels Nov 11, 2024
@primer-integration
Copy link

👋 Hi from github/github! Your integration PR is ready: https://github.com/github/github/pull/350750

@github-actions github-actions bot added integration-tests: passing Changes in this PR do NOT cause breaking changes in gh/gh and removed integration-tests: failing Changes in this PR cause breaking changes in gh/gh labels Nov 11, 2024
@siddharthkp siddharthkp marked this pull request as ready for review November 11, 2024 19:14
@siddharthkp siddharthkp requested a review from a team as a code owner November 11, 2024 19:14
@siddharthkp siddharthkp changed the title Revert "Revert "SelectPanel: Prepare for non-anchored variants"" Bring back "SelectPanel: Prepare for non-anchored variants"" Nov 11, 2024
@siddharthkp siddharthkp changed the title Bring back "SelectPanel: Prepare for non-anchored variants"" Bring back #5255 "SelectPanel: Prepare for non-anchored variants" Nov 11, 2024
@broccolinisoup broccolinisoup added this pull request to the merge queue Nov 12, 2024
Merged via the queue into main with commit fca4ec9 Nov 12, 2024
47 checks passed
@broccolinisoup broccolinisoup deleted the revert-5255-revert-5230-selectpanel-use-anchored-position branch November 12, 2024 02:14
@primer primer bot mentioned this pull request Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: passing Changes in this PR do NOT cause breaking changes in gh/gh staff Author is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants