-
Notifications
You must be signed in to change notification settings - Fork 14
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
Repo setup #28
Repo setup #28
Conversation
Shouldn't the qcforward commits be separated out in its own PR? (at least if we are going to do some rebasing) |
|
||
* Docstrings shall start and end with """ and use Google style. | ||
|
||
* Use pytest as testing engine | ||
|
||
* Code shall be be Python 2.7.13 + and python 3.4 + compliant | ||
* Code shall be Python 2.7.13 + (until mid 2020) and python 3.4 + compliant |
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.
Isn't it 2.7.14 ?
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.
Yes perhaps, but the difference between 2.7.13 and 2.7.14 are not noticable for most users I will think (bug fixes on nitty-gritty details)
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.
I had a look through all of your commits, and from my perspective and limited knowledge everything looks ok ;) I must say I find it quite educational to sit at this review end.. google is my friend at the moment!
* Simplify setup, change form versioneer to setuptools_scm * Edit ignore list * Line length 88 as black * Improve/change setup and versioning system * Disabled Makefile * Clean up generated files and do a code blackening * Added requirements in setup * Changed version system * Added message control list * Improve and correct text * Added doc page for qcforward * Corrected versioning and added qcforward * wip: QCForward class * Blacken code * Added initial test for qcforward
* Simplify setup, change form versioneer to setuptools_scm * Edit ignore list * Line length 88 as black * Improve/change setup and versioning system * Disabled Makefile * Clean up generated files and do a code blackening * Added requirements in setup * Changed version system * Added message control list * Improve and correct text * Added doc page for qcforward * Corrected versioning and added qcforward * wip: QCForward class * Blacken code * Added initial test for qcforward
No description provided.