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

Replacement of hard-coded string in MachineReader.cs #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrVitaly
Copy link

Replacement of hard-coded string "Production" to utilizing the constant Environments.Production provided by the ASP.NET Core framework.
The change enhances the code quality by leveraging framework-provided constants, thereby making the solutions using the example code more maintainable, robust, and aligned with best practices.

Replacement of hard-coded string "Production" to utilizing the constant Environments.Production provided by the ASP.NET Core framework.
@parallelbgls
Copy link
Owner

Got it.
I will not merge this pull request because there are more places uses this code.
I will take this suggestion in next push.

@parallelbgls
Copy link
Owner

Sorry, I don't think your code is correct. Please double check your code.
Snipaste_2023-12-02_07-44-36

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.

None yet

2 participants