Skip to content

Relax networkx pin requirement#17

Merged
proccaserra merged 8 commits intoISA-tools:masterfrom
nsoranzo:master
May 28, 2025
Merged

Relax networkx pin requirement#17
proccaserra merged 8 commits intoISA-tools:masterfrom
nsoranzo:master

Conversation

@nsoranzo
Copy link
Contributor

@nsoranzo nsoranzo commented Jun 29, 2024

Libraries should keep pins as lax as possible, see e.g.: https://stackoverflow.com/questions/28509481/should-i-pin-my-python-dependencies-versions

Main reason to open this now is that the currently pinned version (2.5) is not compatible with NumPy 2.0, while the latest releases work fine.

Also:

  • Add CI testing workflow
  • Migrate to pyproject.toml
  • Drop support for Python <3.9, add support for Python up to 3.13 .
  • Ignore .venv/ directory
  • Update package version
  • Remove unused files
  • Fix SyntaxWarnings
  • Cleanup tox config

nsoranzo added a commit to galaxybot/galaxy that referenced this pull request Jun 30, 2024
nsoranzo added a commit to galaxybot/galaxy that referenced this pull request Jun 30, 2024
nsoranzo added a commit to galaxybot/galaxy that referenced this pull request Jul 3, 2024
nsoranzo added 4 commits May 20, 2025 12:06
Also:
- Drop support for Python 3.8, add support for Python 3.13 .
- Remove 2 unused files in `tests/`
@nsoranzo
Copy link
Contributor Author

@proccaserra Any chance you can review/merge this PR? Or we (Galaxy Community) would be happy to take care of this package, if that's preferable?

@proccaserra proccaserra merged commit 12e5a21 into ISA-tools:master May 28, 2025
2 checks passed
@nsoranzo
Copy link
Contributor Author

@proccaserra Thanks a lot for merging! Could you also make a new 0.10.11 release on GitHub please? I've already bumped the package version as part of this pull request and there's a CI workflow that will upload the sdist and wheel to PyPI.

@natefoo
Copy link

natefoo commented Jun 17, 2025

@proccaserra Thanks for the release! Could you also upload the new version to PyPI when you have a moment? There's a fix in #18.

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.

3 participants

Comments