Skip to content
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

Tox managment, Black formatting #59

Merged
merged 13 commits into from
May 22, 2024

Conversation

shanemcq18
Copy link
Member

@shanemcq18 shanemcq18 commented May 22, 2024

No API changes here, just some changes to the meta.

  • Switched to tox for state-of-the-art dependency and testing management
  • Made black formatting a requirement. Format the code with make format or tox -e format, check style with make style or tox -e style.
  • Formatted the entire package with black (not every file was changed, just those that haven't been touched in a while).
  • Updated contributor instructions for tox and black.
  • Reduced the number of GitHub Actions tests; should have 12 unit test actions (4 python versions, 3 platforms), and a single test for code style.
  • Removing the docs build action for now, need to figure out how to download data files for the notebooks to execute correctly (see Data download script #60).

@shanemcq18 shanemcq18 added the documentation Changes to or problems with the package documenation label May 22, 2024
@shanemcq18 shanemcq18 merged commit 9b3d122 into Willcox-Research-Group:main May 22, 2024
13 checks passed
@shanemcq18 shanemcq18 deleted the toxin branch May 22, 2024 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes to or problems with the package documenation
Development

Successfully merging this pull request may close these issues.

1 participant