-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix: Replace target URL for DOI items #365
Conversation
|
I also cannot reproduce the errors in my local setup on MacOS, hence the many commits |
Actually, I was able to get the same errors. Sorry, It was an error on my part not being able to produce errors. |
Can you hint on what you had to change in your local setup to reproduce the error(s)? |
Silly mistake, I was leaving the |
Interesting. If I run mypy on my local machine, I get a totally different error: EDIT: It seems like if I run "mypy ." it uses the global mypy, instead of the one installed in the virtualenv. I'll need to investigate on this one Anyway, the PR is now ready to be merged @kesara |
Fixes #332