Skip to content

Conversation

@jmgate
Copy link
Collaborator

@jmgate jmgate commented May 9, 2024

Type: Refactor

Commits

  1. test: Rename test fixtures (5f7b063)

    Remove acronyms for clarity.

  2. test: Rename tests (2723351)

    The convention is to name tests test_<function_to_test>, but these tests accidentally omitted the leading underscore.

  3. chore: Remove YAPF comments (86f5cfe)

    No longer needed after switching from YAPF to Ruff.

  4. test: Fix imports for unit/integration tests (e1836e3)

  5. refactor: Reduce setup file (b233553)

    Remove the guts of the setup.py file so it just uses the pyproject.toml under the hood.

  6. chore: Add version to init.py (f42f22d)

  7. docs: Add copyright/license text to source files (b0c22fc)

  8. chore: Ignore mypy warnings (4dbf516)

    Temporarily disable these warnings until there's time to revisit and address them.

jmgate added 7 commits May 9, 2024 17:20
Remove acronyms for clarity.
The convention is to name tests `test_<function_to_test>`, but these
tests accidentally omitted the leading underscore.
No longer needed after switching from YAPF to Ruff.
Remove the guts of the `setup.py` file so it just uses the
`pyproject.toml` under the hood.
@jmgate jmgate self-assigned this May 9, 2024
Temporarily disable these warnings until there's time to revisit and
address them.
@jmgate jmgate merged commit ca6947e into master May 10, 2024
@jmgate jmgate deleted the minor-tweaks branch May 10, 2024 00:17
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