Skip to content

GAUD-10045: removed golden/ auto approval#7002

Merged
maxkrivodonova merged 1 commit into
mainfrom
mkrivodonova/GAUD-10045-remove-golden-auto-approval
May 20, 2026
Merged

GAUD-10045: removed golden/ auto approval#7002
maxkrivodonova merged 1 commit into
mainfrom
mkrivodonova/GAUD-10045-remove-golden-auto-approval

Conversation

@maxkrivodonova
Copy link
Copy Markdown
Contributor

https://desire2learn.atlassian.net/browse/GAUD-10045

From discussion in above jira task. Right now the repo is set up for PRs changing golden/ to be auto approved, but that folder doesn't exist at the root. Even if it does nothing, it sort of implies changes to goldens should be auto approved. So I'm removing that line.

Guide to set up auto approval of certain PRs

@maxkrivodonova maxkrivodonova requested a review from a team as a code owner May 20, 2026 13:33
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/core/pr-7002/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

Copy link
Copy Markdown
Member

@dlockhart dlockhart left a comment

Choose a reason for hiding this comment

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

Thanks! Yeah this is leftover from the old vdiff framework when goldens were stored here.

@maxkrivodonova maxkrivodonova merged commit 698aae1 into main May 20, 2026
14 of 17 checks passed
@maxkrivodonova maxkrivodonova deleted the mkrivodonova/GAUD-10045-remove-golden-auto-approval branch May 20, 2026 14:16
@d2l-github-release-tokens
Copy link
Copy Markdown

🎉 This PR is included in version 3.252.1 🎉

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.

3 participants