Skip to content

GAUD-10114: upgrade and add test reporting#1005

Merged
dlockhart merged 1 commit into
mainfrom
GAUD-10114/upgrade-add-test-reporting
Jun 5, 2026
Merged

GAUD-10114: upgrade and add test reporting#1005
dlockhart merged 1 commit into
mainfrom
GAUD-10114/upgrade-add-test-reporting

Conversation

@dlockhart
Copy link
Copy Markdown
Member

v6 should be a drop-in replacement.

@dlockhart dlockhart force-pushed the GAUD-10114/upgrade-add-test-reporting branch from 59553b1 to 25968c9 Compare June 5, 2026 14:29
@dlockhart dlockhart force-pushed the GAUD-10114/upgrade-add-test-reporting branch from 25968c9 to 6074013 Compare June 5, 2026 14:31
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this because it wasn't categorizing our tests correctly?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The config file is required, I just missed it in the other repos.

On top of that, the Error: Report path must exists errors that I'm getting in lms-core is because experience is required in the config in v5 but dropped in v6 -- which is why this repo is working fine. I just confirmed with QE that releasing this is safe and should allow downstream repos to not set experience.

@dlockhart dlockhart marked this pull request as ready for review June 5, 2026 14:39
@dlockhart dlockhart requested a review from a team as a code owner June 5, 2026 14:39
@dlockhart dlockhart merged commit 197162b into main Jun 5, 2026
4 checks passed
@dlockhart dlockhart deleted the GAUD-10114/upgrade-add-test-reporting branch June 5, 2026 14:39
@d2l-github-release-tokens
Copy link
Copy Markdown

🎉 This PR is included in version 1.46.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants