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

[frontend] CI Verify translation automatically #6590

Merged
merged 9 commits into from Apr 10, 2024

Conversation

jpkha
Copy link
Member

@jpkha jpkha commented Apr 5, 2024

Add a verification step for translation in Drone

@jpkha jpkha marked this pull request as ready for review April 5, 2024 08:42
@jpkha jpkha marked this pull request as draft April 5, 2024 09:15
@Kedae Kedae added the filigran team use to identify PR from the Filigran team label Apr 5, 2024
Copy link

codecov bot commented Apr 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.69%. Comparing base (8791d29) to head (b25d170).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6590      +/-   ##
==========================================
- Coverage   67.70%   67.69%   -0.01%     
==========================================
  Files         532      532              
  Lines       65144    65144              
  Branches     5470     5470              
==========================================
- Hits        44104    44102       -2     
- Misses      21040    21042       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jpkha jpkha marked this pull request as ready for review April 5, 2024 14:54
@SouadHadjiat
Copy link
Member

Do you think it would be possible to sort the translation files during keys generation and automatic translation ? so that the keys are added by alphabetical order, and not at the end of the file. This would avoid all our conflicts where all the keys are added automatically at the end of files. And also sorting will help organize the keys.

@jpkha jpkha force-pushed the issue/add-verification-step-drone branch from ebadd70 to 172be2d Compare April 10, 2024 12:47
@jpkha
Copy link
Member Author

jpkha commented Apr 10, 2024

In case there is missing translation :
image

@jpkha jpkha merged commit af8b83c into master Apr 10, 2024
5 checks passed
@jpkha jpkha deleted the issue/add-verification-step-drone branch April 10, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
filigran team use to identify PR from the Filigran team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants