Refactor YAML parameters and improve variable handling#599
Merged
Conversation
- Updated get-mfa-code.yml to use environment variables for OTP key. - Modified set-commit-hash.yml to utilize environment variables for build source version message. - Enhanced set-facts.yml by standardizing variable references and adding environment variables for better readability. - Adjusted update-github-status.yml to streamline state reporting and use environment variables for description and environment. - Refined create-build-env-vars.yml to improve variable exports and consistency in default values. - Improved deploy-ecs-proxies-retag.yml by standardizing variable references and utilizing environment variables. - Enhanced deploy-service.yml for better readability and consistency in variable handling. - Updated setup-build-name.yml to use environment variables for service name. - Refined utils-pr-pipeline.yml for consistent quoting style and improved readability.
…on dist_dir presence
…eters and improve docstring formatting
…Optional for nullable fields
…ma.py, ecs_service.py, and filter_api_products.py
…ersion.py, scan-secrets.sh, and purge_incorrect_env_vars.py
…larity and efficiency; simplify clean_env function in env_cleaner.py by removing unused parameters.
… and improve environment variable usage for clarity and maintainability.
… names for clarity
…uding variable naming and dictionary initialization. Co-authored-by: Copilot <copilot@github.com>
…tandards Co-authored-by: Copilot <copilot@github.com>
saptarshimandal1
approved these changes
May 15, 2026
Contributor
saptarshimandal1
left a comment
saptarshimandal1
left a comment
There was a problem hiding this comment.
Reviewed and approved.
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
Refactor YAML parameters for consistency and enhance variable handling across multiple files to improve readability and maintainability.