Skip to content

Conversation

@Zeitsperre
Copy link
Collaborator

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • (If applicable) Documentation has been added / updated (for bug fixes / features).
  • (If applicable) Tests have been added.
  • CHANGELOG.rst has been updated (with summary of main changes).

What kind of change does this PR introduce?

  • Removes the obsolete MANIFEST.in file
  • Adds a few more checks to the pre-commit-hooks
  • Updates the authors and maintainers listing in the package metadata
  • Updates the contributors listed in the licensing
  • Adds Zenodo information for all contributors

Does this PR introduce a breaking change?

No.

Other information:

Some of the checks added are very reasonable, e.g.

  • no manual committing to main
  • test files and functions should start with the word "test_"
  • JSON files should be reformatted (excluding Jupyter Notebooks)
  • debug functions should not be found in the library codebase
  • remove the UTF-8 byte order marker

@Zeitsperre Zeitsperre self-assigned this May 6, 2025
@coveralls
Copy link

coveralls commented May 6, 2025

Coverage Status

coverage: 91.136%. remained the same
when pulling 966b16e on attribution-cleanup
into 316164d on main.

@Zeitsperre Zeitsperre merged commit 3338b15 into main Oct 2, 2025
11 checks passed
@Zeitsperre Zeitsperre deleted the attribution-cleanup branch October 2, 2025 15:27
@Zeitsperre Zeitsperre mentioned this pull request Oct 2, 2025
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.

4 participants