add support for native executable entrypoint in container publishing#141
Merged
Gérald Barré (geraldbarre-workleap) merged 3 commits intomainfrom Feb 17, 2026
Merged
add support for native executable entrypoint in container publishing#141Gérald Barré (geraldbarre-workleap) merged 3 commits intomainfrom
Gérald Barré (geraldbarre-workleap) merged 3 commits intomainfrom
Conversation
Copilot started reviewing on behalf of
Gérald Barré (geraldbarre-workleap)
February 17, 2026 20:20
View session
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds support for native executable entrypoints in container publishing for .NET projects. When enabled, containerized applications will use the native executable (e.g., /app/myapp) instead of the default .NET runtime invocation (e.g., dotnet /app/myapp.dll) as the container entrypoint.
Changes:
- Added a new MSBuild property
ContainerUseNativeCommand(default: false) to enable native executable entrypoints - Implemented an MSBuild target that modifies the container entrypoint when publishing single-RID container builds
- Added a test to verify the entrypoint is correctly set when the feature is enabled
- Extended ProjectBuilder test helper with a new
ExecuteDotnetCommandmethod for generic dotnet command execution
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/build/Workleap.DotNet.CodingStandards.props | Adds the ContainerUseNativeCommand property with a default value of false |
| src/build/Workleap.DotNet.CodingStandards.targets | Implements the UpdateContainerCommand target that replaces the container entrypoint with the native executable path for single-RID builds |
| tests/Workleap.DotNet.CodingStandards.Tests/Helpers/ProjectBuilder.cs | Adds a new ExecuteDotnetCommand method to execute arbitrary dotnet commands and capture output |
| tests/Workleap.DotNet.CodingStandards.Tests/ContainerTests.cs | New test file with a test that verifies the native executable entrypoint is used when ContainerUseNativeCommand is enabled |
Comments suppressed due to low confidence (1)
tests/Workleap.DotNet.CodingStandards.Tests/Helpers/ProjectBuilder.cs:6
- The using statements should be ordered alphabetically. System namespaces should be grouped together and sorted alphabetically, followed by other namespaces sorted alphabetically. The expected order would be:
- using System.Text;
- using System.Text.Json;
- using System.Xml.Linq;
- using CliWrap;
- using Xunit.Abstractions;
- using Xunit.Sdk;
using System.Text;
using System.Xml.Linq;
using Xunit.Abstractions;
using System.Text.Json;
using CliWrap;
using Xunit.Sdk;
Mathieu Gamache (PrincessMadMath)
previously approved these changes
Feb 17, 2026
Mathieu Gamache (PrincessMadMath)
approved these changes
Feb 17, 2026
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
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.
Jira issue link: FENG-1866
Description of changes
Breaking changes
Additional checks