Repository navigation
ci: skip tests when only main.pot changes (backport #36) - #37
mergify[bot] wants to merge 1 commit into
Conversation
(cherry picked from commit f62195b) # Conflicts: # .github/workflows/ci.yml
|
Cherry-pick of f62195b has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
[Low impact] The change affects when this repository's CI workflow starts. Resolve the workflow conflict before merging so app tests can run. Reviews (1) · Last reviewed commit: "ci: skip tests when only main.pot change..." · Reviewed by Greptile |
| <<<<<<< HEAD | ||
| - develop | ||
| ======= | ||
| - version-16 | ||
| paths-ignore: | ||
| - "**/locale/main.pot" | ||
| >>>>>>> f62195b (ci: skip tests when only main.pot changes (#36)) |
There was a problem hiding this comment.
Unresolved merge-conflict markers make .github/workflows/ci.yml invalid YAML. GitHub Actions cannot load the workflow, so app tests will not run even when a PR changes application code. Resolve the conflict, keep the target branch’s push filter, and retain both paths-ignore entries.
| <<<<<<< HEAD | |
| - develop | |
| ======= | |
| - version-16 | |
| paths-ignore: | |
| - "**/locale/main.pot" | |
| >>>>>>> f62195b (ci: skip tests when only main.pot changes (#36)) | |
| - develop | |
| paths-ignore: | |
| - "**/locale/main.pot" |
Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/ci.yml
Line: 7-13
Comment:
**Conflict markers break CI**
Unresolved merge-conflict markers make `.github/workflows/ci.yml` invalid YAML. GitHub Actions cannot load the workflow, so app tests will not run even when a PR changes application code. Resolve the conflict, keep the target branch’s push filter, and retain both `paths-ignore` entries.
```suggestion
- develop
paths-ignore:
- "**/locale/main.pot"
```
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
Changes to
main.potdo not change the behavior of the app. The.potfile is only the template for translations. This change adds**/locale/main.pottopaths-ignore, so a pull request that only changes this file does not start the test workflow.A pull request that also changes other files still runs the tests.
This is an automatic backport of pull request #36 done by Mergify.