Add README files for core projects and include them in NuGet packages#264
Add README files for core projects and include them in NuGet packages#264
Conversation
Reviewer's GuideAdds detailed top-level documentation and project-specific READMEs for each core hosting extension (Orleans, Temporalio, ServiceDefaults, UserSecrets), introduces repository-level metadata (badges, tech stack, usage, scripts, env vars, licensing), and ensures these docs are available in NuGet packages via project file updates and a root LICENSE file. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Review rate limit: 0/1 reviews remaining, refill in 24 minutes and 45 seconds.Comment |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The root README lists C# 14 and .NET 10, which don’t currently exist; consider updating these to the actual targeted C#/.NET versions to avoid confusion for consumers.
- In the root README’s Scripts & Commands section, the
dotnet test (TODO: Add tests to the project)note mixes instructions with TODOs; consider either removing the TODO or moving it to an issue/roadmap section to keep the README focused on usable commands.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The root README lists C# 14 and .NET 10, which don’t currently exist; consider updating these to the actual targeted C#/.NET versions to avoid confusion for consumers.
- In the root README’s Scripts & Commands section, the `dotnet test (TODO: Add tests to the project)` note mixes instructions with TODOs; consider either removing the TODO or moving it to an issue/roadmap section to keep the README focused on usable commands.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Closes #263
Summary by Sourcery
Document hosting extensions packages and update root README for NuGet distribution
Documentation: