Skip to content

feat: Pin nugetcommand#36

Merged
mm-psy merged 5 commits intodevelopfrom
feature/419-pin-nugetcommand-by-hash
Feb 6, 2026
Merged

feat: Pin nugetcommand#36
mm-psy merged 5 commits intodevelopfrom
feature/419-pin-nugetcommand-by-hash

Conversation

@mm-psy
Copy link
Contributor

@mm-psy mm-psy commented Feb 3, 2026

No description provided.

@mm-psy mm-psy requested a review from Copilot February 3, 2026 14:04
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA f47b124.
Ensure that dependencies are being submitted on PR branches. Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice.

Scanned Files

None

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds the --locked-mode flag to dotnet restore commands in Dockerfiles to pin NuGet package versions during the restore process.

Changes:

  • Added --locked-mode flag to dotnet restore commands in two Dockerfile build stages
  • Ensures deterministic package restoration by requiring packages to match versions specified in the lock file

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
source/AAS.TwinEngine.Plugin.TestPlugin/Dockerfile Added --locked-mode flag to dotnet restore command
source/AAS.TwinEngine.DataEngine/Dockerfile Added --locked-mode flag to dotnet restore command

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mm-psy mm-psy requested a review from mm-kgi February 3, 2026 14:08
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Test & Coverage Report

Test Results Summary

Metric Count
✅ Passed 460
❌ Failed 0
⏭️ Skipped 0

View Detailed Test Results


Code Coverage

Unit Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 90% 80% 1188
Summary 90% (2051 / 2275) 80% (836 / 1044) 1188

Minimum allowed line rate is 80%

Module Tests Coverage

Package Line Rate Branch Rate Complexity Health
AAS.TwinEngine.DataEngine 56% 40% 1188
Summary 56% (1278 / 2275) 40% (421 / 1044) 1188

@mm-psy mm-psy merged commit 24cc628 into develop Feb 6, 2026
7 checks passed
@mm-psy mm-psy deleted the feature/419-pin-nugetcommand-by-hash branch February 6, 2026 13:51
mm-psy referenced this pull request in mm-psy/AAS.TwinEngine.DataEngine Feb 10, 2026
* Resolved container app errors related to AAS registry endpoint (#33)

* Fix :  TestPlugin : 404 - file entries (#34)

* feat: Update Docker publish workflow for tag handling (#32)

* Update Docker publish workflow for tag handling

* Update .github/workflows/docker-publish.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Clean up docker-publish.yml by removing empty line

Removed an empty line before the Export Application SBOM step.

* Add comment for priority setting

* Add clarifying comment to Docker tag rule for non-develop, non-release branches (exclude tags and PRs)

* Update tag extraction for Trivy scanning to prioritize highest tag

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Revert "feat: Update Docker publish workflow for tag handling (#32)" (#38)

This reverts commit a36238e.

* feat: Pin nugetcommand (#36)

* feat: Update Docker publish workflow for tag handling (#39)

* Update Docker publish workflow for tag handling

* Update .github/workflows/docker-publish.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Clean up docker-publish.yml by removing empty line

Removed an empty line before the Export Application SBOM step.

* Add comment for priority setting

* Add clarifying comment to Docker tag rule for non-develop, non-release branches (exclude tags and PRs)

* Update tag extraction for Trivy scanning to prioritize highest tag

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* #129: Created initial structure of C# Playwright API tests for TestPlugin

* #129: Added Environment.GetEnvironmentVariable("BASE_URL") to ApiTestBase

* #129: FluentAssertions removed and replaced usage by xunit Assert

* #129: Health test added

* #129: Unneeded configuration removed

* #129: Refinment of tests

* #129: Refinement of rest api test

* #129: String comparison refined

* #129: Throws argument null excepion of response is null

* #129: Set copy always for GetSubmodelDescriptorById_Reliability_Expected.json

* workflow_dispatch added to dependency-review (#41)

* #185: Workflow for plaaywright tests added

* Revert "workflow_dispatch added to dependency-review (#41)" (#43)

This reverts commit 4e5d5fc.

* fix: Update condition for publishing Playwright test results

---------

Co-authored-by: Kevalkumar Ghelani <kgi@mm-software.com>
Co-authored-by: Hardi Shah <hsh@mm-software.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: mm-hsn <hsn@mm-software.com>
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.

3 participants