Skip to content

Conversation

@Sugar-pack
Copy link
Contributor

All Submissions:

  • Have you signed our CLA?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
  • Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

@Sugar-pack Sugar-pack changed the title Split integration test CLOUDP-141993: Split integration test Oct 28, 2022
@Sugar-pack Sugar-pack force-pushed the CLOUDP-141993-split-test branch 2 times, most recently from fd80d45 to 8c1e10c Compare October 28, 2022 11:05
@Sugar-pack Sugar-pack force-pushed the CLOUDP-141993-split-test branch 2 times, most recently from c48b359 to bc962d2 Compare October 31, 2022 12:21
Copy link
Collaborator

@igor-karpukhin igor-karpukhin left a comment

Choose a reason for hiding this comment

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

Looks good 👍 Please consider my change for tests naming.

@Sugar-pack Sugar-pack force-pushed the CLOUDP-141993-split-test branch from bc962d2 to ab3d1f6 Compare October 31, 2022 18:22
Copy link
Collaborator

@helderjs helderjs left a comment

Choose a reason for hiding this comment

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

LGTM

@Sugar-pack Sugar-pack merged commit 4d508ba into main Nov 2, 2022
@Sugar-pack Sugar-pack deleted the CLOUDP-141993-split-test branch November 2, 2022 05:06
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.

4 participants