(#3675) Extract all used environment variables #3241
+957
−282
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AdmiringWorm
force-pushed
the
environment-variables
branch
2 times, most recently
from
df1f840 to
76676a3
Compare
|
Putting this back to draft, as we can not make all internal types available for dependent projects. |
AdmiringWorm
force-pushed
the
environment-variables
branch
from
76676a3 to
bf0d8b9
Compare
AdmiringWorm
changed the title
AdmiringWorm
force-pushed
the
environment-variables
branch
2 times, most recently
from
e489690 to
7cfa192
Compare
|
Converted this PR to draft until it is ready to be reviewed. |
AdmiringWorm
force-pushed
the
environment-variables
branch
2 times, most recently
from
4ba0e57 to
c507e12
Compare
These changes extracts all of the environment variables that we make use of while running Chocolatey CLI. Both variables that are used as part of running package scripts, and variables we use only when they are set on the system. A change to make these environment variables and all other internal types available to other products and the test projects has been made.
…sources Update EnvironmentVariables class to reference constants from chocolatey.StringResources instead of hardcoded strings. Mark the old EnvironmentVariables class as obsolete to enforce usage of the new centralized resource definitions. Refactor Helpers/Paths.cs and Helpers/EnvironmentHelper.cs to use EnvironmentVariables.System.Path for PATH variable access and modification, ensuring consistency with the updated resource usage. These changes improve maintainability by consolidating environment variable names in a single source and prevent duplication errors.
AdmiringWorm
force-pushed
the
environment-variables
branch
from
c507e12 to
c439222
Compare
|
@gep13 this PR is now ready for review. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description Of Changes
These changes extracts all of the environment variables that we
make use of while running Chocolatey CLI. Both variables that are
used as part of running package scripts, and variables we use
only when they are set on the system.
A change to make these environment variables and all other internal
types available to other products and the test projects has been made.
Motivation and Context
To be able to have a single location to add and use environment variable names from.
Testing
N/A. Need to be ran through test kitchen.
Operating Systems Testing
Change Types Made
Change Checklist
Related Issue
fixes #3675