-
-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Add GHA testing whether files were added and deleted in the same PR. #21359
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
Conversation
940ffb9
to
cdc9e73
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the other output. |
You can print out a special message which executes a workflow command, creating an annotation. I'm not sure how it looks if the annotation is on a file that is subsequently deleted. |
I'm not sure I really want to try (and loop over lines in bash, which I never remember how to do :-)), but you should feel free to do so :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks helpful!
@anntzer this is ready. Please remove the test commits. |
Done. |
@meeseeksdev backport to v3.5.x |
…d and deleted in the same PR.
…359-on-v3.5.x Backport PR #21359 on branch v3.5.x (Add GHA testing whether files were added and deleted in the same PR.)
Add GHA testing whether files were added and deleted in the same PR.
PR Summary
Currently has two extra commits that test the functionality and should fail the workflow. Will remove these commits once we know this works.
PR Checklist
pytest
passes).flake8
on changed files to check).flake8-docstrings
and runflake8 --docstring-convention=all
).doc/users/next_whats_new/
(follow instructions in README.rst there).doc/api/next_api_changes/
(follow instructions in README.rst there).