Skip to content
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

PEP517 cleanup #2

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

PEP517 cleanup #2

wants to merge 12 commits into from

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

1 participant