Content
View differences
Updated by Alexander Coles 1 day ago
### What
Fix `script/merge-upstream` in openproject-octicons (copied from primer_view_components with view_components-specific
bugs) was repaired, and capture the fork-sync workflow captured as a reusable agent skill.
Branch: `code-maintenance/merge-upstream-skill`
### Status
Changes
- **merge-upstream script fixes — DONE, merged to `main`** (commit `be20037d`):
- Correct upstream remote URL to `primer/octicons` (was `view_components`)
- Tolerate expected merge conflicts (`|| true`) so the script completes under `set -e`
- Auto-detect GNU/BSD `sed` for macOS portability
- Scope changeset rename to the exact `@primer/octicons` token (no longer mangles
`@primer/octicons-react` / `@primer/styled-octicons`)
- Rebuild assets via `yarn install && yarn build`, stage `lib/`, `yarn.lock`
- **`syncing-primer-fork` Add `syncing-primer-fork` skill — in review**, PR #213:
https://github.com/opf/openproject-octicons/pull/213
documenting the merge-parent-of-version-bump procedure, npm/yarn conflict split, and the `merge.ours.driver` prerequisite
### Risk Assessment
- [x] **Low risk** — tooling/docs only, no shipped package code changed
### Checklist
- [x] merge-upstream fixes merged to main
- [ ] Skill PR #213 opened and reviewed and merged
- [ ] Port equivalent fixes to primer_view_components
Fix
bugs) was repaired, and
Branch: `code-maintenance/merge-upstream-skill`
- Correct upstream remote URL to `primer/octicons` (was `view_components`)
`@primer/octicons-react` / `@primer/styled-octicons`)
https://github.com/opf/openproject-octicons/pull/213
- [ ] Skill PR #213