Skip to content

Conversation

@mjambon
Copy link
Member

@mjambon mjambon commented May 14, 2025

This clarifies and few things and matches the upcoming implementation.

Please ensure

  • A subject matter expert (SME) reviews the content
  • A technical writer reviews the content or PR
  • This change has no security implications or else you have pinged the security team
  • Redirects are added if the PR changes page URLs
  • If you have changed any header tag links (doc/#this-kind-of-anchor), update all instances of that link

@mjambon mjambon requested review from nmote and s-santillan May 14, 2025 04:12
@netlify
Copy link

netlify bot commented May 14, 2025

Don't forget to add /docs at the end of the deploy preview site URL!

Name Link
🔨 Latest commit 50fe974
🔍 Latest deploy log https://app.netlify.com/projects/semgrep-docs-prod/deploys/6825043736365e0008d80314
😎 Deploy Preview https://deploy-preview-2144--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
Contributor

@s-santillan s-santillan left a comment

Choose a reason for hiding this comment

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

lgtm!

mjambon and others added 2 commits May 14, 2025 13:59
Co-authored-by: s-santillan <sarasantillan@gmail.com>
Co-authored-by: s-santillan <sarasantillan@gmail.com>
@mjambon mjambon requested review from ajbt200128 and removed request for nmote May 17, 2025 03:31
@ajbt200128
Copy link
Contributor

blocked until https://github.com/semgrep/semgrep-proprietary/pull/3859 is merged and released

@mjambon
Copy link
Member Author

mjambon commented Jul 16, 2025

I need to revise this to accurately reflect the new semgrep behavior (as of 1.129.0): glob patterns containing a middle slash such as foo/bar are deprecated. They're unanchored but will become anchored in a few months.

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.

4 participants