Skip to content

[azp] Support Semgrep#132

Merged
qiluo-msft merged 2 commits intosonic-net:masterfrom
maipbui:semgrep_azp
Jan 9, 2023
Merged

[azp] Support Semgrep#132
qiluo-msft merged 2 commits intosonic-net:masterfrom
maipbui:semgrep_azp

Conversation

@maipbui
Copy link
Contributor

@maipbui maipbui commented Jan 9, 2023

Signed-off-by: maipbui [email protected]

Why I did it

Semgrep is a static analysis tool to find security vulnerabilities.

Ref: - Supported Language - Semgrep Rules

How I did it

Integrate Semgrep into this repository by committing a job configuration file
When this PR merged, it forces developers to follow and must pass Semgrep checker

Signed-off-by: maipbui <[email protected]>
Signed-off-by: maipbui <[email protected]>
@maipbui maipbui requested a review from qiluo-msft January 9, 2023 19:57
@qiluo-msft qiluo-msft merged commit c7411ff into sonic-net:master Jan 9, 2023
@maipbui maipbui deleted the semgrep_azp branch January 10, 2023 00:11
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Jan 30, 2023
Update sonic-py-swsssdk submodule pointer to include the following:
* c7411ff [azp] Support Semgrep ([sonic-net#132](sonic-net/sonic-py-swsssdk#132))
* 6a281c8 Use github code scanning instead of LGTM ([sonic-net#131](sonic-net/sonic-py-swsssdk#131))
* 0d73e48 Use github code scanning instead of ([#GT](https://github.com/sonic-net/sonic-py-swsssdk/pull/GT))

Signed-off-by: dprital <[email protected]>
liat-grozovik pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Feb 1, 2023
Update sonic-py-swsssdk submodule pointer to include the following:
* c7411ff [azp] Support Semgrep ([#132](sonic-net/sonic-py-swsssdk#132))
* 6a281c8 Use github code scanning instead of LGTM ([#131](sonic-net/sonic-py-swsssdk#131))
* 0d73e48 Use github code scanning instead of ([#GT](https://github.com/sonic-net/sonic-py-swsssdk/pull/GT))

Signed-off-by: dprital <[email protected]>
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.

2 participants