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

Reduce access_level requirements when using --dry-run #449

Closed
thompson-shaun opened this issue Oct 1, 2022 · 2 comments · Fixed by #452
Closed

Reduce access_level requirements when using --dry-run #449

thompson-shaun opened this issue Oct 1, 2022 · 2 comments · Fixed by #452

Comments

@thompson-shaun
Copy link
Contributor

thompson-shaun commented Oct 1, 2022

My team has a few projects where we require doing a --dry-run to obtain nextRelease information in an MR. Ideally we could use a token with reduced scope (say just read_repository and read_api).

As a test this snippet in verify.js worked well enough to pass local testing.

if(context.options.dryRun && ((projectAccess && projectAccess.access_level >= 10) || (groupAccess && groupAccess.access_level >= 10)))
{
  ...
}
else if(...existing check...)

Ultimately it would be great if #156 would get closed out by GitLab adding some functionality to adjust the CI_JOB_TOKEN scope.

@fgreinacher
Copy link
Contributor

This sounds like a very useful feature to me! PRs welcome :)

@github-actions
Copy link

github-actions bot commented Nov 3, 2022

🎉 This issue has been resolved in version 9.5.0 🎉

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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants