-
Notifications
You must be signed in to change notification settings - Fork 31
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
Now we store the metadata in setup.cfg. #24
Conversation
setup.py
Outdated
cfg["metadata"]["version"]=__version__ | ||
#print(cfg) | ||
cfg["options"].update(cfg["metadata"]) | ||
cfg=cfg["options"] |
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.
Please fix this broken formatting. Literally all style checkers (pep8, pycodestyle, pylint) complain about:
C: 6, 0: Exactly one space required around assignment
C: 8, 0: Exactly one space required around assignment
C: 11, 0: Exactly one space required around assignment
Probably optional:
C: 6, 0: Constant name "this_dir" doesn't conform to UPPER_CASE naming style (invalid-name)
C: 7, 0: Constant name "cfg" doesn't conform to UPPER_CASE naming style (invalid-name)
C: 11, 0: Constant name "cfg" doesn't conform to UPPER_CASE naming style (invalid-name)
C: 3, 0: standard import "import os" should be placed before "from setuptools import setup" (wrong-import-order)
setup.cfg
Outdated
@@ -1,5 +1,39 @@ | |||
[metadata] | |||
name = kaitaistruct | |||
version = 0.1.0 |
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.
Shall we have it here in the first place, given that we override it in the code? If it's mandatory, could we put something clearly invalid (like ???
or xxx
or 0.0.0
) here?
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.
Oops, I have forgotten to remove it after I have copied my boilerplate.
Thanks for this contribution! Could you fix these style issues, so I can merge it? |
25f233c
to
5bdc694
Compare
Looks legit! Thanks! |
No description provided.