Skip to content

Fix syntax for pytest coverage report command #718

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.

Sign up for GitHub

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

Jump to bottom
Merged
ionelmc merged 1 commit into from
Sep 9, 2025
Merged

Fix syntax for pytest coverage report command #718

ionelmc merged 1 commit into from
Sep 9, 2025
+1 −1

Conversation

@artpelling
Copy link
Contributor

@artpelling artpelling commented Sep 9, 2025

I had trouble getting the advertised command to work. I have changed the docs with fixed syntax.
@ionelmc ionelmc merged commit f6d4ccf into pytest-dev:master Sep 9, 2025
@ionelmc
Copy link
Member

ionelmc commented Sep 9, 2025

Aaah oops. The trailing dot was causing the problem. The rest will work either way.
@webknjaz
Copy link
Member

webknjaz commented Sep 9, 2025

This doesn't look right. I bet it was just the dot that needed to be removed.
@artpelling
Copy link
Contributor Author

artpelling commented Sep 9, 2025

To clarify: removing just the dot did not work for me.
@artpelling
Copy link
Contributor Author

artpelling commented Sep 9, 2025

This is my setup: https://github.com/acoular/acoular/pull/531/files
@ionelmc
Copy link
Member

ionelmc commented Sep 9, 2025

I guess hatch doesn't support the ${foobar} env var syntax.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@artpelling @ionelmc @webknjaz