Skip to content

Fix #10: Remove hardcoded secret from history Update readme#20

Merged
vannu07 merged 5 commits intovannu07:mainfrom
krishnagaud6:update-readme
Oct 12, 2025
Merged

Fix #10: Remove hardcoded secret from history Update readme#20
vannu07 merged 5 commits intovannu07:mainfrom
krishnagaud6:update-readme

Conversation

@krishnagaud6
Copy link
Copy Markdown
Contributor

Summary

This pull request completely removes the hardcoded secret from the commit history.
The branch has been cleaned using an interactive rebase and all previous commits containing the secret have been squashed into a single clean commit.

Changes

  • Dropped the original secret-introducing commit.
  • Squashed all subsequent commits into one clean commit.
  • Verified that the repository history no longer contains the secret.

Purpose

  • Resolves security issue reported by GitGuardian.
  • Ensures this repository complies with security best practices.
  • Prepares branch for Hacktoberfest contribution without flagged secrets.

How to Verify

  • The repository history has been rewritten; no commits contain the secret.
  • All current files are intact and working as intended.

This fixes issue #10 and makes the repository safe for PR review and Hacktoberfest.

@vannu07
Copy link
Copy Markdown
Owner

vannu07 commented Oct 12, 2025

@krishnagaud6, fantastic work! 🎉 You successfully cleaned up the history—the GitGuardian check is now passing! I've gone ahead and added the hacktoberfest-accepted label.

The PR is now only being blocked by the Black Code Formatter checks (the new code style automation).

To fix this and get merged, you just need to format your Python files:

Install Black (if you haven't already):

  • pip install black
    Run Black on your code (e.g., your news fetcher file or the entire repository):
  • black .

Commit the formatting changes that Black makes.

Push the new commit to your branch.

Once the Black checks turn green, the PR will be ready for merge! Great job on the security fix!

@vannu07 vannu07 merged commit b8b05bc into vannu07:main Oct 12, 2025
6 checks passed
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.

2 participants