Build app and pot files before upload pos to Crowdin #748

Merged
KitaitiMakoto merged 1 commits from build-pos into master 4 years ago
KitaitiMakoto commented 4 years ago (Migrated from github.com)

so that crowdin command recognize updated *.pot files.

See #747 for details.

so that `crowdin` command recognize updated `*.pot` files. See #747 for details.
codecov[bot] commented 4 years ago (Migrated from github.com)

Codecov Report

Merging #748 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #748      +/-   ##
==========================================
+ Coverage   39.00%   39.01%   +0.01%     
==========================================
  Files          73       73              
  Lines        9699     9699              
  Branches     2229     2229              
==========================================
+ Hits         3783     3784       +1     
  Misses       4753     4753              
+ Partials     1163     1162       -1     
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/748?src=pr&el=h1) Report > Merging [#748](https://codecov.io/gh/Plume-org/Plume/pull/748?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/4e43c676b421c8f38485196c04ffb333f4439898&el=desc) will **increase** coverage by `0.01%`. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #748 +/- ## ========================================== + Coverage 39.00% 39.01% +0.01% ========================================== Files 73 73 Lines 9699 9699 Branches 2229 2229 ========================================== + Hits 3783 3784 +1 Misses 4753 4753 + Partials 1163 1162 -1 ```
elegaanz commented 4 years ago (Migrated from github.com)

I can't tell for sure if it will work, since this job only runs on master, but let's merge and see!

I can't tell for sure if it will work, since this job only runs on master, but let's merge and see!
elegaanz (Migrated from github.com) approved these changes 4 years ago
KitaitiMakoto commented 4 years ago (Migrated from github.com)

Thank you! Now plume.pot on Crowdin has been updated!

Thank you! Now plume.pot on Crowdin has been updated!

Reviewers

The pull request has been merged as 8ba0c17db5.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b build-pos master
git pull origin build-pos

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff build-pos
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#748
Loading…
There is no content yet.