Switch to pyproject.toml to support python 3.12; drop python 3.9; fixes #127

Goals of this MR:

• Add python 3.12; drop python 3.9
• Fix tests for python 3.12 by dropping setuptools and pkg_resources
• Switch to pyproject.toml from setup.py
• Use more standard testing tools from CI templates
• Fix versioning after dropping versioneer.py
• Add back in dynamic versioning

added 4 commits

• a2a756a6 - Set up for versioneer.py
• 3d1a9234 - Fix last commit
• f2b1b3da - Add build system
• c449ee52 - Fix version

Compare with previous version

added 1 commit

• 9f7e6a06 - Try to fix source

Compare with previous version

added 1 commit

• f03bc5b3 - Try to fix source

Compare with previous version

added 1 commit

• 0efaf071 - Fix .yml

Compare with previous version

added 1 commit

• b3946988 - Fix lint

Compare with previous version

added 1 commit

• e73637b9 - Revert source

Compare with previous version

added 1 commit

• 072f39b4 - Try to fix paths

Compare with previous version

setuptools looks to not be natively supported in python 3.12 environments but is still dependency of versioneer.py , so need to think about how to proceed here (drop it or add as a dependency).

For this MR I will move forward by dropping versioneer.py and eventually (either at the end of this MR or in a future one) add support for dynamic versioning again.

Please don't remove support for dynamic versioning. If you do that, then there will be no runtime record of the version number, which will degrade our telemetry.

Oh. Sorry. I thought this was a GWCelery merge request. Dynamic versioning is not critical for this project. Do whatever you feel like.

I hope to keep it, just that it will probably be one of the later issues to fix (after the build and tests work).

mentioned in merge request !175 (merged)

Looks like just adding setuptools for python 3.12 doesn't work: https://git.ligo.org/brandon.piotrzkowski/ligo-followup-advocate/-/jobs/3594145

My guess is we need to remove this dependence altogether.

changed milestone to %Version 1.2.11

added 1 commit

• bdd50ebf - Update tests and build

Compare with previous version

added 1 commit

• 904332ea - Restore __init__.py

Compare with previous version

added 1 commit

• 8d5385fa - Add importlib

Compare with previous version

added 1 commit

• d629c4ee - Fix .yml

Compare with previous version

added 1 commit

• c1438a53 - Add tests require

Compare with previous version

added 1 commit

• 75d7ad08 - Try to fix versioning

Compare with previous version

added 1 commit

• f422310b - Fix obvious bug

Compare with previous version

mentioned in merge request lscsoft/raven!130

added 1 commit

• 1f6b4e3d - Add test for getting version

Compare with previous version

added 1 commit

• cac524cd - Fix lint

Compare with previous version

added 1 commit

• cb03e365 - Remove pkg_resources from dependencies

Compare with previous version

added 1 commit

• c6edaa98 - Name test properly

Compare with previous version

added 1 commit

• aa07e269 - Don't use test as pytest fixture

Compare with previous version

added 1 commit

• d8fc7de6 - Add remove nones back to test

Compare with previous version

added 1 commit

• 5483e439 - Remove test and pkg_resources

Compare with previous version

added 1 commit

• 8f2e3a2b - Fix filenames

Compare with previous version

added 1 commit

• ed3b6c6a - Use different importing method

Compare with previous version

added 1 commit

• 54933d65 - Try to fix path

Compare with previous version

added 1 commit

• 1fc8069f - Fix importing of files in tests

Compare with previous version

changed the description

Somehow added a slash. Will revert.

changed this line in version 27 of the diff

marked this merge request as ready

I don't have a clear way to add back in dynamic versioning yet, so going to create an issue to add this back in sometime in the future.

mentioned in issue #128

added 1 commit

• b9d88de3 - Remaining small changes

Compare with previous version

added 1 commit

• 8e6e6cf6 - Move long description to README.md

Compare with previous version

added 3 commits

• 8e6e6cf6...84abebc7 - 2 commits from branch emfollow:master
• 92db55a5 - Switch to pyproject.toml to support python 3.12; drop python 3.9; fixes #127 (closed)

Compare with previous version

added 1 commit

• bae66cac - Add changelog

Compare with previous version

added 1 commit

• 412be072 - Switch to pyproject.toml to support python 3.12; drop python 3.9; fixes #127 (closed)

Compare with previous version

added 1 commit

• 0c01b754 - Switch to pyproject.toml to support python 3.12; drop python 3.9; fixes #127 (closed)

Compare with previous version

approved this merge request

enabled an automatic merge when all merge checks for 0c01b754 pass

merged

changed title from Switch to pyproject.toml to support python 3.12; drop python 3.9 to Switch to pyproject.toml to support python 3.12; drop python 3.9; fixes #127

mentioned in issue #127 (closed)