Skip to content

chore: Update release process with latest updates#3525

Open
louismaximepiton wants to merge 7 commits into
ouds/mainfrom
ouds/main-lmp-fix-after-release
Open

chore: Update release process with latest updates#3525
louismaximepiton wants to merge 7 commits into
ouds/mainfrom
ouds/main-lmp-fix-after-release

Conversation

@louismaximepiton

@louismaximepiton louismaximepiton commented Apr 17, 2026

Copy link
Copy Markdown
Member

Following the release just made.
Merge Orange-OpenSource/ouds-web-doc#17 in the same time.

Closes #3530.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Comment thread stories/create-stories-from-doc.js Outdated
Comment thread package.json
Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md
@netlify

netlify Bot commented Apr 17, 2026

Copy link
Copy Markdown

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 33cffdb
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/6a3bd7bbd798a200083969c0
😎 Deploy Preview https://deploy-preview-3525--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md
Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md Outdated
Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md Outdated
Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md Outdated
Comment thread .github/PULL_REQUEST_TEMPLATE/ouds-release-template.md Outdated
@boosted-bot boosted-bot moved this from Need Dev Review to Dev Review In Progress in 🟣 [Orange-Boosted-Bootstrap] PRs Board Apr 21, 2026
Co-authored-by: Maxime Lardenois <maxime.lardenois@orange.com>

@MaxLardenois MaxLardenois left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just maybe add the algolia link and it's good

@boosted-bot boosted-bot moved this from Dev Review In Progress to Need Lead Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Apr 22, 2026
@louismaximepiton louismaximepiton linked an issue Apr 29, 2026 that may be closed by this pull request
2 tasks
- [ ] 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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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

Comment thread package.json
"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",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Need Lead Dev Review

Development

Successfully merging this pull request may close these issues.

Remove references to Netlify on our prod

5 participants