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: upgrade - flutter_svg, camera #3945

Merged
merged 6 commits into from
May 13, 2023

Conversation

monsieurtanuki
Copy link
Contributor

@monsieurtanuki monsieurtanuki commented May 8, 2023

What

  • Upgrade of packages flutter_svg and matomo_tracker (minor breaking changes) and camera (no breaking changes)

Impacted files

  • analytics_helper.dart: minor refactoring with new matomo version
  • edit_product_page.dart: minor refactoring with new svg version
  • pubspec.lock: wtf
  • scanner/pubspec.yaml: new svg version
  • smooth_app/pubspec.yaml: new svg, camera and matomo versions
  • smooth_product_card_thanks.dart: minor refactoring with new svg version
  • svg_async_asset.dart: minor refactoring with new svg version
  • svg_cache.dart: minor refactoring with new svg version
  • user_preferences_settings.dart: minor refactoring with new matomo version

Impacted files:
* `analytics_helper.dart`: minor refactoring with new matomo version
* `edit_product_page.dart`: minor refactoring with new svg version
* `pubspec.lock`: wtf
* `scanner/pubspec.yaml`: new svg version
* `smooth_app/pubspec.yaml`: new svg, camera and matomo versions
* `smooth_product_card_thanks.dart`: minor refactoring with new svg version
* `svg_async_asset.dart`: minor refactoring with new svg version
* `svg_cache.dart`: minor refactoring with new svg version
* `user_preferences_settings.dart`: minor refactoring with new matomo version
@monsieurtanuki monsieurtanuki requested a review from a team as a code owner May 8, 2023 13:47
@github-actions github-actions bot added 📈 Analytics We use Sentry and Matomo, with an opt-in system dependencies 🥫 Product page labels May 8, 2023
@teolemon
Copy link
Member

teolemon commented May 8, 2023

Screenshot_20230508-162011.png

Unrelated failures

@monsieurtanuki
Copy link
Contributor Author

Unrelated failures

They seem related actually, if you expand the errors - something like "matomo not initialized".

Copy link
Member

@teolemon teolemon left a comment

Choose a reason for hiding this comment

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

it seems that errors are related to matomo

@github-actions github-actions bot removed the 📈 Analytics We use Sentry and Matomo, with an opt-in system label May 11, 2023
@monsieurtanuki monsieurtanuki changed the title chore: upgrade - flutter_svg, camera, matomo_tracker chore: upgrade - flutter_svg, camera May 11, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 11, 2023

Codecov Report

Merging #3945 (177795d) into develop (72e98df) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #3945      +/-   ##
===========================================
- Coverage    11.00%   10.99%   -0.01%     
===========================================
  Files          265      265              
  Lines        13042    13046       +4     
===========================================
  Hits          1435     1435              
- Misses       11607    11611       +4     
Impacted Files Coverage Δ
..._app/lib/cards/category_cards/svg_async_asset.dart 0.00% <0.00%> (ø)
...smooth_app/lib/cards/category_cards/svg_cache.dart 0.00% <0.00%> (ø)
...ards/product_cards/smooth_product_card_thanks.dart 8.33% <ø> (ø)
...mooth_app/lib/pages/product/edit_product_page.dart 0.45% <0.00%> (-0.01%) ⬇️

📣 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.

👍🏼 👍🏼

@M123-dev M123-dev enabled auto-merge (squash) May 13, 2023 08:28
@M123-dev
Copy link
Member

Somehow automerge won't work as long as @teolemon is requesting changes

@monsieurtanuki
Copy link
Contributor Author

Somehow automerge won't work as long as @teolemon is requesting changes

@teolemon ping

@M123-dev M123-dev merged commit cd04135 into openfoodfacts:develop May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants