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

feat: support for pytest trace back parsing + directory-mapping #238

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

unki2aut
Copy link

@unki2aut unki2aut commented Mar 13, 2023

When the JUnit XML file is created with pytest --tb=short --junitxml=my/path/report.xml, the pytest-junit reporter can now parse this and the failing tests in a PR will be annotated.

I also implemented a directory-mapping: <from>:<to> property for the action, so when the tests are run in a Docker container the paths might not match the ones in the repository and with this you can fix it, so code annotations work again 🙂

@unki2aut unki2aut changed the title feat: support for pytest trace back parsing feat: support for pytest trace back parsing + directory-mapping Mar 27, 2023
# Conflicts:
#	.github/workflows/check-dist.yml
#	.github/workflows/ci.yml
#	.github/workflows/manual-run.yml
#	.github/workflows/test-report.yml
#	README.md
@jozefizso
Copy link
Collaborator

Hi @unki2aut, can you update your pull requests so it is based from a named branch and not from unki2aut:main? Creating pull requests from the main branch of a fork is not ideal because maintainers cannot fix conflicts by making commit to the base branch of the fork.

@jozefizso jozefizso added the enhancement New feature or request label Dec 18, 2023
@dancek
Copy link

dancek commented Feb 6, 2024

Would be great to see this merged. Is there any way I could help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants