Skip to content

Updating ChangeLog since release of v17.5.0#2443

Merged
johlju merged 1 commit intomainfrom
updateChangelogAfterv17.5.0
Jan 30, 2026
Merged

Updating ChangeLog since release of v17.5.0#2443
johlju merged 1 commit intomainfrom
updateChangelogAfterv17.5.0

Conversation

@dscbot
Copy link
Copy Markdown
Contributor

@dscbot dscbot commented Jan 30, 2026

This change is Reviewable

@dscbot dscbot requested a review from johlju as a code owner January 30, 2026 08:48
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 30, 2026

Walkthrough

Adds a new CHANGELOG.md entry for version 17.5.0 documenting changes to SqlServerDsc.Common including moving functions into individual files and using ModuleBuilder for assembly, along with integration test refactors and command adjustments.

Changes

Cohort / File(s) Summary
Changelog Update
CHANGELOG.md
Added new version section 17.5.0 (2026-01-30) with changes for SqlServerDsc.Common refactoring, integration test updates, and command adjustments.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description contains only boilerplate HTML/Reviewable markup with no meaningful content about the changeset, making it impossible to assess relevance. Add a brief description explaining what changes are documented in the v17.5.0 release notes or provide context about the update.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating the changelog to document the release of v17.5.0, which directly aligns with 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.

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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
Copy Markdown

@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 (2)
CHANGELOG.md (2)

12-19: ⚠️ Potential issue | 🟡 Minor

Apply Markdown formatting + wrap long lines.

Module/product names should be italicized and long list items wrapped.

✍️ Suggested diff
-- SqlServerDsc.Common
-  - Moved functions into individual files and use ModuleBuilder to assemble.
-- SqlServerDsc
-  - Refactor integration tests for _SQL Server Reporting Services_ and _Power BI_
-    _Report Server_ ([issue `#2431`](https://github.com/dsccommunity/SqlServerDsc/issues/2431)).
-  - `Connect-Sql` create connection and server objects as per documentation.
-  - `Invoke-SqlDscQuery` remove disconnect as there is not an explicit connect.
+- _SqlServerDsc.Common_
+  - Moved functions into individual files and use _ModuleBuilder_ to assemble.
+- _SqlServerDsc_
+  - Refactor integration tests for _SQL Server Reporting Services_ and _Power BI_
+    _Report Server_ ([issue `#2431`](https://github.com/dsccommunity/SqlServerDsc/issues/2431)).
+  - `Connect-Sql` create connection and server objects as per
+    documentation.
+  - `Invoke-SqlDscQuery` remove disconnect as there is not an explicit connect.

As per coding guidelines: “Format resource/module/product names using italic”, and “Wrap lines at word boundaries when over 80 characters (except tables/code blocks)”.


23-25: ⚠️ Potential issue | 🟡 Minor

Fix issue link format.

The issue reference uses backticks and does not match the required format.

✍️ Suggested diff
-    prereleases ([issue `#2437`](https://github.com/dsccommunity/SqlServerDsc/issues/2437)).
+    prereleases ([issue `#2437`](https://github.com/dsccommunity/SqlServerDsc/issues/2437)).

As per coding guidelines: “Reference issues using format issue #<issue_number> in CHANGELOG.md”.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026

Codecov Report

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

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #2443   +/-   ##
=====================================
  Coverage     94%     94%           
=====================================
  Files        225     225           
  Lines      10769   10769           
=====================================
  Hits       10142   10142           
  Misses       627     627           
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 f8f557d into main Jan 30, 2026
40 checks passed
@johlju johlju deleted the updateChangelogAfterv17.5.0 branch January 30, 2026 15:46
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