-
Notifications
You must be signed in to change notification settings - Fork 16
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
Format Vala code #41
Format Vala code #41
Conversation
8680b50
to
c5afa53
Compare
# Vala | ||
# ./Workbench/build-aux/fun workbench-cli ci vala demos/**/*.vala | ||
# Blueprint | ||
find demos/ -type f -name "*blp" -print0 | xargs -0 ./build-aux/fun blueprint-compiler format | ||
# ./Workbench/build-aux/fun workbench-cli ci blueprint demos/**/*.blp | ||
# CSS | ||
# ./Workbench/build-aux/fun workbench-cli ci css demos/**/*.css | ||
# Rust | ||
./Workbench/build-aux/fun rustfmt --check --edition 2021 demos/**/*.rs | ||
# Python | ||
./Workbench/build-aux/fun black --check demos/**/*.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I commented those that do not pass yet.
I will fix the demos in a subsequent PR
workbench-cli already uncovered quite a few issues 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does "do not pass" mean? Isn't all vala code formatted with the formatter, so linting should succeed, or is the linter or formater not 100% doing its job? (So that would mean fixing the linter/formater?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
look at the code for the context of the comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be alright. I am just going to trust the formatter, from a quick look its mostly indentation anyways.
This is using an "experimental" Workbench branch which exposes a CLI (sort of headless Workbench)
workbenchdev/Workbench#858
Formatted with