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

Only run Python tests when PR contains more than .po's #2133

Merged
merged 2 commits into from
Nov 9, 2023

Conversation

sarina
Copy link
Contributor

@sarina sarina commented Nov 6, 2023

Copy link

github-actions bot commented Nov 6, 2023

✅ All translation files are valid.

This comment has been posted by the validate-translation-files.yml GitHub Actions workflow.

@sarina sarina force-pushed the sarina/python-tests-action branch from 3de9c5c to aeef38d Compare November 6, 2023 20:27
@sarina
Copy link
Contributor Author

sarina commented Nov 7, 2023

@brian-smith-tcril hmm this is interesting, when the test doesn't run via this mechanism, it just shows the test as pending. I guess we have to do something different, like testing the filepath ourselves and give it a mock pass?

@sarina
Copy link
Contributor Author

sarina commented Nov 7, 2023

OK, using this action - https://github.com/dorny/paths-filter#examples - is what's best.

@sarina sarina force-pushed the sarina/python-tests-action branch from c4ecc95 to 3fcf411 Compare November 7, 2023 02:04
@sarina sarina force-pushed the sarina/python-tests-action branch from 3fcf411 to 972b190 Compare November 7, 2023 02:08
Copy link
Contributor

@brian-smith-tcril brian-smith-tcril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

paths-filter is a great find!

:shipit:

@brian-smith-tcril brian-smith-tcril merged commit e2ecbee into main Nov 9, 2023
@brian-smith-tcril brian-smith-tcril deleted the sarina/python-tests-action branch November 9, 2023 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants