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.

Sign up for GitHub

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

Jump to bottom

feat: exec command implementation #3696

Closed
wants to merge 34 commits into from
Closed

feat: exec command implementation #3696

wants to merge 34 commits into from

Conversation

levkohimins
Copy link
Contributor

Description

  1. exec command implementation.
  2. Sorting and renaming CLI flags.
  3. Automatic generation of environment variables.

TODOs

Read the Gruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].

Migration Guide

@levkohimins
fix: exec command implementation
73bec7a
@levkohimins
fix: flags duplication
e5414c0
@levkohimins
chore: update golangci-lint
58a1e68
@levkohimins
fix: log level
e4fb601
@levkohimins
fix: revert pointer
6252aae
@levkohimins
fix: no-color flag duplication
4b1b428
@levkohimins
fix: golint
07da3b2
@levkohimins
fix: strict mode test
00e47f3
@levkohimins
chore: deprecated name funcs improvement
3c5c517
@levkohimins
fix: deprecated names prefix
06950e7
@levkohimins
chore: handle formatter error
38e95b4
@levkohimins
chore: debugging provider cache
2e56b81
@levkohimins
chore: remove debug code
aa9afe4
@levkohimins
chore: debug code
d17e292
@levkohimins
fix: update test envs
e07d93e
@levkohimins
chore: exec-cmd integration test
bd710c4
@levkohimins
chore: docs update
2919cbb
@levkohimins
fix: app_test
a05afe3
@levkohimins
fix: markdown
b9b82fc
@levkohimins
fix: markdown
7281497
@levkohimins
fix: tests
4f04342
@levkohimins
fix: render-json flag names
6635e0c
@levkohimins
chore: get rid of unnecessary flags for some commands
a064757
@levkohimins
chore: docs update
1248c4c
@levkohimins
chore: make -non-interactive flag global.
49d0be4
@levkohimins
fix: test
c3e8b99
@levkohimins
chore: docs update
472e7fe
@levkohimins
Merge branch 'main' into feat/exec-cmd
1a21c47
@levkohimins
chore: docs update
5706817
@levkohimins levkohimins marked this pull request as ready for review December 20, 2024 23:34
@levkohimins
Copy link
Contributor Author

Since another #3723 was opened that contains the same changes and in addition fixes, this PR will be closed.

@levkohimins levkohimins closed this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@denis256 denis256 Awaiting requested review from denis256 denis256 is a code owner

@yhakbar yhakbar Awaiting requested review from yhakbar yhakbar is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@levkohimins