Skip to content

Conversation

@sarnold
Copy link
Member

@sarnold sarnold commented Nov 19, 2023

switch to setuptools/pep517 packaging, cleanup tests and imports

  • prefer setuptools over poetry, simple is better than overkill
  • prefer pytest as test runner instead of nose/unittest, mark slow tests
  • remove superfluous files, move tool configs to pyproject.toml
  • add tox file with development workflows (no pre-commit just yet)

…ports

* prefer setuptools over poetry, simple is better than overkill
* prefer pytest as test runner instead of nose/unittest, mark slow tests
* remove superfluous files, move tool configs to pyproject.toml
* add tox file with development workflows (no pre-commit just yet)

Signed-off-by: Stephen L Arnold <[email protected]>
* add bandit tox env and fix one class of security warning
* needs a lot more sec cleanup, mainly subprocess and assert calls

Signed-off-by: Stephen L Arnold <[email protected]>
* bump debian package deps to python3 only, add new deps

Signed-off-by: Stephen L Arnold <[email protected]>
* update workflows for now to use 20.04 instead of latest
* remove windows for now and use brew on Macos
* skip linux-only tests on mac or windows
* cleanup more imports and env typos

Signed-off-by: Stephen L Arnold <[email protected]>
@sarnold sarnold force-pushed the pep517 branch 2 times, most recently from b2fd952 to e182218 Compare November 20, 2023 19:58
* remove setuptools_scm toml feature and bump build-system requires
* note setuptools versions greater than 59 are only available for
  py37 and higher while minimum required for setuptools_scm is 61
  while setuptools_scm 8 is also too high for py37
* make sure debian control file depends on python3-setuptools-scm
  because it gets parsed by deb-helper scripts
* update rules file for proper cleaning and deselect CLI tests
* fix even more typos and remove superflous MANIFEST.in file
* make sure debian pkg workflow gets a compliant tag version

Signed-off-by: Stephen L Arnold <[email protected]>
@sarnold sarnold force-pushed the pep517 branch 3 times, most recently from c77ceaa to 646cd85 Compare April 6, 2024 02:31
@sarnold sarnold force-pushed the pep517 branch 8 times, most recently from 0c54c94 to cf1f8d6 Compare April 7, 2024 02:01
* make sure six is a test-only requirement used by oldcodebase
* make sure debuild can see new (python) package name
* enhance the deb workflow for artifact action v4
* update .gitchangelog.rc and add keepachangelog template file
* revert previous changelog updates to original	(upstream)

Signed-off-by: Steve Arnold <[email protected]>
@sarnold sarnold closed this Sep 1, 2025
@sarnold sarnold deleted the pep517 branch September 1, 2025 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants