Skip to content

Conversation

@krassowski
Copy link
Member

References

Fixes #16730

This should help with updating snapshots in #16866

Code changes

Allows the galata snapshot update job to run if the comment author is owner/member/collaborator (in addition to allowing to update snapshots if the PR author is owner/member/collaborator).

User-facing changes

None

Backwards-incompatible changes

None

@krassowski krassowski added this to the 4.2.x milestone Oct 16, 2024
@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski krassowski changed the title Fix update snapshot workflows when triggered by comment on first-time contributor Fix job updating snapshots when triggered by comment on first-time contributor PR Oct 16, 2024
Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@jtpio jtpio merged commit 80a25c6 into jupyterlab:main Oct 16, 2024
@jtpio
Copy link
Member

jtpio commented Oct 16, 2024

Do we need to backport to 4.2.x? Normally this workflow should run from the main branch?

@krassowski krassowski modified the milestones: 4.2.x, 4.3.0 Oct 16, 2024
@krassowski
Copy link
Member Author

I think this is right, I changed the milestone.

@krassowski krassowski deleted the check-for-comment-author-association branch October 16, 2024 12:05
@krassowski
Copy link
Member Author

Confirmed it worked on #16866:

image

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update snapshot workflows sometimes do not run

2 participants