k-paxian / dart-package-publisher

Action to Publish Dart / Flutter Package To https://pub.dev When you need to publish a package, just bump the version in pubspec.yaml
MIT License
66 stars 10 forks source link

Job doesn't fail when dryRunOnly is true #26

Closed gabsn closed 2 years ago

gabsn commented 2 years ago

When dryRunOnly: true and the publishing fails, the job doesn't fail. I would expect it to fail.

My usecase is to dry run the publish job for the pull request and if everything is fine merge it on main and have it actually published.

k-paxian commented 2 years ago

fixed in master