Skip to content

fix: clean up stale repo links and typo for OSS release - #77

Merged
blue4209211 merged 2 commits into
mainfrom
oss-readiness/docs-cleanup
May 19, 2026
Merged

fix: clean up stale repo links and typo for OSS release#77
blue4209211 merged 2 commits into
mainfrom
oss-readiness/docs-cleanup

Conversation

@blue4209211

Copy link
Copy Markdown
Contributor

Summary

  • Replace remaining links to the defunct rikatz/kubepug repo in docs/install.md and docs/database.md with nudgebee/kubepug.
  • Point snapcraft.yaml source at the fork so any snap build pulls from this repo, not upstream.
  • Fix a "Contextthat" typo in the README example output.
  • Add **/.DS_Store to .gitignore to prevent future accidental commits (existing local .DS_Store files were never tracked).

Loose ends from an OSS-readiness review ahead of going public.

Test plan

  • CI green
  • grep -r rikatz docs/ returns nothing
  • grep Contextthat README.md returns nothing
  • snapcraft.yaml source: points to nudgebee/kubepug

🤖 Generated with Claude Code

Replace remaining links to the defunct rikatz/kubepug repo in
docs/install.md and docs/database.md with the nudgebee fork. Point
snapcraft.yaml at the fork so snap builds use this source. Fix a
"Contextthat" typo in README example output, and ignore .DS_Store
to prevent future accidental commits.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates repository URLs to point to a fork, adds macOS metadata to the .gitignore, and corrects a typo in the README. Feedback suggests further improving the README's readability by fixing another spacing issue and warns that the snapcraft versioning logic may become inconsistent as it still references the upstream snap info while using the fork as the source.

Comment thread README.md Outdated
Comment thread snapcraft.yaml Outdated
- Fix second missing space on README:96 (container.Deprecated → container. Deprecated).
- Delete snapcraft.yaml: the fork does not publish to snap (no snap entries in .goreleaser.yaml), and the override-pull versioning logic still queried the upstream snap channel — easier to drop than maintain.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@blue4209211
blue4209211 merged commit c3ce09a into main May 19, 2026
2 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.

2 participants