chore: Update release process with latest updates#3525
chore: Update release process with latest updates#3525louismaximepiton wants to merge 7 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the repository’s release workflow documentation and helper scripts to reflect the current multi-brand docs/output structure and add convenience commands for packaging/publishing.
Changes:
- Update Storybook story generation to read built docs from the new
_site/<brand>/docs/<version>/...location. - Add root-level npm scripts for packing and publishing the root package and all brand packages.
- Refresh the OUDS release PR template to match the updated release/publish steps.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
stories/create-stories-from-doc.js |
Points Puppeteer to the new built-docs path used for generating Storybook stories. |
package.json |
Adds pack/publish helper scripts for multi-package release operations. |
.github/PULL_REQUEST_TEMPLATE/ouds-release-template.md |
Updates the human release checklist to use the new scripts and tagging steps. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Co-authored-by: Maxime Lardenois <maxime.lardenois@orange.com>
MaxLardenois
left a comment
There was a problem hiding this comment.
Just maybe add the algolia link and it's good
| - [ ] check every `index.html` used as redirections to redirect to the new release | ||
| - [ ] run `npm run storybook-build` from `Orange-Boosted-Bootstrap` (be careful that `release-sri` doesn't update `config.yml`s) | ||
| - [ ] remove `storybook` folder from `ouds-web-doc` | ||
| - [ ] copy `Orange-Boosted-Bootstrap/_site` to the `ouds-web-doc#main` branch |
There was a problem hiding this comment.
We are working with branches in ouds-web-doc, it might be understood as commiting directly on main. I think the idea is to start from main but we should still have a review
| "docs-serve-only": "sirv _site --no-clear --port 9001", | ||
| "lockfile-lint": "lockfile-lint --allowed-hosts npm --allowed-schemes https: --empty-hostname false --type npm --path package-lock.json", | ||
| "update-deps": "ncu -u -x eslint,eslint-config-xo,eslint-plugin-unicorn,karma-browserstack-launcher,karma-rollup-preprocessor,sass", | ||
| "pack": "npm pack && cd packages/orange-compact && npm pack && cd ../sosh && npm pack && cd ../orange && npm pack", |
There was a problem hiding this comment.
Having each script in séquence is intended to stop is an error occurs ? We could have launched each pack and publish in parallel with npm-run-all
Co-authored-by: Vincent Prothais <vincent.prothais@orange.com>
Following the release just made.
Merge Orange-OpenSource/ouds-web-doc#17 in the same time.
Closes #3530.