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/#90 - matching the coding rules in order to get a better pub.dev score #99

Merged
merged 2 commits into from
Feb 12, 2021
Merged

Conversation

monsieurtanuki
Copy link
Contributor

Impacted files:

  • analysis_options.yaml: added a reference to pedantic, with limits
  • pubspec.yaml: added a reference to pedantic: ^1.9.2 for code analysis
  • tons of dart files: minor fixes in order to match the coding rules

…score

Impacted files:
* `analysis_options.yaml`: added a reference to `pedantic`, with limits
* `pubspec.yaml`: added a reference to `pedantic: ^1.9.2` for code analysis
* tons of `dart` files: minor fixes in order to match the coding rules
@MohamedFBoussaid
Copy link
Member

@monsieurtanuki Thanks, this will improve the score.
I saw that the flutter analyser on Github actions here : https://github.com/monsieurtanuki/openfoodfacts-dart/runs/1881588895?check_suite_focus=true , other things that can be improved :
info • Unnecessary new keyword • example/lib/main.dart:134:11 • unnecessary_new
info • Unnecessary new keyword • lib/utils/HttpHelper.dart:65:27 • unnecessary_new

@monsieurtanuki
Copy link
Contributor Author

@MohamedFBoussaid Sorry about that, I've just fixed both lines.

Actually as far as I understand that may not improve the score for the moment, as stated in #90 (about SearchResult.dart). But there's a bit of confusion about which criteria are used in the pub.dev score.

@MohamedFBoussaid
Copy link
Member

@monsieurtanuki thanks, yeah the analyzer is a bit confusing on pub.dev

@monsieurtanuki monsieurtanuki merged commit 81c297f into openfoodfacts:master Feb 12, 2021
@monsieurtanuki monsieurtanuki deleted the fix/#90 branch February 15, 2021 08: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.

None yet

2 participants