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.

Sign up for GitHub

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

Jump to bottom

Remove AbstractConstMultiSet #69

Merged
merged 11 commits into from
Apr 10, 2024
Merged

Remove AbstractConstMultiSet #69

merged 11 commits into from
Apr 10, 2024

Conversation

lbressler13
Copy link
Owner

What was changed?

  • Move code from AbstractConstMultiSet back into ConstMultiSet class

Checks

  • New and existing unit tests pass with my changes
  • Unit tests have been written for all new and changed functions
  • Comments have been added where needed
  • Docstrings have been updated with any modified function signatures
  • Anything that has been updated for one array type has been updated for all array types
    • I know. I'm sorry. It's important.
  • ktlint has run successfully
  • The version number has been updated if necessary

How was this tested?

New and existing tests

Additional Comments

Oops

@lbressler13 lbressler13 merged commit acd582e into nextversion Apr 10, 2024
2 checks passed
@lbressler13 lbressler13 deleted the feat/remove-abstract-cms branch April 10, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lbressler13