-
Notifications
You must be signed in to change notification settings - Fork 0
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: Switch to analyzer that annotates on the PR #30
Conversation
Needs somebody from @Betterment/better_test_reporter-maintainers to claim domain review Use the shovel operator to claim, e.g.:
|
2 similar comments
Needs somebody from @Betterment/better_test_reporter-maintainers to claim domain review Use the shovel operator to claim, e.g.:
|
Needs somebody from @Betterment/better_test_reporter-maintainers to claim domain review Use the shovel operator to claim, e.g.:
|
Needs somebody from @Betterment/better_test_reporter-maintainers to claim domain review Use the shovel operator to claim, e.g.:
|
<<domain tafn |
@CelticMajora needs to incorporate feedback from @samandmoore. Bump when done. |
4 similar comments
@CelticMajora needs to incorporate feedback from @samandmoore. Bump when done. |
@CelticMajora needs to incorporate feedback from @samandmoore. Bump when done. |
@CelticMajora needs to incorporate feedback from @samandmoore. Bump when done. |
@CelticMajora needs to incorporate feedback from @samandmoore. Bump when done. |
bump nanda |
Needs @samandmoore to provide domain review When you finish a round of review, be sure to say you've finished or sign off on the PR, e.g.:
If you're too busy to review, unclaim the PR, e.g.:
|
Needs @samandmoore to provide domain review When you finish a round of review, be sure to say you've finished or sign off on the PR, e.g.:
If you're too busy to review, unclaim the PR, e.g.:
|
@CelticMajora I just pushed up a test commit with issues to see what it looks like 😄 |
domainlgtm okay it seems to work well! I'm into it |
okay! domainlgtm I reverted my commits in the wrong order 😮💨 so I fixed it |
📰 Summary of changes
Switches out our analyze step for one produces by invertase that will annotate the issues on the PR itself
https://github.com/invertase/github-action-dart-analyzer
Reviewers
/domain @Betterment/better_test_reporter-maintainers
/no-platform