actions-flutter-pub-publisher
actions-flutter-pub-publisher copied to clipboard
Not to check credentials when dry run only
When we run dry run on forks, it always checks credentials on the forked repo, which causes checks to fail since the forked repo typically does not have a secret of the pub.
It's better we bypass verify steps when users run dry run only.
Not really, what if someone want to run dry-run: true
to check if the credentials are valid?
Better we create a new option like dry-run-bypass-credentials: true
rather than modifying this current behavior.
As I concern, credentials doesn't do anything during the dry run. So it should not be checked.
Oh I see now. In this case why not just create an empty repo var?
Not sure if you're indicating a solution.
When forks are submitting PRs, we might use dry-run for them. Since their repo is missing the credential, checks will failed immediately because the credential check is placed at the first place.