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.

Sign up for GitHub

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

Jump to bottom

Format all source files with buildifier #90

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Format all source files with buildifier #90

wants to merge 3 commits into from

Conversation

bartekpacia
Copy link

@bartekpacia bartekpacia commented Dec 28, 2023 •
edited
Loading

I also updated GH workflows and added enforcing formatting to CI.

Apart from that, I just run buildifier -r . in root directory.

@bartekpacia bartekpacia force-pushed the format_with_buildifier branch from 80029b9 to c6c704a Compare December 28, 2023 17:13
@SpencerC
Copy link

SpencerC commented Jan 9, 2024

Oops just saw this! Added #91 and #92 that do the same thing. @cbracken looks like the choice is yours. 😝

@bartekpacia
Copy link
Author

bartekpacia commented Jan 9, 2024 •
edited
Loading

Hahah, nice to see someone else being interested in Bazel for Dart! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bartekpacia @SpencerC