ecosystem
ecosystem copied to clipboard
have a way to configure publishing dry run checks
Right now some packages have intentional dart pub publish --dry-run warnings. firehose fails the build on the dry-run checks; we may want some additional configurability here. See https://github.com/dart-lang/test/pull/1923 for context.
Some ideas are:
- for pub publish dry-run to support some
ignore: xxxsyntax in pubspec files - have a way for a repo to configure that dry run checks shouldn't be run for specific packages
- have a way for
firehoseto distinguish dry-run warnings vs errors and not fail builds for warnings (as pub is currently, any warnings sets an exit code of65)
Fwiw, if the ignore syntax is feasible it would be my preference. This would in general make publishing safer on packages that have known warnings (only new warnings would appear), and document those warnings that are known to exist.
One other way we could configure this to ignore publishing warnings is by supporting something like a publishing-ignore-warnings label; that's something a repo maintainer could add to a PR if they knew the particular warnings wasn't relevant.