You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.
This has already been changed/improved in the documentation side and I think it would also make sense to improve on the code comments side. Please check the pull request in the documentation repo: dotnet/AspNetCore.Docs#1777
I spent a few hours myself trying to figure out why my environment variables weren't being loaded and why weren't they overriding my variables in the appsettings.json. Lets help other developers with this small change in the comments.
The text was updated successfully, but these errors were encountered:
@divega I could do the PR but as far as I know I'd have to sign the Contributor License Agreement which I haven't done yet and today I'm going on a short vacation until the end of the month.
I'll start by signing the Contributor License Agreement and this gets approved today I'll try to do the PR before going on vacations.
The /// comment for the prefix parameter of the AddEnvironmentVariables method is a bit misleading as it doesn't only filter the environment variables but also removes the provided prefix from the variable name. This behavior should be mentioned in the comments.
This has already been changed/improved in the documentation side and I think it would also make sense to improve on the code comments side. Please check the pull request in the documentation repo: dotnet/AspNetCore.Docs#1777
I spent a few hours myself trying to figure out why my environment variables weren't being loaded and why weren't they overriding my variables in the appsettings.json. Lets help other developers with this small change in the comments.
The text was updated successfully, but these errors were encountered: