Skip to content

ci: fix pa11y ci and replace config.version_short in package.json#3591

Open
MaxLardenois wants to merge 8 commits into
ouds/mainfrom
ouds/main-ml-fix-pa11y-ci
Open

ci: fix pa11y ci and replace config.version_short in package.json#3591
MaxLardenois wants to merge 8 commits into
ouds/mainfrom
ouds/main-ml-fix-pa11y-ci

Conversation

@MaxLardenois

@MaxLardenois MaxLardenois commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Types of change

  • Non-breaking change
  • Breaking change (fix or feature that would change existing functionality and usage)

Context & Motivation

Description

Checklists

  • I have read the contributing guidelines
  • My change follows the developer guide
  • My change pass all tests
  • My change is compatible with a responsive display
  • I have added tests (Javascript unit test or visual) to cover my changes
  • My change introduces changes to the documentation that I have updated accordingly
    • Title and DOM structure is correct
    • Links have been updated (title changes impact links)
    • CSS for the documentation
  • I have checked all states and combinations of the component with my change
  • I have checked all the impacts for the other components and core behavior (grid, reboot, utilities)

Checklist (for Core Team only)

  • The changes need to be in the migration guide
  • The changes are well displayed in Storybook (be careful if example order has changed for DSM)
  • The changes are compatible with RTL
  • Manually test browser compatibility with BrowserStack (Chrome 120, Firefox 121, Edge 120, Safari 15.6, iOS Safari, Chrome & Firefox on Android)

Progression (for Core Team only)

Live previews

@netlify

netlify Bot commented Jun 5, 2026

Copy link
Copy Markdown

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 166b774
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/6a2675faa9b3bc0008cdc0b7
😎 Deploy Preview https://deploy-preview-3591--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.

@MaxLardenois MaxLardenois changed the title ci: fix pa11y ci ci: fix pa11y ci and replace config.version_short in package.json Jun 5, 2026
@MaxLardenois MaxLardenois marked this pull request as ready for review June 5, 2026 13:39
@boosted-bot boosted-bot moved this from In Progress / Draft to Need Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Jun 5, 2026
@louismaximepiton louismaximepiton requested a review from Copilot June 8, 2026 07:36

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

This PR updates the repository’s accessibility testing tooling (Pa11y CI) and standardizes how the short docs version is stored/read by moving version_short from package.json#config.version_short to a top-level version_short field.

Changes:

  • Move version_short out of config and into a top-level version_short field in the root and package workspaces.
  • Update build/zip-examples.mjs to read pkg.version_short from the root package.json.
  • Bump pa11y-ci / pa11y-ci-reporter-html versions and update Pa11y CI config to specify a Chrome executable path.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/sosh/package.json Move version_short to a top-level field.
packages/orange/package.json Move version_short to a top-level field.
packages/orange-compact/package.json Move version_short to a top-level field.
package.json Add top-level version_short; bump Pa11y CI dependencies.
package-lock.json Lockfile updates corresponding to Pa11y CI dependency bumps.
build/zip-examples.mjs Read version_short from the new top-level field.
build/.pa11yci.json Add chromeLaunchConfig.executablePath for Pa11y CI runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread build/.pa11yci.json
@boosted-bot boosted-bot moved this from Need Dev Review to Need Lead Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Need Lead Dev Review

Development

Successfully merging this pull request may close these issues.

5 participants