Skip to content

Formatting Dart code + enforcing it on CI. #4073

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

isoos
Copy link
Collaborator

@isoos isoos commented Jul 3, 2025

Note: I have no idea whether this is good/required for the template files with lots of changes, but for the rest of the files it seems somewhat important to make them consistent with other project workflows where the dart format is prerequisite of the approval.

I've extended the tool to run dart format . after the tool generates the template files.

I've also extended the CI tool to run it before the dart analyzer is run (in the same command block, for simplicity, and also because there is very little reason to run them separately).

@isoos isoos requested a review from sigurdm July 3, 2025 14:41
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.

1 participant