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 2 typos #62
Fix 2 typos #62
Conversation
Fix: ``` gxformat2/yaml.py:8: error: Library stubs not installed for "yaml" (or incompatible with Python 3.8) gxformat2/yaml.py:8: note: Hint: "python3 -m pip install types-PyYAML" gxformat2/interface.py:10: error: Library stubs not installed for "six" (or incompatible with Python 3.8) gxformat2/interface.py:10: note: Hint: "python3 -m pip install types-six" gxformat2/cytoscape.py:8: error: Library stubs not installed for "pkg_resources" (or incompatible with Python 3.8) gxformat2/cytoscape.py:8: note: Hint: "python3 -m pip install types-setuptools" gxformat2/schema/v19_09.py:76: error: Library stubs not installed for "requests" (or incompatible with Python 3.8) gxformat2/schema/v19_09.py:76: note: Hint: "python3 -m pip install types-requests" gxformat2/schema/v19_09.py:76: note: (or run "mypy --install-types" to install all missing stub packages) gxformat2/schema/v19_09.py:76: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports ```
@@ -25,37 +25,33 @@ def main(argv): | |||
else: | |||
dev_version = re.compile(r'dev([\d]+)').search(version).group(1) | |||
new_dev_version = int(dev_version) + 1 | |||
new_version = version.replace("dev%s" % dev_version, "dev%s" % new_dev_version) | |||
new_version = version.replace(f"dev{dev_version}", f"dev{new_dev_version}") |
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.
This script has been copy-pasted between various projects, do we need to correct this in other places as well ?
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.
It's not really a correction, I just modernised the code with pyupgrade now that we support Python >=3.6 only.
I think updating these scripts together with the rest of each project is OK, but we could surely update the "source" from where they are copied from, if there's one (@jmchilton should know).
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.
Who knows 😆 - I just copied it from whatever project I had last touched probably.
Thanks for the updates and fixes.
Co-authored-by: Marius van den Beek <m.vandenbeek@gmail.com>
Also update code for Python >=3.6 only