Skip to content

Fix license deprecations and align setuptools requirements#161

Merged
sfmig merged 12 commits intoneuroinformatics-unit:mainfrom
AlgoFoe:update-lic
Feb 5, 2026
Merged

Fix license deprecations and align setuptools requirements#161
sfmig merged 12 commits intoneuroinformatics-unit:mainfrom
AlgoFoe:update-lic

Conversation

@AlgoFoe
Copy link
Contributor

@AlgoFoe AlgoFoe commented Jan 26, 2026

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other (package update)

Why is this PR needed?

  • Recent CI warnings indicate that the license syntax and license classifiers used in pyproject.toml are deprecated. Since this PR addressing it has been merged in movement, the same changes are needed here to keep the packaging setup consistent and future-proof.

What does this PR do?

  • Updates packaging config to follow current standards and align build and pre-commit tooling.

References

How has this PR been tested?

  • Tested locally by running the full pre-commit and build checks successfully.

Is this a breaking change?

  • No.

Does this PR require an update to the documentation?

  • No.

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@AlgoFoe
Copy link
Contributor Author

AlgoFoe commented Feb 2, 2026

Hey @sfmig! Gentle ping, all checks are green. Thanks for taking a look when you get time.

@sfmig sfmig self-requested a review February 2, 2026 17:55
Copy link
Member

@sfmig sfmig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @AlgoFoe, thanks for addressing this update!

I left a few comments in-line but it should be almost good to go.

I also noticed we should additionally update the license syntax and the build system requirements in the pyproject.toml file of the cookiecutter project itself (this file). Would you mind adding that?

Thanks again!

@AlgoFoe AlgoFoe requested a review from sfmig February 3, 2026 16:42
@sfmig
Copy link
Member

sfmig commented Feb 5, 2026

thanks @AlgoFoe ! I think this is ready to go 🚀

@sfmig sfmig merged commit b75225d into neuroinformatics-unit:main Feb 5, 2026
10 checks passed
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.

Deprecations for license syntax

2 participants