Skip to content

Conversation

@jay-nanduri
Copy link
Contributor

Added the learnmore command for execution

Need to create new release for CLI before merging this PR(Checkmarx/ast-cli#428)

@cx-pedro-lopes cx-pedro-lopes requested review from a team and diogopcx and removed request for a team July 12, 2022 20:55
@jay-nanduri jay-nanduri changed the title add learn more descriptions command to wrapper and tests Add learn more descriptions command to wrapper Jul 13, 2022
Copy link
Contributor

@diogopcx diogopcx left a comment

Choose a reason for hiding this comment

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

We should format the changes as well, besides the specific comments

@diogopcx
Copy link
Contributor

diogopcx commented Aug 29, 2022

Logo
Checkmarx AST – Scan Summary & Detailsb50401e2-297e-4ae9-b771-d8c5901438fe

No New Or Fixed Issues Found

@tiagobcx tiagobcx requested review from diogopcx and removed request for a user August 29, 2022 14:54
@tiagobcx tiagobcx added enhancement New feature or request feature and removed enhancement New feature or request labels Aug 29, 2022
@tiagobcx tiagobcx merged commit 8f2adb9 into main Aug 29, 2022
@tiagobcx tiagobcx deleted the AST-14142-sast-descriptions-command branch August 29, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants