Skip to content

chore: update SHC and package version - BED-10484 - #237

Open
mykeelium wants to merge 2 commits into
2.Xfrom
mcuomo/BED-10484
Open

mykeelium wants to merge 2 commits into
2.Xfrom
mcuomo/BED-10484

Conversation

@mykeelium

@mykeelium mykeelium commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Description

Updates the version of SHC and library version for release.

Motivation and Context

This PR addresses: BED-10484

How Has This Been Tested?

Creating a build and running collection.

Screenshots (if appropriate):

Types of changes

  • Chore (a change that does not modify the application functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Summary by CodeRabbit

  • Chores
    • Updated the product and file version labels to 2.18.0. This release is identified by the new version number in project metadata. No end-user feature or behavior changes are described in this update.

@mykeelium mykeelium self-assigned this Nov 10, 2026
@mykeelium mykeelium added the enhancement New feature or request label Nov 10, 2026
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The project and file versions change from 2.17.1 to 2.18.0. CommonLibsVersion changes from 4.9.1 to 4.9.2.

Changes

Release version updates

Layer / File(s) Summary
Version properties
Sharphound.csproj
The project and file versions change to 2.18.0. CommonLibsVersion changes to 4.9.2.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related PRs

Suggested reviewers: lrfalslev

Merge Risk: Minimal · up to 42539

No actionable release-version or dependency-resolution risk is established; the change is mergeable after normal checks.

Security architecture risk: Low · up to 42539

The update changes which shared-library versions builds consume, without changing how those dependencies are selected. No introduced security weakness was demonstrated, but the dependency contents and actual restore results were unavailable, limiting assessment of their security behavior.

Retained concerns
No architecture-level concerns identified.

Security review details

Security Blast Radius

  • inferred — The visible impact is on SharpHound builds consuming the two shared-library package references. The supplied evidence does not establish a newly exposed entrypoint, additional privileges, or a tenant, environment, or asset-level exposure increase.

Trust Boundaries and Controls

  • observed — The existing Dev, Stable, and Local source choices, source-value validation, package-reference conditions, and local DLL paths are unchanged. The PR changes the dependency baseline within that existing boundary rather than introducing a new source-selection policy.
Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed The title clearly identifies the SHC and package version update for BED-10484.
Description check Passed The description follows the required sections, explains the release version updates, identifies BED-10484, describes testing, and completes the change-type and checklist sections.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the version line,
Two point eighteen now looks just fine.
The library hops to four point nine,
A fresh release is marked in time.
Then off I go, through fields that shine!

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a8931388-8b95-46ab-a5c2-95aaa34014c1
Commits

Reviewing files that changed from the base of the PR and between ce7878f and 42539ed.

Files selected for processing: 1

  • Sharphound.csproj

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants