Skip to content

demo: trigger VG001 smoke#1

Open
eitamring wants to merge 1 commit intomainfrom
demo/vg001-smoke
Open

demo: trigger VG001 smoke#1
eitamring wants to merge 1 commit intomainfrom
demo/vg001-smoke

Conversation

@eitamring
Copy link
Copy Markdown
Contributor

This PR proves the open-source valk-guard integration in valk-guard-example by introducing one intentional violation (VG001) in demo content.

  • Adds: demo/violations/VG001/query.sql
  • Expected scan result: exactly 1 finding (VG001)

@@ -0,0 +1 @@
SELECT * FROM users LIMIT 1;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [valk-guard] reported by reviewdog 🐶
VG001: avoid SELECT *; project only required columns | Query: SELECT * FROM users LIMIT 1

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