Skip to content

Remove Avatar component from @primer/styled-react#7685

Merged
jonrohan merged 3 commits intomainfrom
remove_styled_react/avatar
Mar 25, 2026
Merged

Remove Avatar component from @primer/styled-react#7685
jonrohan merged 3 commits intomainfrom
remove_styled_react/avatar

Conversation

@jonrohan
Copy link
Member

@jonrohan jonrohan commented Mar 19, 2026 •
edited
Loading

Summary

Remove the Avatar component and its related exports from the @primer/styled-react package. There are no more uses found https://primer-query.githubapp.com/?query=package%3A%22%40primer%2Fstyled-react%22+name%3A%22Avatar%22

Changes

  • Remove Avatar and AvatarProps exports from packages/styled-react/src/index.tsx
  • Delete packages/styled-react/src/components/Avatar.tsx
  • Remove Avatar import and test from packages/styled-react/src/__tests__/primer-react.browser.test.tsx
  • Update exports snapshot in packages/styled-react/src/__tests__/__snapshots__/exports.test.ts.snap

@jonrohan jonrohan requested a review from a team as a code owner March 19, 2026 19:59
@jonrohan jonrohan requested review from TylerJDev and Copilot March 19, 2026 19:59
@changeset-bot
Copy link

changeset-bot bot commented Mar 19, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: f68e46d

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

This PR includes changesets to release 1 package
Name Type
@primer/styled-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 Mar 19, 2026
@jonrohan
Remove Avatar component from @primer/styled-react
fb0456d 
This change removes the Avatar component from the @primer/styled-react package.
Copilot AI reviewed Mar 19, 2026
View reviewed changes
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 Avatar wrapper component from the @primer/styled-react package, including its public exports and related test/snapshot coverage.

Changes:

  • Removed Avatar and AvatarProps exports from packages/styled-react/src/index.tsx.
  • Deleted the packages/styled-react/src/components/Avatar.tsx wrapper implementation.
  • Removed the Avatar sx support test and updated the exports snapshot accordingly.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

File Description
packages/styled-react/src/index.tsx Drops Avatar / AvatarProps from the public API exports.
packages/styled-react/src/components/Avatar.tsx Deletes the styled-react wrapper around @primer/react’s Avatar.
packages/styled-react/src/tests/primer-react.browser.test.tsx Removes the Avatar supports sx prop test and import.
packages/styled-react/src/tests/snapshots/exports.test.ts.snap Updates the exports snapshot to no longer include Avatar.

@github-actions github-actions bot requested a deployment to storybook-preview-7685 March 19, 2026 20:06 Abandoned
@jonrohan jonrohan enabled auto-merge March 19, 2026 20:14
francinelucca
francinelucca approved these changes Mar 25, 2026
View reviewed changes
Copy link
Member

@francinelucca francinelucca left a comment

Choose a reason for hiding this comment

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

🎉

@jonrohan jonrohan added the integration-tests: skipped manually Changes in this PR do not require an integration test label Mar 25, 2026
@jonrohan jonrohan added this pull request to the merge queue Mar 25, 2026
@jonrohan jonrohan removed this pull request from the merge queue due to a manual request Mar 25, 2026
@jonrohan jonrohan removed the integration-tests: skipped manually Changes in this PR do not require an integration test label Mar 25, 2026
@primer-integration
Copy link

primer-integration bot commented Mar 25, 2026

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

@jonrohan jonrohan added the integration-tests: skipped manually Changes in this PR do not require an integration test label Mar 25, 2026
@primer-integration
Copy link

primer-integration bot commented Mar 25, 2026

⚠️ Integration PR Outdated

This integration PR does not have the latest commit from the primer/react PR.

Integration PR references: fb0456d580c41bd203600ecf6e37b90a013abe31
Latest commit on primer/react PR: f68e46dda84cae9c5b5eea9f58922ca6a66986f4

Please update the integration PR to reference the latest commit from the primer/react PR before reviewing workflow results.

@jonrohan jonrohan merged commit 1c7c598 into main Mar 25, 2026
52 of 53 checks passed
@jonrohan jonrohan deleted the remove_styled_react/avatar branch March 25, 2026 16:45
@primer primer bot mentioned this pull request Mar 25, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@francinelucca francinelucca francinelucca approved these changes

@TylerJDev TylerJDev Awaiting requested review from TylerJDev TylerJDev is a code owner automatically assigned from primer/engineer-reviewers

Assignees

No one assigned

Labels

integration-tests: skipped manually Changes in this PR do not require an integration test staff Author is a staff member

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jonrohan @francinelucca