-
-
Notifications
You must be signed in to change notification settings - Fork 357
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
docs: cleaning and dispatching product schema #8672
Conversation
234523e
to
5e0e5e9
Compare
Codecov Report
@@ Coverage Diff @@
## main #8672 +/- ##
=======================================
Coverage 48.76% 48.76%
=======================================
Files 117 117
Lines 21819 21819
Branches 4860 4860
=======================================
Hits 10639 10639
Misses 9882 9882
Partials 1298 1298
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
lang: | ||
type: string | ||
description: | | ||
**TODO** explain difference with `lc` |
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.
There's no difference (or rather there should not be a difference), we have 2 fields for historical reasons.
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.
Awesome, thank you!
A few minor precisions in comments.
@alexgarel can we merge? (and possibly make changes later). As it's a big change, I'd like to merge it before doing further changes / additions to the API doc. |
Kudos, SonarCloud Quality Gate passed! |
The product schema is in bad shape… this PR: