-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
Add ReadyToTouch website with companies and jobs using Go in production #5765
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
Thank you for contributing to awesome-go. We will review your contribution as soon as possible. Make sure you add the links in the body of the pull request that are requested in the contribution guide:
|
WalkthroughAdded a new entry “ReadyToTouch” under the Websites section in README.md, linking to its GitHub repo and referencing production Go usage via readytotouch.com. No code or API changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
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: 0
🧹 Nitpick comments (1)
README.md (1)
3698-3698
: Prefer linking to the website itself and minimize deep links.In the Websites section we usually point the item to its primary website and keep descriptions free of extra links. This reduces link-check noise and stays consistent with surrounding entries.
Apply this diff:
- - [ReadyToTouch](https://github.com/readytotouch/readytotouch) - Open-source list of [companies](https://readytotouch.com/golang/companies) and [jobs](https://readytotouch.com/golang/jobs) using Go in production. + - [ReadyToTouch](https://readytotouch.com/golang/) - Open-source list of companies and jobs using Go in production.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md
(1 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
README.md (1)
main_test.go (1)
goqueryFromReadme
(34-48)
⏰ 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
🔇 Additional comments (2)
README.md (2)
3698-3699
: Alphabetical placement and copy look good.The entry is correctly placed between pkg.go.dev and studygolang, and the description is concise.
3698-3698
: All ReadyToTouch links return HTTP 200
I rancurl -L
on the three URLs and confirmed they all resolve with a 200 status (root →/golang/companies
,/golang/companies
,/golang/jobs
). No updates to the README are needed.
Missing: SemVer release (0 tags - requires at least one vX.Y.Z), pkg.go.dev badge, Go Report Card badge, coverage badge, CI setup, and badge links in PR body. |
This is not a Go library, but an open-source website ( https://readytotouch.com/golang/companies ). |
We want to ensure high quality of the packages. Make sure that you've checked the boxes below before sending a pull request.
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