-
Notifications
You must be signed in to change notification settings - Fork 10
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
How to configure? #80
Comments
I'm having the same issue. I'm running inside of a container, I've set |
@daveisfera or @ilyakamens Can you share an example of how you're doing that in GitHub Actions or set up a MRE repo? (My suspicion is that the container's output is being redirected to stdout and GitHub doesn't detect it there?) |
I mean something even simpler than that. The actual output isn't changed at all when this is installed and that's with both |
Yeah, consider this working example: FROM python:3.13-slim
RUN pip install pytest pytest-github-actions-annotate-failures
RUN echo "def test_fail():\n assert False" > test_fail.py
ENV GITHUB_ACTIONS=true
CMD ["pytest", "test_fail.py"] Then
|
I just realized that I was expecting the output to be completely different. I thought it would change the actual output in some way and not that it output a new type of line to |
Ok, but I did notice that the link in the error it generates doesn't open the line of the failed test and appears to just open the commit that failed. Is that an issue with the output that it makes or with how GitHub Actions handles it? |
@daveisfera do you have an example of that? |
Here's an example: |
Hi! I'm having trouble getting this to work. I've installed the plugin via our
requirements.txt
file. We use docker, and so I set the env var as mentioned. I see the following in the test output:That path looks correct to me.
foo
is a top-level directory in our repository. Is there something else I need to do to get this to work? We use a Jenkins pipeline + shell command to run tests. Thanks!The text was updated successfully, but these errors were encountered: