Skip to content
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

fix CI, add more checks #304

Merged
merged 23 commits into from
Dec 19, 2022

Conversation

lukepighetti
Copy link
Contributor

@lukepighetti lukepighetti commented Dec 19, 2022

This adds CI checks and ensures that the codebase passes

Uses flutter instead of dart because the code coverage command is flaky on dart and it allows us to check the example as well.

Also commits formatted code back to the repo (if a contributor forgets to format).

Also completes code coverage reporting with an action that comments simple coverage reports removed in 98e1c14 due to failing, perhaps due to permissions on this repo?

@lukepighetti lukepighetti marked this pull request as draft December 19, 2022 12:51
@lukepighetti lukepighetti marked this pull request as ready for review December 19, 2022 14:18
@lukepighetti lukepighetti changed the title dec 2022 fix CI, add more checks Dec 19, 2022
@escamoteur escamoteur merged commit 2ac359a into fluttercommunity:master Dec 19, 2022
@lukepighetti lukepighetti deleted the lukepighetti/dec-2022 branch December 19, 2022 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants