-
Notifications
You must be signed in to change notification settings - Fork 45
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
simple code QA for a better scoring #379
Conversation
oh we are using a line-length with 120 in our checks. It looks like, it's more as the checker for the pub.dev-scoring is using. |
just only 5 points more ;-) |
LGTM! It would be great to modify https://github.com/fzyzcjy/flutter_convenient_test/blob/master/.all-contributorsrc correspondingly, o/w the next time it will override the readme. |
No worries, we can format it to be 80. Indeed I have done so for https://github.com/fzyzcjy/flutter_rust_bridge for the pub scores (though I personally prefer 120). |
replace http with https
this check will be failed if we format it to be 80:
|
format should be 80 (default)
grrr ... this was a huge refactoring in a lot of files. the score-page (https://pub.dev/packages/convenient_test/score) displays formatting warnings in only 2 files. But after I started the reformatting in these 2 files, the CI failed. |
Just change the CI :) |
@all-contributors please add @meltzow for code |
"we" want to have a full scoring. see https://pub.dev/packages/convenient_test/score ;-)