Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Omit Environment.Version from CodeDom generated files #35458

Merged
merged 3 commits into from
Feb 21, 2019

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