-
Notifications
You must be signed in to change notification settings - Fork 16
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 liccheck #416
Add liccheck #416
Conversation
Dunno why the pre-commit hook is failing here... |
I reproduced the CI error on my local machine. To be clear, the error is:
I noticed in the CI output (and on my machine) that the environment for
Removing
|
@markbandstra I think the error you saw just means you don't have ...and I get it on the CI here. Strange that I didn't see it on my local machine when the |
Dunno, feels like a bug. Specifying |
You're right, it's looking in the virtual environment, which does not have requirements = list(requirements)[::-1] so the last requirement in the from So it seems like we want the system's I think maybe you didn't see it on your local machine because you probably ran |
Possibly because there's no |
https://github.com/lbl-anp/becquerel/actions/runs/9304327492/job/25608628469?pr=416
🤷 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, good idea!
Now our |
As we continue to modernize project configuration, we'll probably move requirements into the |
https://github.com/dhatim/python-license-check