Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Omit Environment.Version from CodeDom generated files#35458

Merged
jkotas merged 3 commits intodotnet:masterfrom
jkotas:codedom-runtime-version
Feb 21, 2019
Merged

Omit Environment.Version from CodeDom generated files#35458
jkotas merged 3 commits intodotnet:masterfrom
jkotas:codedom-runtime-version

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented Feb 20, 2019

Contributes to dotnet/coreclr#22664

@jkotas
Copy link
Member Author

jkotas commented Feb 20, 2019

This is fixing the test failures caused by Environment.Version returning actual runtime version.

The alternative way to fix this is to update the test baseline to expect the variable version.

@ericstj
Copy link
Member

ericstj commented Feb 20, 2019

You need to fix

Output.Write("// ");
Output.Write(SR.AutoGen_Comment_Line3);
Output.WriteLine(Environment.Version.ToString());

@jkotas jkotas force-pushed the codedom-runtime-version branch from 2d6f9b2 to aaab092 Compare February 21, 2019 08:19
@jkotas jkotas merged commit 89f72f6 into dotnet:master Feb 21, 2019
@jkotas jkotas deleted the codedom-runtime-version branch February 23, 2019 07:28
@karelz karelz added this to the 3.0 milestone Mar 18, 2019
rainersigwald added a commit to rainersigwald/msbuild that referenced this pull request Nov 25, 2020
It looks like new CodeDOM doesn't include this any more: dotnet/corefx#35458
rainersigwald added a commit to dotnet/msbuild that referenced this pull request Dec 21, 2020
It looks like new CodeDOM doesn't include this any more: dotnet/corefx#35458
rainersigwald added a commit to rainersigwald/msbuild that referenced this pull request Apr 27, 2021
It looks like new CodeDOM doesn't include this any more: dotnet/corefx#35458
rainersigwald added a commit to rainersigwald/msbuild that referenced this pull request Sep 30, 2021
It looks like new CodeDOM doesn't include this any more: dotnet/corefx#35458
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…35458)

* Omit Environment.Version from CodeDom generated files

Contributes to dotnet/coreclr#22664

* Disable tests on .NET Framework


Commit migrated from dotnet/corefx@89f72f6
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants