feat(DEV-2781): Notify Not Running in a Git Repo @milldr (#990)
## what - Warn when not running from a git repowhy
- Atmos is intended to be run from within a project -- typically a git repo
- Alternatively you can specify the base path with an env var
references
Summary by CodeRabbit
- New Features
- Added a warning message when running the CLI outside a Git repository without an Atmos configuration, helping users identify misconfigurations.
- Enhanced Atmos configuration validation with Git repository presence checks.
- Bug Fixes
- Improved detection of Git repository presence to ensure accurate warnings and error handling.
- Tests
- Introduced new test cases covering CLI behavior across Git repository and configuration scenarios.
- Added snapshot files to validate expected CLI output and warning messages.
Warn on specific TF Environment Variables @milldr (#1206)
## what - Only warn on these environment variables:"TF_CLI_ARGS"
"TF_VAR_"
"TF_CLI_ARGS_"
"TF_WORKSPACE"
- Deleted duplicated warning on
atmos terraform shell
why
- Previously we warned on all
TF_*environment variables as requested. This is far too noisy and unnecessary for the majority of use cases. - Inspired by terraform-exec, we're going to adopt similar convention.
references
No warning
Warning with 1 input
Warning with 2 inputs
Summary by CodeRabbit
- Bug Fixes
- Improved detection of environment variables that may interfere with Terraform by warning only on specific prefixes, reducing unnecessary warnings.
- Enhanced warning messages with clearer, structured logging for easier troubleshooting.
🚀 Enhancements
chore: Log Level for TTY Warning @milldr (#1212)
## what - Change the TTY warning in the following to debug ``` + atmos vendor pull INFO Vendoring from '/atmos/vendor.yaml' WARN No TTY detected. Falling back to basic output. This can happen when no terminal is attached or when commands are pipelined. INFO ✓ terraform-aws-vpc (v5.19.0) INFO Vendored 1 components. ```why
- Change TTY warning to
log.Debugas it's not important enough to surface at the warn level.
references
Summary by CodeRabbit
- Style
- Adjusted log message severity for "No TTY detected" from warning to debug, reducing unnecessary warning messages during CLI operations.
- Tests
- Updated test snapshots to reflect the new debug log level for TTY detection messages.
- Renamed a test case and modified its configuration to improve reliability and accuracy of test results.