-
Notifications
You must be signed in to change notification settings - Fork 794
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
simplify EnvironmentVariablesConfigurationProvider.Load #6095
simplify EnvironmentVariablesConfigurationProvider.Load #6095
Conversation
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.
From the header of the file:
// <auto-generated /> (Turns off StyleCop analysis in this file.)
// Licensed to the .NET Foundation under one or more agreements.
// The .NET Foundation licenses this file to you under the MIT license.
I suppose that the current construction is for the reason.
Maybe benchmarking? Did you have chance to verify it/and or double check the source of the file?
@Kielek what generates this file? |
Please check #4092. It is marked as generated to avoid stylecop check |
so if u manually duplicate a file, without also duplicating the tooling that generates it, then it is no longer a generated file |
Yes, it is copied, and adjusted, but all parts are still valid:
|
@SimonCropp The reason for the while loop is the same as described in this discussion, and it is from a file copied from the .NET runtime. How do you want to proceed on this PR? |
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
Fixes #
Design discussion issue #
Changes
Please provide a brief description of the changes here.
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes