-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
Add MyGoLibrary link to Awesome Go list #5911
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Automated Quality Checks (from CONTRIBUTING minimum standards)
These checks are a best-effort automation and do not replace human review. |
WalkthroughAdded two new public library entries named MyGoLibrary to the Web Frameworks section of README.md, each with a GitHub link and short description. No code, exported declarations, or runtime behavior were changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
README.md
📄 CodeRabbit inference engine (AGENTS.md)
README.md: When modifying the Awesome list, keep categories with at least three entries
Descriptions in the Awesome list must end with a period
Avoid promotional copy; keep descriptions concise and neutral in the Awesome list
Do not remove existing list content unless removal is requested and justified
Files:
README.md
**/*.md
📄 CodeRabbit inference engine (AGENTS.md)
Keep documentation in English
Files:
README.md
{README.md,COVERAGE.md}
📄 CodeRabbit inference engine (AGENTS.md)
Align rendered documentation (README.md, COVERAGE.md, etc.) with behavior changes in main.go or helper packages
Files:
README.md
🧠 Learnings (1)
📓 Common learnings
Learnt from: elymanuba
PR: avelino/awesome-go#5762
File: README.md:0-0
Timestamp: 2025-08-14T23:46:18.545Z
Learning: When reviewing contributions to the awesome-go repository, maintain a relaxed and less pedantic approach. Users may find overly strict feedback about formatting, alphabetical ordering, or description tone to be unnecessary. Focus on major issues rather than minor stylistic preferences.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Codacy Static Code Analysis
| - [MyGoLibrary](https://github.com/manikonda-anjali/mygolibrary) - A simple Go package for XYZ. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
Alphabetical placement, description, and category need adjustments.
- Placement: Entries are alphabetical. Move “MyGoLibrary” under M, after “Microservice” and before “patron”.
- Description: Replace the placeholder “A simple Go package for XYZ.” with a concise, neutral, specific one-liner that ends with a period. Also state it is a “web framework” (or pick a more fitting category if it isn’t).
- Quality gates: Please ensure the project meets Awesome Go requirements (pkg.go.dev page, Go Report Card with good grade, tests + coverage link, tagged release, go.mod, and CI).
Suggested relocation and placeholder fix:
- - [MyGoLibrary](https://github.com/manikonda-anjali/mygolibrary) - A simple Go package for XYZ.
...
- [Microservice](https://github.com/claygod/microservice) - The framework for the creation of microservices, written in Golang.
+ [Microservice](https://github.com/claygod/microservice) - The framework for the creation of microservices, written in Golang.
+ [MyGoLibrary](https://github.com/manikonda-anjali/mygolibrary) - <concise, neutral web framework description>.
- [patron](https://github.com/beatlabs/patron) - Patron is a microservice framework following best cloud practices with a focus on productivity.
+ [patron](https://github.com/beatlabs/patron) - Patron is a microservice framework following best cloud practices with a focus on productivity.To speed things up, please confirm these via the PR description:
- pkg.go.dev page URL
- Go Report Card URL and grade
- Coverage report URL (Codecov/Coveralls)
- Latest semver tag (release)
- CI status (tests passing)
I'll now verify the review comment by examining the README structure and confirming repository details.
🏁 Script executed:
#!/bin/bash
# Check lines around 3028-3029 in README.md and show broader context
sed -n '3020,3035p' README.md | cat -nLength of output: 1488
🏁 Script executed:
#!/bin/bash
# Find all entries starting with M in the Web Frameworks section to verify alphabetical ordering
# First, find the Web Frameworks section and extract entries to understand structure
grep -n "## Web Frameworks" README.mdLength of output: 22
🏁 Script executed:
#!/bin/bash
# Get a wider view of Web Frameworks section (after finding line number)
sed -n '3000,3050p' README.md | grep -E '^\s*-\s*\[' | head -20Length of output: 2498
🌐 Web query:
mygolibrary Go package github manikonda-anjali
💡 Result:
Do you mean the GitHub repository/package named "mygolibrary" owned by the user "manikonda-anjali"? If so, I can't find that repo with the info provided — do you have a link or more details (exact repo name, Go module path, or screenshot)?
I need to verify the repository exists and check the full alphabetical structure of the Web Frameworks section.
🏁 Script executed:
#!/bin/bash
# Search for other entries starting with M in Web Frameworks section
sed -n '3025,3200p' README.md | grep -E '^\s*-\s*\[[A-Z]' | grep -i '^\s*-\s*\[m'Length of output: 223
🏁 Script executed:
#!/bin/bash
# Get full Web Frameworks section to show proper alphabetical sequence
sed -n '3025,3150p' README.md | grep -E '^\s*-\s*\[' | head -30Length of output: 4995
🌐 Web query:
github.com/manikonda-anjali/mygolibrary
💡 Result:
I tried to open https://github.com/manikonda-anjali/mygolibrary but it returns 404 (not found), so that repository isn’t publicly available at that URL. [1]
The GitHub account manikonda-anjali exists and has other public repos, so the repo may be private, deleted, or the URL may be mistyped. [2]
Would you like me to search that user's public repositories for a similarly named project or check for forks/archives?
Repository URL returns 404—entry cannot be added until verified.
The GitHub link https://github.com/manikonda-anjali/mygolibrary is not accessible (404 error). Please verify and correct the repository URL before proceeding. Once the repository is confirmed accessible, address:
- Alphabetical placement: Move entry under M, after Atreugo and before Microservice.
- Description: Replace "A simple Go package for XYZ." with a concrete, neutral one-liner describing the actual web framework functionality.
🤖 Prompt for AI Agents
In README.md around lines 3028 to 3029, the GitHub link for MyGoLibrary returns
404 and the entry needs verification, a corrected URL, a concrete neutral
one-line description of the project's web framework functionality, and
relocation in alphabetical order under M (place it after Atreugo and before
Microservice); verify the correct repository URL and replace the broken link,
update the description to a factual single-sentence summary of the web framework
(e.g., what it does, main features), and move the entire entry to the proper
alphabetical position under M.
Added MyGoLibrary under the "Web Frameworks" section.
Follows project contribution guidelines.
Not every repository (project) will require every option, but most projects should. Check the Contribution Guidelines for details.
Please provide some links to your package to ease the review
Pull Request content
Category quality
Note that new categories can be added only when there are 3 packages or more.
Packages added a long time ago might not meet the current guidelines anymore. It would be very helpful if you could check 3-5 packages above and below your submission to ensure that they also still meet the Quality Standards.
Please delete one of the following lines:
Thanks for your PR, you're awesome! 😎
Summary by CodeRabbit