Skip to content

Updating ChangeLog since release of v17.5.1#2448

Merged
johlju merged 1 commit intomainfrom
updateChangelogAfterv17.5.1
Feb 6, 2026
Merged

Updating ChangeLog since release of v17.5.1#2448
johlju merged 1 commit intomainfrom
updateChangelogAfterv17.5.1

Conversation

@dscbot
Copy link
Contributor

@dscbot dscbot commented Feb 5, 2026

This change is Reviewable

@dscbot dscbot requested a review from johlju as a code owner February 5, 2026 20:36
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Walkthrough

A new release entry for version 17.5.1 is added to the CHANGELOG documenting new SQL escaping helper functions, security improvements to DSC_SqlReplication for T-SQL argument handling, and various enhancements to test infrastructure and related components.

Changes

Cohort / File(s) Summary
CHANGELOG Documentation
CHANGELOG.md
New 17.5.1 release entry documenting additions of ConvertTo-SqlString and ConvertTo-EscapedQueryString helper functions, DSC_SqlReplication security improvements (T-SQL escaping), and test infrastructure updates including SChannelDsc prerequisites and TLS-related enhancements.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Updating ChangeLog since release of v17.5.1' clearly matches the changeset, which updates CHANGELOG.md to document the v17.5.1 release.
Description check ✅ Passed The description contains only a Reviewable link with minimal content. While not detailed, it is related to the pull request itself and does not contradict the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

@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: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
CHANGELOG.md (1)

23-26: ⚠️ Potential issue | 🟡 Minor

Add an issue reference and wrap long lines.

The “Updated Pester test guidance…” change lacks an issue link, and Line 25–26 exceeds 80 characters and has double spaces before the issue link. Please add the required issue reference and re-wrap at word boundaries.
As per coding guidelines: “Reference issues using format issue #<issue_number> in CHANGELOG.md” and “Wrap lines at word boundaries when over 80 characters.”

🧹 Nitpick comments (1)
CHANGELOG.md (1)

12-19: Italicize resource/module/product names in the new entry.

The guidelines require resource/module/product names to be formatted in italics. Please update names like SqlServerDsc, DSC_SqlReplication, and SQL Server 2025 accordingly.
As per coding guidelines: “Format resource/module/product names using italic in Markdown documentation.”

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94%. Comparing base (78a2c04) to head (89ae8ec).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #2448   +/-   ##
=====================================
  Coverage     94%     94%           
=====================================
  Files        227     227           
  Lines      10778   10778           
=====================================
  Hits       10149   10149           
  Misses       629     629           
Flag Coverage Δ
unit 94% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@johlju johlju merged commit e2a8aaf into main Feb 6, 2026
38 of 40 checks passed
@johlju johlju deleted the updateChangelogAfterv17.5.1 branch February 6, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants