Skip to content
This repository was archived by the owner on May 8, 2024. It is now read-only.

Conversation

GuillaumeFalourd
Copy link

Signed-off-by: Guillaume Falourd [email protected]

Checklist Reviewer

  • Check if the pull request references the link (url) of the ISSUE or TASK related to the implementation.

  • Make sure the pull request has a clear description of what was implemented, with gifs (using terminalizer) if possible.

  • Check if the pull request has an appropriate label for the state it is in (WIP, ready-for-review, bug, etc...).

  • Check if the pull request needs a walkthrough for reviewers to test the implementation.

  • Check if the code present in the pull request has been tested (unit and integrated tests) if necessary.

  • Check that the pull request was opened against the correct branch (main for fix, release-x.y.x for new features or improvements).


Issue Description

Add sast pipeline

Solution

Following APP SEC suggestion for implementation

Results

New sast pipeline

Screenshots (if appropriate):

Signed-off-by: Guillaume Falourd <[email protected]>
@GuillaumeFalourd GuillaumeFalourd self-assigned this Nov 29, 2022
organization: ${{github.repository_owner}}
environment: main
secrets:
report-secret: ${{ secrets.VULN_MNG_REPORT_KEY }} No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

This secret does not exist in this repo.

Copy link
Author

@GuillaumeFalourd GuillaumeFalourd Nov 29, 2022

Choose a reason for hiding this comment

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

Let's wait for the APP SEC team to add it before merging this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants