Skip to content

QE-252: update build number validation - #1030

Merged
sarahboyd merged 1 commit into
mainfrom
syoung/ci-build-number-schema
Oct 15, 2026
Merged

sarahboyd merged 1 commit into
mainfrom
syoung/ci-build-number-schema

Conversation

@sarahboyd

Copy link
Copy Markdown
Contributor

LMS CI builds have names like 20.26.11-ci16663, but report validation currently only accepts dotted values like 20.26.11.16663. This adds support for the CI format in schemas v3 and v4 so reporting can preserve the distinction between CI and release builds.

The CI suffix accepts an unpadded number from 1–99999, matching the LMS producer. Existing dotted validation is unchanged, and v1/v2 are left as-is since only v3/v4 are in use. Also adds coverage for both formats and updates the format docs.

This needs to be released and picked up by the uploader before LMS switches its reporting input to versionName.

@sarahboyd

Copy link
Copy Markdown
Contributor Author

/jira --points 1 QE-252

@github-actions github-actions Bot added documentation Improvements or additions to documentation javascript Pull requests that update JavaScript code labels Oct 15, 2026
@sarahboyd sarahboyd changed the title update build number validation QE-252: update build number validation Oct 15, 2026
@sarahboyd
sarahboyd marked this pull request as ready for review October 15, 2026 14:58
@sarahboyd
sarahboyd requested a review from a team as a code owner October 15, 2026 14:58
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report

Report

Report Validation

jest 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

mocha 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

node 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

playwright 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

@web/test-runner 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

webdriverio 🟢

Schema: PASSED 🟢
Contents: PASSED 🟢

Report

@sarahboyd
sarahboyd enabled auto-merge (squash) October 15, 2026 14:58

@Dan-DeAraujo Dan-DeAraujo 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.

👍

@sarahboyd
sarahboyd merged commit 4523eae into main Oct 15, 2026
8 checks passed
@sarahboyd
sarahboyd deleted the syoung/ci-build-number-schema branch October 15, 2026 18:27
@d2l-github-release-tokens

Copy link
Copy Markdown

🎉 This PR is included in version 6.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@d2l-github-release-tokens d2l-github-release-tokens Bot added the released Pull requests that have been released label Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation javascript Pull requests that update JavaScript code released Pull requests that have been released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants