FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Added CodeQL SAST scanning and detect-secrets pre-commit hook by ntkathole · Pull Request #5983 · feast-dev/feast · GitHub

feat: Added CodeQL SAST scanning and detect-secrets pre-commit hook - #5983

Merged
ntkathole merged 1 commit into
feast-dev:masterfrom
ntkathole:sec_checks
Feb 19, 2026
Merged

feat: Added CodeQL SAST scanning and detect-secrets pre-commit hook#5983
ntkathole merged 1 commit into
feast-dev:masterfrom
ntkathole:sec_checks

Conversation

ntkathole commented Feb 18, 2026
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Member

What this PR does / why we need it:

This PR adds CodeQL SAST scanning and detect-secrets hook that runs on every commit to catch accidentally committed secrets.

CodeQL SAST scanning that runs on:

  • PRs targeting master
  • Pushes to master
  • Weekly schedule (Mondays 6am UTC)
  • Scoped to python and javascript-typescript

.pre-commit-config.yaml (modified) - added detect-secrets hook that runs on every commit to catch accidentally committed secrets. Excludes lock files, requirements, SVGs, and HTML to avoid false positives.


ntkathole self-assigned this Feb 18, 2026

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

ntkathole force-pushed the sec_checks branch 2 times, most recently from 4c863b9 to 14295e2 Compare February 19, 2026 04:24
Signed-off-by: ntkathole <nikhilkathole2683@gmail.com>
ntkathole merged commit 547b516 into feast-dev:master Feb 19, 2026
17 of 20 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL