Skip to content
This repository was archived by the owner on Dec 4, 2025. It is now read-only.

Conversation

@tris203
Copy link
Owner

@tris203 tris203 commented Dec 3, 2025

seblyng/roslyn.nvim#268

Summary by CodeRabbit

  • Documentation
    • Added a deprecation notice: Razor/CSHTML support is now provided via cohosting with the Roslyn language server; recommends using se_blyng/roslyn.nvim as the integration path.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 3, 2025

Walkthrough

The README.md was updated to add a deprecation notice stating Razor/CSHTML support is now provided via cohosting with the Roslyn Language Server and recommending seblyng/roslyn.nvim; no code or public API changes were made.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added deprecation notice for Razor/CSHTML support and a recommendation to use seblyng/roslyn.nvim (Roslyn LSP cohosting); documentation-only change

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Review focus: confirm wording and links in README.md.

Poem

🐰 A tiny note I hopped to write,
Deprecation whispered soft and light,
Roslyn guides the code-filled way,
New cohosting dawns the day,

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title references 'IDisposable' but the changeset only adds a deprecation notice to README.md about Razor/CSHTML support moving to roslyn.nvim cohosting, with no code changes. Update the title to reflect the actual change, such as 'chore!: Add deprecation notice for Razor/CSHTML support' to accurately represent the README update about moving to roslyn.nvim.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch deprecate

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 63f956e and c27c45d.

📒 Files selected for processing (1)
  • README.md (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

📥 Commits

Reviewing files that changed from the base of the PR and between c1f48e5 and 63f956e.

📒 Files selected for processing (1)
  • README.md (1 hunks)

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@tris203 tris203 merged commit 2c754a8 into main Dec 4, 2025
7 of 9 checks passed
@tris203 tris203 deleted the deprecate branch December 4, 2025 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants