Skip to content

Conversation

@imilinovic
Copy link
Contributor

Description

Please briefly explain the changes you made here.

Pull request type

  • Bugfix
  • Algorithm/Module
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

######################################

@imilinovic imilinovic self-assigned this Jul 25, 2023
@imilinovic imilinovic added the status: ready PR is ready for review label Jul 25, 2023
Copy link
Contributor

@antoniofilipovic antoniofilipovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few small changes but good work

@antoniofilipovic antoniofilipovic added status: change PR reviewed - needs changes and removed status: ready PR is ready for review labels Jul 25, 2023
@imilinovic imilinovic added status: ready PR is ready for review and removed status: change PR reviewed - needs changes labels Jul 27, 2023
@antoniofilipovic antoniofilipovic added status: ship it PR approved and removed status: ready PR is ready for review labels Jul 27, 2023
@imilinovic imilinovic merged commit 7211cd4 into E-ivan-add-collections Jul 27, 2023
@imilinovic imilinovic deleted the T422-MAGE-implement-containsAll branch July 27, 2023 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ship it PR approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants