-
Notifications
You must be signed in to change notification settings - Fork 27
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
Python versions to support #199
Comments
@ga4gh/vr-maintainers |
Yes, this sounds good to me. |
good with me too |
Hey @korikuzma, the changes that I made in my last commit in #198, were correct, or I still got it wrong? |
@aniketh3014 is what we need now:
Decided to just do one PR since I think it's cleaner |
@korikuzma Thank you for taking the time to clarify and guide me through. I am trying to understand the code base and learn the technologies used, I truly appreciate your support. |
@aniketh3014 Feel free to make new issues if you have questions. We have some issues with the label |
In
setup.cfg
we specifypython_requires = >= 3.6
. I think we should change topython_requires = >= 3.8
since 3.7 end-of-life was 6 days ago. We should also add 3.11 to CI checks. @ga4gh/vrs-python-maintainers do you agree?The text was updated successfully, but these errors were encountered: