Skip to content

Attribution cleanup #89

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

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

Attribution cleanup #89

wants to merge 6 commits into from

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

Coverage Status

coverage: 91.034%. remained the same
when pulling 565c9ec on attribution-cleanup
into ef3f1f0 on main.

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.

2 participants