octicons icon indicating copy to clipboard operation
octicons copied to clipboard

Avoid duplicate build folder in @primer/octicons

Open fregante opened this issue 1 year ago • 1 comments

  • Probably fixes https://github.com/primer/octicons/issues/699

In my local tests, running yarn build in lib/octicons_node generates build/build/svg but after this change it generates build/svg, so it should fix the issue.

However the build is a bit of spaghetti between workflows, docker entrypoints.js and multiple package.json, so it's not entirely clear if further work is required (probably not)

fregante avatar Dec 20 '23 03:12 fregante

⚠️ No Changeset found

Latest commit: 5750be4b36fa947f0a0fb6ffc44ee179b7b053b2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

changeset-bot[bot] avatar Dec 20 '23 03:12 changeset-bot[bot]

Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

github-actions[bot] avatar Mar 11 '24 23:03 github-actions[bot]

@broccolinisoup ☺️

fregante avatar Mar 11 '24 23:03 fregante

I'm not sure what's going on with these build failures, but I assume it's because this PR was created from a fork. I just pushed a copy up to origin to see if that fixes things: https://github.com/primer/octicons/pull/1010

camertron avatar Mar 12 '24 16:03 camertron

Merged in #1010

camertron avatar Mar 12 '24 22:03 camertron