Skip to content

Fix: truffle hog, invalid commit hash.#47

Merged
abhisek merged 1 commit into
mainfrom
fix-truffle-hog
Aug 14, 2025
Merged

Fix: truffle hog, invalid commit hash.#47
abhisek merged 1 commit into
mainfrom
fix-truffle-hog

Conversation

@KunalSin9h

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Kunal Singh <kunalsin9h@gmail.com>
@KunalSin9h KunalSin9h requested review from abhisek and Copilot August 13, 2025 08:52

Copilot AI 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.

Pull Request Overview

This PR fixes an issue with the TruffleHog secret scanning workflow by replacing hardcoded git references with dynamic commit hashes from the GitHub pull request event.

  • Updates the base reference from hardcoded "main" to the actual base commit SHA from the pull request
  • Updates the head reference from hardcoded "HEAD" to the actual head commit SHA from the pull request

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Comment thread .github/workflows/secret_scan.yml
Comment thread .github/workflows/secret_scan.yml
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 5.29%. Comparing base (e6c55bd) to head (a2adf9b).

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #47   +/-   ##
=====================================
  Coverage   5.29%   5.29%           
=====================================
  Files         17      17           
  Lines        869     869           
=====================================
  Hits          46      46           
  Misses       813     813           
  Partials      10      10           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@abhisek abhisek merged commit c26cf5c into main Aug 14, 2025
7 checks passed
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