Skip to content
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

Implemented environment specific configuration file loading. #24

Merged
merged 1 commit into from
May 2, 2023

Conversation

Aodai
Copy link
Collaborator

@Aodai Aodai commented May 2, 2023

This pull request implements the suggested features in #14

@Aodai Aodai requested review from iSmokeDrow and Nexitisyo May 2, 2023 15:26
@Nexitisyo
Copy link
Collaborator

I think its missing the setting of an active environment via CLI.
Maybe a new command?
$ set env
Not sure if the application is already smart enough to swap environments at runtime. Maybe we need to reboot the CLI after that?

@Nexitisyo Nexitisyo self-assigned this May 2, 2023
@Aodai
Copy link
Collaborator Author

Aodai commented May 2, 2023

It reads the environment from the DOTNET_ENVIRONMENT env variable as explained in the Discord channel which is the default behaviour for the generic host builder.

@Nexitisyo Nexitisyo merged commit aa4d0ae into Development May 2, 2023
@Nexitisyo Nexitisyo deleted the appsettings branch November 29, 2023 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend appsettings and launch settings to support different environments
2 participants