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

add example for denylist, update ci #275

Merged
merged 4 commits into from
Feb 28, 2023

Conversation

seanbreckenridge
Copy link
Contributor

@seanbreckenridge seanbreckenridge commented Feb 28, 2023

pushed denylist to my branch now that this is merged here, added that to the docs

just a small fix to link to that and fix module names for hpi module install in the CI (didnt do that in the last PR to avoid possible merge conflicts)

@seanbreckenridge
Copy link
Contributor Author

seanbreckenridge commented Feb 28, 2023

Yeah wow, is totally unpredictable (worked in the second commit I pushed to here but not the third?), I had previously disabled CI on windows because of a Win32Error, perhaps the issue this happens sometimes and not others is because theres some race condition of it checking for the install when installing in parallel?

tried disabling parallel on mac to see if that fixes it

pasted the error in __main__.py as well:

# WARNING: Error parsing requirements for sqlalchemy: [Errno 2] No such file or directory: '/Users/runner/work/HPI/HPI/.tox/mypy-misc/lib/python3.7/site-packages/SQLAlchemy-2.0.4.dist-info/METADATA'

Edit: seems to have worked? But no way to know till its repeated a few times. This can be merged now

@karlicoss karlicoss merged commit db2cd00 into karlicoss:master Feb 28, 2023
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.

2 participants