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.

Sign up for GitHub

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

Jump to bottom

Use pull_request_target to have write access on fork #390

Merged
merged 1 commit into from Sep 6, 2022
Merged

Use pull_request_target to have write access on fork #390

merged 1 commit into from Sep 6, 2022

Conversation

fcollonval
Copy link
Member

Write permission cannot be granted to pull_request event from forks.

This uses pull_request_target to fix the CI jobs.

See documentation

@fcollonval fcollonval added the maintenance Dependencies, build, technical debt, etc. label Sep 6, 2022
@fcollonval fcollonval merged commit 2d9b7aa into jupyterlab:main Sep 6, 2022
@fcollonval fcollonval deleted the bug/fix-ci-permissions branch September 6, 2022 08:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@fcollonval fcollonval

Labels
maintenance Dependencies, build, technical debt, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@fcollonval