tools-python icon indicating copy to clipboard operation
tools-python copied to clipboard

Moved the metadata into setup.cfg.

Open KOLANICH opened this issue 4 years ago • 6 comments

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 avatar Feb 24 '21 10:02 KOLANICH

@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.

pombredanne avatar Jun 11 '21 12:06 pombredanne

This has been merged after a rebase. Thanks!

pombredanne avatar Jun 11 '21 12:06 pombredanne

Fat fingers! Not merged yet. Sorry for the noise

pombredanne avatar Jun 11 '21 12:06 pombredanne

@KOLANICH, do you still plan to address these change requests?

@pombredanne, are these changes still up-to-date?

armintaenzertng avatar Oct 20 '22 10:10 armintaenzertng

@pombredanne, are these changes still up-to-date?

I think they are... but your call to verify this.

pombredanne avatar Oct 20 '22 14:10 pombredanne

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.

armintaenzertng avatar Oct 21 '22 09:10 armintaenzertng

Thanks for the update, @KOLANICH! CircleCI is still broken, though. Do you plan to look into this?

armintaenzertng avatar Oct 25 '22 09:10 armintaenzertng

@pombredanne, this still has a blocker from you, would you mind to revisit/lift this? :)

armintaenzertng avatar Oct 27 '22 10:10 armintaenzertng

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.

KOLANICH avatar Oct 31 '22 09:10 KOLANICH

Please see https://github.com/spdx/tools-python/pull/362

jayvdb avatar Dec 09 '22 02:12 jayvdb