Skip to content

Referring back to the sample ci snippets#2487

Merged
sebasrevuelta merged 3 commits intomainfrom
UPDATE_AZURE_SNIPPET
Mar 5, 2026
Merged

Referring back to the sample ci snippets#2487
sebasrevuelta merged 3 commits 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 b3a1bc8
🔍 Latest deploy log https://app.netlify.com/projects/semgrep-docs-prod/deploys/69a98243a29471000832b258
😎 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.

Copy link

@semgrep-shichao semgrep-shichao left a comment

Choose a reason for hiding this comment

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

look good to me

@sebasrevuelta sebasrevuelta requested review from semgrep-shichao and removed request for armchairlinguist March 4, 2026 08:53
Copy link

@semgrep-shichao semgrep-shichao left a comment

Choose a reason for hiding this comment

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

LGTM

@sebasrevuelta sebasrevuelta requested a review from khorne3 March 4, 2026 09:49
Co-authored-by: Katie Horne <katie.horne@semgrep.com>
@sebasrevuelta sebasrevuelta merged commit cd7f823 into main Mar 5, 2026
8 checks passed
@sebasrevuelta sebasrevuelta deleted the UPDATE_AZURE_SNIPPET branch March 5, 2026 15:31
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