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

feat: add --skip-remote-check option #2426

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thislooksfun
Copy link
Contributor

@thislooksfun thislooksfun commented Apr 30, 2022

This is a first draft of a new option to bypass checking remote access. It only works in dry run mode. There is still some work that needs to be done before it's ready to merge:

  • Write a concise description of the new flag (cli.js#37)
  • Add tests
  • Make run throw instead of returning false? (index.js#63)

I took a first stab at writing some tests for this new flag, but I got lost in the tests and couldn't figure out how to make some of them work. If someone with more experience in this test suite could help out that would be greatly appreciated.

Once this is ready I will opened corresponding PRs on @semantic-release/github and @semantic-release/npm to use this option.

Closes #2232.

This new option will bypass checking remote access. It only works in dry
run mode. This addresses semantic-release#2232.
@codejedi365
Copy link

Any movement on this? I would love to see this come to fruition.

@boxrick
Copy link

boxrick commented Apr 25, 2023

This would also help my current issue!

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.

--dry-run shouldn't test push permissions
4 participants