BirthdayResearch / defichain-wallet

DeFiChain Wallet. The DeFi Blockchain Light Wallet for iOS, Android & Web.
MIT License
125 stars 46 forks source link

missing translation reporting at merge #8

Closed fuxingloh closed 3 years ago

fuxingloh commented 3 years ago

What would you like to be added:

A custom GitHub workflow to report on missing translation at merge.

Why is this needed:

Allow the translation team and community to see what translation are missing to provide translation PR.

defichain-bot commented 3 years ago

@fuxingloh: Thanks for opening an issue, it is currently awaiting triage.

The triage/accepted label can be added by members by writing /triage accepted in a comment.

Details I am a bot created to help the [DeFiCh](https://github.com/DeFiCh) developers manage community feedback and contributions. You can check out my [manifest file](https://github.com/DeFiCh/fiber/blob/main/.github/governance.yml) to understand my behavior and what I can do. If you want to use this for your project, you can check out the [DeFiCh/oss-governance-bot](https://github.com/DeFiCh/oss-governance-bot) repository.
fuxingloh commented 3 years ago

To comment on a PR you can use GitHub script:

https://github.com/fuxingloh/multi-labeler/blob/07c6e4e2cc0a8820226a0c826588c5fff4e016b5/.github/workflows/ci-use.yml#L15-L23

fuxingloh commented 3 years ago

You also need to disable forked PR as they won't have permission to comment.

https://github.com/DeFiCh/jellyfish/blob/2bfb1d10333a3f1b8133d1f65574bcaa35aae1d8/.github/workflows/ci.yml#L48

fuxingloh commented 3 years ago

To comment on a PR you can use GitHub script:

https://github.com/fuxingloh/multi-labeler/blob/07c6e4e2cc0a8820226a0c826588c5fff4e016b5/.github/workflows/ci-use.yml#L15-L23

You can use this it has de-dup ability https://github.com/marketplace/actions/sticky-pull-request-comment

There are actually more tools in this category, e.g. in line code comment: https://github.com/reviewdog/reviewdog

fuxingloh commented 3 years ago

But we just need a simple one first.