OpenRCT2 / OpenRCT2/Localisation
Improve translation check action
Open
Nobody has claimed this yet.
- Dominant language
- No language data
- Stars
- 72
- Forks
- 213
- Avg merge
- 10h 33m
- Merged PRs (30d)
- 20
Description
Few things came up after seeing the script in action. Short intro how it works:
- Checkout the branch of the fork (newest commit) to the
prdirectory - Checkout the master branch of the fork (not OpenRCT/Localisation)
- Calculate and checkout the
merge-baseof the two asmaster
Using the image as example, pr will point to ca1ff5e and master will point to 0339427

Problems with that:
- If the pull request is created using the fork master branch it will not detect any changes (as the merge-base is the same as the newest commit). Example here: #1996. Already mentioned here: https://github.com/OpenRCT2/Localisation/pull/1992#issuecomment-711331634
- If the fork master branch is not rebased it will not contain the newest or not contain at all the check script. Example here: #1997. This should be easy to fix: additionally checkout the master of OpenRCT2/Localisation and run the script from there
- Do not post a comment if there are no changes
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the translation check action and its checkout sequence for the PR branch, fork master, and merge base. Use examples #1996 and #1997 to verify the failure cases, then check that the script runs from OpenRCT2/Localisation master and that no comment is posted when there are no changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git, github-actions
- Domain
- ci-cd
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100