-
Notifications
You must be signed in to change notification settings - Fork 3
Add translation integrity report to CI #351
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
base: main
Are you sure you want to change the base?
Add translation integrity report to CI #351
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #351 +/- ##
========================================
Coverage 57.69% 57.69%
========================================
Files 45 45
Lines 3539 3539
Branches 509 509
========================================
Hits 2042 2042
Misses 1367 1367
Partials 130 130 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Can you point me to a report file that was already generated
| report = "# Ontology Health Report\n" | ||
|
|
||
| if elastic_translation_codeable_concept: | ||
| report += "## Missing translations for elastic search codeable concepts\n" |
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.
Would it be a good idea to use JSON as the output format, such that we have the option to process the report in the future?
|
Don't forget to rebase |
- this will generate the translation report as md from the generated elasticsearch files - languages are found dynamically
f8da0a3 to
66eee6c
Compare
No description provided.