Skip to content

docs(readme): REMOVED archived sandid repo - #6580

Closed
jgriegershs wants to merge 1 commit into
avelino:mainfrom
jgriegershs:main
Closed

docs(readme): REMOVED archived sandid repo#6580
jgriegershs wants to merge 1 commit into
avelino:mainfrom
jgriegershs:main

Conversation

@jgriegershs

Copy link
Copy Markdown

The repo sandid was archived by @aofei on Aug 7, 2025.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

docs(readme): remove archived sandid repository link

📝 Documentation 🕐 Less than 5 minutes

Grey Divider

AI Description

• Remove archived sandid repository from the README “Other” libraries list.
• Keep the curated list limited to active/maintained projects.
High-Level Assessment

The following are alternative approaches to this PR:

1. Keep entry but mark as archived
  • ➕ Preserves historical reference for users already familiar with the library
  • ➖ Adds noise to an “active recommendations” list
  • ➖ Encourages adoption of unmaintained software
2. Replace with an actively maintained alternative ID library
  • ➕ Maintains list density/utility while avoiding dead links
  • ➖ Requires research and subjective selection criteria beyond the PR’s stated scope

Recommendation: Removing the archived repository link is the best fit for a curated README list. If there’s a desire to preserve historical references, consider a separate “Archived/Unmaintained” section rather than keeping archived items inline.

Files changed (1) +0 / -1

Documentation (1) +0 / -1
README.mdRemove archived sandid entry from library list +0/-1

Remove archived sandid entry from library list

• Deletes the README list item linking to the archived 'aofei/sandid' repository under the miscellaneous libraries section.

README.md

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can describe a rule in plain language on the Rules page and Qodo drafts it for you

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@github-actions

Copy link
Copy Markdown

Automated Quality Checks

Required checks

Repo link: missing from PR body

How to fix: Add the following to your PR description:

Forge link: https://github.com/your-org/your-project

❌ **pkg.go.dev**: missing from PR body
  > **How to fix:** Add the following to your PR description:
  > ```
pkg.go.dev: https://pkg.go.dev/github.com/your-org/your-project

Go Report Card: missing from PR body

How to fix: Add the following to your PR description:

goreportcard.com: https://goreportcard.com/report/github.com/your-org/your-project


### Additional checks

⚠️ **Coverage**: missing from PR body
  > **How to fix:** Add a coverage service link to your PR description:
  > ```
Coverage: https://app.codecov.io/gh/your-org/your-project

Popular options: Codecov, Coveralls. Integrate one with your CI to track coverage automatically.


Action needed: one or more required checks failed. Please update your PR body with the missing links and ensure the repository meets the quality standards.

These checks are automated and do not replace maintainer review. See CONTRIBUTING.md for full guidelines.


PR Diff Validation

Content checks

Files changed: only README.md
Single item: one package removed

Automated diff validation — does not replace maintainer review.

@qodo-code-review

Copy link
Copy Markdown

Qodo Fixer

No findings are available for this PR yet. Findings appear here once Qodo has reviewed the PR.

@jgriegershs

Copy link
Copy Markdown
Author

Closed this PR. I cannot provide the requested links and it does not make sense for deleting an entry to request all the information. It also does not comply with the CONTRIBUTING.md docs.

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.

1 participant