ci: switch to Dart Analyzer GitHub action - Githubissues

csells / go_router

The purpose of the go_router for Flutter is to use declarative routes to reduce complexity, regardless of the platform you're targeting (mobile, web, desktop), handling deep linking from Android, iOS and the web while still allowing an easy-to-use developer experience.

https://gorouter.dev

441 stars 96 forks source link

ci: switch to Dart Analyzer GitHub action #313

Closed Salakar closed 2 years ago

Salakar commented 2 years ago

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)

This adds the new Dart Analyzer action that we've built - which adds in line annotations on PR files for review visibility:

CI itself will also still fail on analyzer problems:

:arrow_heading_down: What is the current behavior?

N/A

:new: What is the new behavior (if this is a feature change)?

N/A

:boom: Does this PR introduce a breaking change?

N/A

:bug: Recommendations for testing

N/A

:memo: Links to relevant issues/docs

https://github.com/invertase/github-action-dart-analyzer

:thinking: Checklist before submitting

[ ] I made sure all projects build.
[ ] I updated CHANGELOG.md to add a description of the change.
[ ] I updated the relevant documentation.
[ ] I updated the relevant code example.
[ ] I rebased onto current master.