Skip to content

Comments

Referring back to the sample ci snippets#2487

Open
sebasrevuelta wants to merge 1 commit intomainfrom
UPDATE_AZURE_SNIPPET
Open

Referring back to the sample ci snippets#2487
sebasrevuelta wants to merge 1 commit intomainfrom
UPDATE_AZURE_SNIPPET

Conversation

@sebasrevuelta
Copy link
Collaborator

This PR pretends to simplify the Azure snippets in our documentation.
Two main works done:

  1. Improve the sample provided in the Sample CI page.
  2. Referring back to this sample in the Azure PR comments page.

For the first point:

The sample in our Sample CI page has been improved with:

  • No need to declare the env section with the SEMGREP_APP_TOKEN (It must be set in the Semgrep_Variables).
  • Added additional comments to clarify the need to indicate the default branch.
  • Added a default image with ubuntu-latest, as it could help customers who want to copy/paste the snippet and see it working.

For the second point

The paragraph that explains how to get a PR comment has been improved.
The snippet has been removed, and it now refers back to the sample CI snippet.

@netlify
Copy link

netlify bot commented Feb 9, 2026

Deploy Preview for semgrep-docs-prod ready!

Name Link
🔨 Latest commit 7c8ee77
🔍 Latest deploy log https://app.netlify.com/projects/semgrep-docs-prod/deploys/6989f5889f08bf0008054c77
😎 Deploy Preview https://deploy-preview-2487--semgrep-docs-prod.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant