forked from gruntwork-io/terragrunt
-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Actualize dependencies, fix tests and solve bugs #699
Merged
Merged
Conversation
This file contains 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
jonapich
approved these changes
Jul 10, 2024
jocgir
force-pushed
the
DT-7003-Build-new-terragrunt-version
branch
from
July 10, 2024 14:47
c5b10fc
to
ab901fb
Compare
We will consider moving to v2 of go-getter and urfave later
We will consider moving to v2 of go-getter and urfave later
This allows reading terraform files containing new syntax introduced after Terraform 0.15.3. While promoting Terraform to official version v1, v2, ... Hashicorp moved some packages under internal making them non accessible as a library from outer program like terragrunt.
…ersion' into DT-7003-Build-new-terragrunt-version
…urs during loading terraform file
jocgir
changed the title
Just update the hardcoded version to start a pull request.
Actualize dependencies, fix tests and solve bugs
Oct 3, 2024
jocgir
force-pushed
the
DT-7003-Build-new-terragrunt-version
branch
3 times, most recently
from
October 5, 2024 14:35
fe8e815
to
8ff22fa
Compare
jocgir
force-pushed
the
DT-7003-Build-new-terragrunt-version
branch
from
October 5, 2024 16:03
8ff22fa
to
9a4f5b3
Compare
jocgir
force-pushed
the
DT-7003-Build-new-terragrunt-version
branch
from
October 5, 2024 16:22
9a4f5b3
to
7a35e42
Compare
jocgir
force-pushed
the
DT-7003-Build-new-terragrunt-version
branch
from
October 5, 2024 16:26
7a35e42
to
a3266c7
Compare
jonapich
approved these changes
Oct 7, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request includes several updates to GitHub Actions workflows, Go dependencies, and codebase improvements for better maintainability and performance. The most important changes are grouped into workflow updates, Go dependency updates, and codebase improvements.
Workflow Updates:
v2
tov3
in.github/workflows/codeql.yml
forInitialize CodeQL
,Autobuild
, andPerform CodeQL Analysis
steps. [1] [2] [3]GoVersion
environment variable and updated Go version to1.23
in.github/workflows/pr.yml
and.github/workflows/tag.yml
. [1] [2] [3] [4] [5] [6] [7] [8]Go Dependency Updates:
goreleaser-action
version tov6
and specified version~> v2
in.github/workflows/pr.yml
and.github/workflows/tag.yml
. [1] [2] [3]staticcheck
andstringer
tools to their latest versions inMakefile
.Codebase Improvements:
ioutil
package withos
package inawshelper/s3.go
,cli/download_source.go
, and related test files. [1] [2] [3] [4] [5] [6] [7]cli/cli_app.go
. [1] [2] [3]fmtcheck
target and simplifiedfmt
target inMakefile
.This pull request includes updates to dependencies, improvements to configuration files, and code refactoring to use newer APIs and correct typos. The most important changes include updating GitHub Actions dependencies, upgrading the Go version, and refactoring code to replace deprecated functions.GitHub Actions Updates:
.github/workflows/codeql.yml
: Updatedcodeql-action
versions fromv2
tov3
forinit
,autobuild
, andanalyze
steps. [1] [2] [3].github/workflows/pr.yml
: Updatedsetup-go
action to use Go version1.23
andgoreleaser-action
tov6
. Also updated Terraform versions in the matrix. [1] [2] [3] [4] [5] [6].github/workflows/tag.yml
: Updatedsetup-go
action to use Go version1.23
andgoreleaser-action
tov6
. [1] [2]Code Refactoring:
awshelper/s3.go
: Replaced deprecatedioutil
functions withos
package functions. [1] [2] [3]cli/cli_app.go
: Updated import paths and corrected typos in variable names and comments. [1] [2] [3] [4] [5]cli/download_source.go
: Replaced deprecatedioutil
functions withos
package functions and corrected typos in comments. [1] [2] [3]cli/download_source_test.go
: Updated test cases to useos.MkdirTemp
instead ofioutil.TempDir
. [1] [2] [3] [4] [5]Configuration File Improvements:
.goreleaser.yml
: Addedversion: 2
to the build customization.Makefile
: Simplifiedfmt
target and updated dependencies to use the latest versions. [1] [2]Minor Fixes:
config/config.go
: Corrected typos in debug messages and removed redundant parameters from function calls. [1] [2] [3] [4] [5]