-
Notifications
You must be signed in to change notification settings - Fork 443
Supporting dotnet 10.0 preview version #1305
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Kaniska244
wants to merge
10
commits into
devcontainers:main
Choose a base branch
from
Kaniska244:dotnet_10_preview_version_support
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
28f5a93
Supporting dotnet 10.0 preview version
Kaniska244 d2855d9
Changing the test to fetch actual LTS version.
Kaniska244 e0501a4
To resolve the conflict
Kaniska244 6a504bd
To resolve conflict again
Kaniska244 f9dc5a7
Version bump and adding back the azureedge url's as those should be r…
Kaniska244 481d6e4
To resolve conflict again
Kaniska244 2729ad1
Merge branch 'main' into dotnet_10_preview_version_support
Kaniska244 34ff0c4
Final version bump
Kaniska244 893ca63
Reverting back the test script change as created separate PR for that.
Kaniska244 3f8fec7
Merge branch 'main' into dotnet_10_preview_version_support
Kaniska244 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
using Newtonsoft.Json; | ||
|
||
string json = """ | ||
{ | ||
"Name": "Inception", | ||
"ReleaseDate": "2010-07-08T00:00:00", | ||
"Genres": [ | ||
"Action", | ||
"Thriller" | ||
] | ||
} | ||
"""; | ||
|
||
Movie? m = JsonConvert.DeserializeObject<Movie>(json); | ||
|
||
if (m == default) | ||
{ | ||
Console.WriteLine("Decoding failed!"); | ||
} | ||
else | ||
{ | ||
Console.WriteLine($"Movie name: {m.Name}"); | ||
Console.WriteLine($"Release Date: {m.ReleaseDate}"); | ||
Console.WriteLine($"Genres: {string.Join(", ", m.Genres)}"); | ||
} | ||
|
||
class Movie(string? name, DateTime releaseDate, List<string>? genres) | ||
{ | ||
public string Name { get; set; } = name ?? "Default Name"; | ||
public DateTime ReleaseDate { get; set; } = releaseDate; | ||
public List<string> Genres { get; set; } = genres ?? []; | ||
} |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
<Project Sdk="Microsoft.NET.Sdk"> | ||
|
||
<PropertyGroup> | ||
<OutputType>Exe</OutputType> | ||
<TargetFramework>net10.0</TargetFramework> | ||
<ImplicitUsings>enable</ImplicitUsings> | ||
<Nullable>enable</Nullable> | ||
</PropertyGroup> | ||
|
||
<ItemGroup> | ||
<PackageReference Include="Newtonsoft.Json" Version="13.0.3" /> | ||
</ItemGroup> | ||
|
||
</Project> |
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we change it to 10.0-preview?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @eljog ,
The url
https://builds.dotnet.microsoft.com/dotnet/Sdk/
used in dotnet feature to download the installation package, expects this version 10 to be passed on as10.0
in the same way like other versions. The only way to label it as preview version would be possible if the exact version is provided like10.0.100-preview.3.25201.16
.Whereas in case of the dotnet devcontainer image, the base image is downloaded from
mcr.microsoft.com/dotnet/
registry where the dotnet 10 versions are actuality labeled as10.0-preview
for e.g.mcr.microsoft.com/dotnet/sdk:10.0-preview-trixie-slim
. Thats why it was possible there to label the version as10.0-preview
for the image.So in case of the dotnet feature if we want to change the version from
10.0
to10.0-preview
, we will need a small customization in the feature to remove the preview part from the version parameter & use it in the script. Kindly let me know if we should do something like that for this PR.With Regards,
Kaniska