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

updating various ci bits, codecov action may be broken due to change … #482

Merged
merged 5 commits into from
Jun 19, 2024

Conversation

AndrewAnnex
Copy link
Owner

…in tokenless uploads. Source-dist install I think failed due to logic in setup.py that overwrites the commands in wheel is installed

…in tokenless uploads. Source-dist install I think failed due to logic in setup.py that overwrites the commands in wheel is installed
@pep8speaks
Copy link

pep8speaks commented Jun 19, 2024

Hello @AndrewAnnex! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 117:31: W291 trailing whitespace
Line 104:54: W291 trailing whitespace

Comment last updated at 2024-06-19 18:48:41 UTC

@AndrewAnnex
Copy link
Owner Author

codecov ci reports are currently broken due to changes in their action, but not a huge concern given the high coverage

@AndrewAnnex AndrewAnnex merged commit 184b679 into main Jun 19, 2024
27 checks passed
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.

None yet

2 participants