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

chore: off-dart 2.3.1 and its new sorted suggestions #3717

Merged
merged 3 commits into from
Feb 21, 2023

Conversation

monsieurtanuki
Copy link
Contributor

Deleted file:

  • tmp_matched_product_v2.dart: now in off-dart

Impacted files:

  • personalized_ranking_model.dart: removed reference to now deleted file tmp_matched_product_v2.dart
  • personalized_ranking_page.dart: removed reference to now deleted file tmp_matched_product_v2.dart
  • Podfile.lock: wtf
  • pubspec.lock: wtf
  • pubspec.yaml: upgraded off-dart to 2.3.1
  • simple_input_text_field.dart: now using the new getSuggestions method that sorts the result

What

Screenshot

Capture d’écran 2023-02-19 à 09 00 41

# Conflicts:
#	packages/smooth_app/pubspec.lock
#	packages/smooth_app/pubspec.yaml
Deleted file:
* `tmp_matched_product_v2.dart`: now in off-dart

Impacted files:
* `personalized_ranking_model.dart`: removed reference to now deleted file `tmp_matched_product_v2.dart`
* `personalized_ranking_page.dart`: removed reference to now deleted file `tmp_matched_product_v2.dart`
* `Podfile.lock`: wtf
* `pubspec.lock`: wtf
* `pubspec.yaml`: upgraded off-dart to 2.3.1
* `simple_input_text_field.dart`: now using the new `getSuggestions` method that sorts the result
@codecov-commenter
Copy link

Codecov Report

Merging #3717 (58d558b) into develop (70bdceb) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #3717      +/-   ##
===========================================
+ Coverage    10.41%   10.45%   +0.03%     
===========================================
  Files          273      272       -1     
  Lines        13795    13746      -49     
===========================================
  Hits          1437     1437              
+ Misses       12358    12309      -49     
Impacted Files Coverage Δ
...pp/lib/data_models/personalized_ranking_model.dart 2.43% <ø> (ø)
...mooth_app/lib/pages/personalized_ranking_page.dart 0.00% <ø> (ø)
...app/lib/pages/product/simple_input_text_field.dart 0.00% <0.00%> (ø)
...pp/lib/pages/product/common/product_refresher.dart 1.75% <0.00%> (-0.21%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@M123-dev M123-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼 👍🏼 👍🏼

@monsieurtanuki monsieurtanuki merged commit e5bf964 into openfoodfacts:develop Feb 21, 2023
@monsieurtanuki
Copy link
Contributor Author

Thank you @M123-dev for the review!
For the record my browser didn't recognize your latest emoji, I hope it's decent ;)

@M123-dev
Copy link
Member

It is a thumbs up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants