Skip to content

feat: add octicons-react-symbols package#8029

Draft
joshblack wants to merge 17 commits into
mainfrom
feat/add-octicons-react-symbols-exploration
Draft

feat: add octicons-react-symbols package#8029
joshblack wants to merge 17 commits into
mainfrom
feat/add-octicons-react-symbols-exploration

Conversation

@joshblack

Copy link
Copy Markdown
Member

Closes #

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

@changeset-bot

changeset-bot Bot commented Jun 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9c5c2ab

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

This PR includes changesets to release 1 package
Name Type
@primer/octicons-react-symbols Minor

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 Jun 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Action required

👋 Hi, this pull request contains changes to the source code that github/github-ui depends on. If you are GitHub staff, test these changes with github/github-ui using the integration workflow. If this doesn't work, you can also use the original workflow here. Check the integration testing docs for step-by-step instructions. Or, apply the integration-tests: skipped manually label to skip these checks.

To publish a canary release for integration testing, apply the Canary Release label 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 Jun 22, 2026
@primer

primer Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🤖 Lint issues have been automatically fixed and committed to this PR.

@github-actions github-actions Bot requested a deployment to storybook-preview-8029 June 22, 2026 20:40 Abandoned
@primer

primer Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🤖 Lint issues have been automatically fixed and committed to this PR.

@github-actions github-actions Bot requested a deployment to storybook-preview-8029 June 22, 2026 20:45 Abandoned
@primer

primer Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🤖 Lint issues have been automatically fixed and committed to this PR.

@github-actions github-actions Bot requested a deployment to storybook-preview-8029 June 22, 2026 22:03 Abandoned
@joshblack

Copy link
Copy Markdown
Member Author

@copilot fix CI

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@copilot fix CI

Fixed in a39dae7. I addressed the CI failures by fixing lint/type-check issues, making Storybook build @primer/octicons-react-symbols before @primer/react Storybook, and adding a changeset.

@github-actions github-actions Bot requested a deployment to storybook-preview-8029 June 24, 2026 14:20 Abandoned
@joshblack

Copy link
Copy Markdown
Member Author

@copilot fix CI

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@copilot fix CI

Addressed in 4f4ed4f. I fixed the CI failure by updating OcticonSymbols props typing to correctly accept children, which resolves the TS2322 build/type-check error in @primer/octicons-react-symbols.

@github-actions github-actions Bot requested a deployment to storybook-preview-8029 June 24, 2026 21:52 Abandoned
@primer

primer Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

🤖 Lint issues have been automatically fixed and committed to this PR.

@primer

primer Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

🤖 Lint issues have been automatically fixed and committed to this PR.

@joshblack

Copy link
Copy Markdown
Member Author

@copilot fix CI failures

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

@copilot fix CI failures

Fixed in 5b21d22. I resolved the CI failures by restoring JS output for @primer/octicons-react-symbols builds, making lint:npm build before publint, and updating the package’s local React devDependency ranges to avoid the React 19 test mismatch.

"scripts": {
"build": "node script/build.ts && rolldown -c",
"clean": "rimraf dist",
"lint:npm": "npm run build && publint --types",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@copilot instead of explicitly building, can we add build as a dependency to this task in turbo?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Done in 9c5c2ab. I removed the explicit npm run build from lint:npm so the task now relies on Turbo’s lint:npm -> build dependency ordering.

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>
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 staff Author is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants