Skip to content

Refactor TaPS package#57

Merged
gpauloski merged 8 commits intomainfrom
issue-41
May 31, 2024
Merged

Refactor TaPS package#57
gpauloski merged 8 commits intomainfrom
issue-41

Conversation

@gpauloski
Copy link
Copy Markdown
Contributor

Description

A big refactor that addresses many issues. Look at the commit history and related issues for more details.

Fixes

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (internal implementation changes)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (no changes to the code)
  • CI change (changes to CI workflows, packages, templates, etc.)
  • Version changes (changes to the package or dependency versions)

Testing

N/A

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Relevant tags are added (breaking, bug, dependencies, documentation, enhancement, refactor).
  • Code changes pass pre-commit (e.g., ruff, mypy, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

@gpauloski gpauloski added breaking Backwards incompatible change to public interfaces documentation Improvements or additions to documentation internal Refactoring, style changes, testing, or code optimizations labels May 31, 2024
@gpauloski gpauloski merged commit d52b22a into main May 31, 2024
@gpauloski gpauloski deleted the issue-41 branch May 31, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Backwards incompatible change to public interfaces documentation Improvements or additions to documentation internal Refactoring, style changes, testing, or code optimizations

Projects

None yet

1 participant