-
-
Notifications
You must be signed in to change notification settings - Fork 400
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: new misc tags for packaging imports, + set packagings_complete #8409
Conversation
Codecov Report
@@ Coverage Diff @@
## main #8409 +/- ##
=======================================
Coverage 48.44% 48.45%
=======================================
Files 113 113
Lines 21251 21260 +9
Branches 4767 4769 +2
=======================================
+ Hits 10296 10302 +6
- Misses 9674 9676 +2
- Partials 1281 1282 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM
@@ -23,13 +23,11 @@ | |||
"vegetarian" : "maybe" | |||
}, | |||
{ | |||
"id" : "en:basil-leaf", | |||
"id" : "fr:feuilles-de-basilic", |
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.
do you know why we have this change ?
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.
My mistake, I updated the tests with a wrong version of the taxonomy.
Kudos, SonarCloud Quality Gate passed! |
This PR creates new fields for the number of packaging components (in particular to ease debugging of packagings imports from producers).
When we get packagings imported from producers, it also sets the packagings_complete field to 1.