Pin Tutor to known-good commit for master CI matrix entry#832
Closed
asadali145 with Copilot wants to merge 3 commits into
Closed
Pin Tutor to known-good commit for master CI matrix entry#832asadali145 with Copilot wants to merge 3 commits into
master CI matrix entry#832asadali145 with Copilot wants to merge 3 commits into
Conversation
Copilot
AI
changed the title
[WIP] Update CI to pin Tutor to specific commit
Pin Tutor to known-good commit for Jul 23, 2026
master CI matrix entry
Contributor
There was a problem hiding this comment.
Pull request overview
Pins the Tutor version used by the GitHub Actions CI matrix for the master Open edX branch to a specific commit, avoiding breakages from tracking Tutor’s moving main branch and restoring reproducible CI runs.
Changes:
- Replace
git clone --branch=mainwith a plain clone plus an explicitgit checkout <commit>for themastermatrix entry. - Add an inline comment documenting the reason for the pin (upstream packaging regression).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI for the
mastermatrix entry was tracking Tutormainlive, which picked up a packaging toolchain regression (introduced inoverhangio/tutor@f9c2a39) that causesmysqlclient==2.2.8to fail building withValueError: invalid pyproject.toml config: project.license.Change
Pin Tutor to commit
859b5bc394565de25abbee7e83f4d1dc5ae2f505(known-good, pre-regression) instead of cloning--branch=main:This restores reproducibility for the
masterjob without changing behavior for theteakorulmomatrix entries. The pin should be relaxed once the upstream regression is resolved in Tutor.