New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix panic on run --dry-run #767

Merged

fujiwara merged 2 commits into v2 from fix/run-dry

Nov 6, 2024

Contributor

fujiwara commented Nov 6, 2024

Fixes #764

Refactor task definition handling and related tests.

Introduced taskDefinitionForRun type to encapsulate task definition details. (run.go)
Replaced taskDefinitionArnForRun method with resolveTaskDefinitionForRun to return taskDefinitionForRun instances. (run.go)

fujiwara added 2 commits

November 6, 2024 12:58


          fix panic on run --dry-run

a569d71


          fix if

f733c1e

fujiwara merged commit 7190398 into v2

4 checks passed

fujiwara deleted the fix/run-dry branch

November 6, 2024 04:31

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet