Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paver Removal 2.5/3 #36046

Merged
merged 3 commits into from
Dec 18, 2024
Merged

Conversation

kdmccormick
Copy link
Member

@kdmccormick kdmccormick commented Dec 18, 2024

There's some insanity in how edx-proctoring provides worker scripts (writeup to come later) that is blocking the full "Paver Removal 3/3" PR. It's still TBD how long we will let that issue block the Paver removal. But, in the meantime, I'd like to merge that PR's non-breaking doc updates and npm run test* improvements.

For description, supporting info, and test instructions, see #34832 , but just ignore all the "breaking changes".

This is technically a breaking change, but these commands were added a
week ago and were not yet documented, so I'm not worried about breaking
anyone's workflow with this commit.
@kdmccormick kdmccormick requested a review from feanil December 18, 2024 15:49
@kdmccormick kdmccormick mentioned this pull request Dec 18, 2024
Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

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

In case you still need to land this first.

@kdmccormick kdmccormick reopened this Dec 18, 2024
@kdmccormick kdmccormick enabled auto-merge (rebase) December 18, 2024 16:06
@kdmccormick kdmccormick mentioned this pull request Dec 18, 2024
@kdmccormick kdmccormick merged commit 29d4c69 into openedx:master Dec 18, 2024
80 of 96 checks passed
@kdmccormick kdmccormick deleted the kdmccormick/paver-rm-2.5 branch December 18, 2024 16:26
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants