Skip to content

NLog.RegEx - Fixed Wiki-link in nuget-package #6042

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.

Sign up for GitHub

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

Jump to bottom
Merged
snakefoot merged 1 commit into from
Nov 11, 2025
Merged

NLog.RegEx - Fixed Wiki-link in nuget-package #6042

snakefoot merged 1 commit into from
Nov 11, 2025
+1 −1

Conversation

@snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Nov 11, 2025

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Nov 11, 2025 •
edited
Loading

Walkthrough

Updated the PackageReleaseNotes URL in the NLog.RegEx project file from "Replace-Layout-Renderer" to "RegEx-Replace-Layout-Renderer" to reflect the correct package documentation reference.

Changes

Cohort / File(s) Summary
Package configuration
src/NLog.RegEx/NLog.RegEx.csproj		 Updated PackageReleaseNotes URL reference from Replace-Layout-Renderer to RegEx-Replace-Layout-Renderer

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single-line URL update in project metadata
  • No logic, behavioral, or build-process changes
  • Straightforward documentation reference correction

Poem

🐰 A URL corrected with care,
From old name to new, fair and square,
RegEx takes its rightful place,
In the docs with proper grace,
One small hop, one giant leap! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to evaluate relevance to the changeset. Add a pull request description explaining the fix and why the Wiki link needed to be corrected.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a Wiki link URL in the NLog.RegEx nuget package configuration file.
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
📜 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 478301e and 4265114.

📒 Files selected for processing (1)
  • src/NLog.RegEx/NLog.RegEx.csproj (1 hunks)
🔇 Additional comments (1)
src/NLog.RegEx/NLog.RegEx.csproj (1)

17-20: Correct wiki link in package release notes.

The PackageReleaseNotes URL update is verified as accurate. The RegEx Replace Layout Renderer wiki page exists on the NLog GitHub Wiki, confirming the updated reference to "RegEx-Replace-Layout-Renderer" is correct and more specific than the generic "Replace-Layout-Renderer".

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.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 11, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@snakefoot snakefoot merged commit 6c8fefe into NLog:dev Nov 11, 2025
5 of 6 checks passed
@snakefoot snakefoot added this to the 6.0.7 milestone Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size/XS

Projects

None yet

Milestone

6.0.7

Development

Successfully merging this pull request may close these issues.

1 participant

@snakefoot