Skip to content

Conversation

@tiagobcx
Copy link
Contributor

@tiagobcx tiagobcx commented Nov 9, 2021

No description provided.

@tiagobcx tiagobcx requested a review from jay-nanduri November 9, 2021 09:28
@cx-pedro-lopes
Copy link
Contributor

Logo
Checkmarx AST - Scan Summary & Details - 17cdfee8-f990-4fe1-b4a1-9501de46cf42

No Violations Found

Copy link
Contributor

@jay-nanduri jay-nanduri left a comment

Choose a reason for hiding this comment

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

looks like the test cases are failing in ci/cd. Can you check once?

@tiagobcx tiagobcx requested a review from jay-nanduri November 10, 2021 13:22
@tiagobcx
Copy link
Contributor Author

looks like the test cases are failing in ci/cd. Can you check once?

fixed

Copy link
Contributor

@jay-nanduri jay-nanduri left a comment

Choose a reason for hiding this comment

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

changes look fine

@tiagobcx tiagobcx requested a review from jay-nanduri November 12, 2021 17:56
@tiagobcx tiagobcx merged commit 0bb6570 into master Nov 15, 2021
@tiagobcx tiagobcx deleted the feature/AST-6782_branches branch November 15, 2021 09:23
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.

3 participants