chore: clean up comments, fix go.mod replace and cyclomatic complexity#83
Merged
Conversation
**Added:** - Introduced new `ami` CLI command for AMI image management, including subcommands for building, purging, and listing EC2 Image Builder pipeline resources (`cli/cmd/ami.go`) - Added `env` CLI command for managing deployment environments, supporting creation (with variant config generation) and listing of environments (`cli/cmd/env_cmd.go`) **Changed:** - Updated `go.mod` and `go.sum` to include dependencies for warpgate v3, AWS Image Builder, and related tooling - Upgraded various indirect dependencies to match requirements for AMI and variant config features - Modified `terragrunt/runner.go` to: - Change run-all invocation to `terragrunt run --all` instead of `run-all` - Use `TG_TF_PATH` environment variable for Terraform binary override - Remove redundant or outdated comments for clarity - Improve argument handling and logging for consistency with new commands
l50
changed the title
**Changed:** - Suppressed lint warnings by explicitly ignoring return values from `Close()` and `Setenv()` calls using blank identifiers - Updated calls to `color.New(...).Fprintf` to ignore returned values, preventing unused result warnings
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
1 participant
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.
Key Changes:
amicommand group for AMI image management (build, purge, list)envcommand group for environment creation and listingAdded:
amicommand group with subcommands:build: Build AMIs from warpgate templates, supports parallel builds,progress bars, and resource reuse
purge: Remove leftover EC2 Image Builder pipeline resourceslist-resources: List pipeline resources created by warpgateenvcommand group with subcommands:create: Scaffold new deployment environments with infra and config files(supports randomized variant config and force overwrite)
list: List available environments, regions, and config statusgo.modandgo.sumto support new AWS, warpgate,and related functionality
Changed:
run-allexecution to useterragrunt run --allfor consistency withrecent terragrunt versions
TERRAGRUNT_TFPATHtoTG_TF_PATHadditional_windows_ami_filtersin test and staging deployments tomatch only timestamped AMIs, excluding older
goad-dc-base-2016imagesRemoved:
internal/terragrunt/runner.goto streamline code and maintain clarity