tools-python
tools-python copied to clipboard
Moved the metadata into setup.cfg.
Added pyproject.toml. Renamed console_scripts because their names are too generic to have this package installed into system. Versions are now fetched from git tags.
@KOLANICH can you also add your DCO signoff? This was an informal requirement which was not consistently applied until now... this way I can then merge.
This has been merged after a rebase. Thanks!
Fat fingers! Not merged yet. Sorry for the noise
@KOLANICH, do you still plan to address these change requests?
@pombredanne, are these changes still up-to-date?
@pombredanne, are these changes still up-to-date?
I think they are... but your call to verify this.
Thanks @KOLANICH for updating this! :)
Removing setup.py seems like a breaking change, though, and CircleCI workflows also depend on it. I think keeping a minimal setup.py as Philippe suggested here would be the best option for now.
Thanks for the update, @KOLANICH! CircleCI is still broken, though. Do you plan to look into this?
@pombredanne, this still has a blocker from you, would you mind to revisit/lift this? :)
just noting this is not mentioned at as belonging in project table at https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#setuptools-specific-configuration
That doc mentions only setuptools-specific configuration. The most of pyproject.toml is the standardized one for all the major backends except poetry according to PEP 621.
And if we're renaming these, which IMO is a good idea as these names are too generic, can we please change "convertor" to "converter" in order to be using the main word in the dictionaries.
To be honest, convertor feels better for me and I guess for people too: searching in Bing (Google doesn't give out this info at least when JS is disabled) reveals:
convertor 566 000 000 results
converter 564 000 000 results
The rest of suggestions are implemented, thanks.
Please see https://github.com/spdx/tools-python/pull/362