Bump Scrutor from 6.1.0 to 7.0.0 - #20
Closed
dependabot[bot] wants to merge 2 commits into
Closed
Conversation
dependabot
Bot
added
.NET
#21) * Fix CWD-relative resource paths; add screenshot and docker quick start The simulator resolved its four .uanodes files relative to the working directory, so `dotnet run --project src` only worked when started from src/ even though the README says to run it from the repo root. The loaders now anchor to AppContext.BaseDirectory (where the build copies the files), keeping the SNAP override. The browser's PkiRoot used GetFolderPath(LocalApplicationData) with the default SpecialFolderOption.None, which returns "" when ~/.local/share does not exist — as in the published container image — turning PkiRoot into a CWD-relative path under the root-owned /app and breaking Connect with "Access to the path '/app/DruOPC' is denied". SpecialFolderOption.Create resolves and creates the folder instead. README: front-page screenshot under the badges, and a Docker subsection at the top of Quick start matching compose.yaml. Verified: `dotnet run --project src` from the repo root starts clean, and `docker compose up` with a locally built browser image connects to opc.tcp://simulator:50000 and browses the address space. * Move simulator pn.json host port in compose to 5840 Host port 8080 is a popular dev port and frequently taken; 5840 rides alongside the OPC UA convention (4840 default, 50000 here) and stays conflict-free. Container port is unchanged. * Restore original line endings on untouched lines The previous commit normalized Boiler2PluginNodes.cs and SimpleEventsNodeManager.cs to CRLF throughout, making dozens of untouched lines show as changed. Both files now differ from main only in the uanodes path fix. * Extract AppFilesHelper for output-relative resource paths The SNAP-or-BaseDirectory resolution was duplicated across the four uanodes loaders; each is now a single AppFilesHelper.GetPath call. Drops the System.IO usings that became dead. * Embed .uanodes models instead of copying loose files LoadFromBinaryResource reads from the assembly's embedded resources before falling back to the filesystem, so embedding the four compiled information models removes the runtime dependency on file layout entirely. The CWD-sensitive path resolution, the SNAP special case, and AppFilesHelper all go away; the loaders pass a plain resource name (LogicalName in opc-plc.csproj keeps the slash-style names). * Rename embedded model resources to single-word names BoilerModel1, BoilerModel2, OpcUaDi, SimpleEvents — the slash-style names looked like directory paths, which the embedded resources specifically are not.
--- updated-dependencies: - dependency-name: Scrutor dependency-version: 7.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/nuget/src/Scrutor-7.0.0
branch
from
August 12, 2026 19:35
bebb0b8 to
47b082e
Compare
Owner
Contributor
Author
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
drusteeby
added a commit
that referenced
this pull request
Aug 13, 2026
Scrutor 7 replaces the conflicted dependabot PR #20; the only usage is the basic assembly scan in Program.cs, unaffected by the major bump. All 74 tests pass. The claude-review job fails on every dependabot PR because dependabot-triggered runs only receive Dependabot secrets, so CLAUDE_CODE_OAUTH_TOKEN is empty. Skip the job for the bot - version bumps don't need an AI review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated Scrutor from 6.1.0 to 7.0.0.
Release notes
Sourced from Scrutor's releases.
7.0.0
What's Changed
New Contributors
Full Changelog: khellang/Scrutor@v6.1.0...v7.0.0
Commits viewable in compare view.