Skip to content

Conversation

@pksjce
Copy link
Contributor

@pksjce pksjce commented Sep 16, 2025

Closes #https://github.com/github/primer/issues/5536

Changelog

New

Changed

Removed

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

Copilot AI review requested due to automatic review settings September 16, 2025 05:41
@pksjce pksjce requested a review from a team as a code owner September 16, 2025 05:41
@pksjce pksjce requested a review from siddharthkp September 16, 2025 05:41
@changeset-bot
Copy link

changeset-bot bot commented Sep 16, 2025

🦋 Changeset detected

Latest commit: 0160b5c

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

This PR includes changesets to release 2 packages
Name Type
@primer/react Major
@primer/styled-react Major

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 integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Sep 16, 2025
@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!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the deprecated PointerBox component from the @primer/react package as part of the major release cleanup. The PointerBox was previously deprecated in v38.0.0 and is now being completely removed.

  • Complete removal of PointerBox component including implementation, tests, stories, and documentation
  • Updates to export files and test snapshots to reflect the component removal
  • Documentation updates to remove references to PointerBox

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/react/src/index.ts Removes PointerBox exports from main package exports
packages/react/src/deprecated/index.ts Removes PointerBox exports from deprecated exports
packages/react/src/tests/snapshots/exports.test.ts.snap Updates test snapshots to reflect removed exports
packages/react/src/PointerBox/index.ts Deletes the PointerBox index file
packages/react/src/PointerBox/PointerBox.tsx Deletes the main PointerBox component implementation
packages/react/src/PointerBox/PointerBox.test.tsx Deletes PointerBox unit tests
packages/react/src/PointerBox/PointerBox.stories.tsx Deletes PointerBox Storybook stories
packages/react/src/PointerBox/PointerBox.docs.json Deletes PointerBox documentation metadata
packages/react/script/react-compiler.mjs Removes PointerBox from React compiler unsupported files list
contributor-docs/behaviors.md Updates documentation to remove PointerBox reference

@github-actions github-actions bot requested a deployment to storybook-preview-6857 September 16, 2025 05:45 Abandoned
@github-actions github-actions bot temporarily deployed to storybook-preview-6857 September 16, 2025 05:53 Inactive
@pksjce pksjce enabled auto-merge September 17, 2025 04:13
@pksjce pksjce added this pull request to the merge queue Sep 22, 2025
auto-merge was automatically disabled September 22, 2025 17:37

Pull Request is not mergeable

Merged via the queue into main with commit b9b558e Sep 22, 2025
40 of 41 checks passed
@pksjce pksjce deleted the pk/remove-pointerboc branch September 22, 2025 17:40
@primer primer bot mentioned this pull request Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants