Skip to content

Conversation

@yuokada
Copy link
Contributor

@yuokada yuokada commented Jan 17, 2021

Follow up #72 & #66

This PR should be merged after #72

@cla-bot cla-bot bot added the cla-signed label Jan 17, 2021
Copy link
Member

@ebyhr ebyhr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change commit message as "Add python 3.7, 3.8 and 3.9 to classifiers in setup.py" since 3.7 is missing in the current message.

@yuokada
Copy link
Contributor Author

yuokada commented Jan 18, 2021

Sorry, fixed.

@yuokada yuokada changed the title Update setup.py to add python 3.8 and 3.9 to classifiers Update setup.py to add python 3.7, 3.8 and 3.9 to classifiers Jan 18, 2021
@ebyhr ebyhr merged commit d99d2b8 into trinodb:master Jan 18, 2021
@ebyhr
Copy link
Member

ebyhr commented Jan 18, 2021

@yuokada Merged, thanks! You don't need to separate a PR when the commit is related to each other tightly. Adding a commit is fine in such case.

@yuokada yuokada deleted the classifiers-py37 branch September 27, 2024 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants