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.

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

update dependencies #278

Closed
wants to merge 1 commit into from

Conversation

a-hat
Copy link

@a-hat a-hat commented May 13, 2023

Pull Request

Related Github Issues

  • [none]

Description

Upgrades all dependencies to their latest versions. The terragrunt version was already a bit antique, our team needs the latest version to align with our atlantis installation.

There seemed to be some kind of race condition which caused the build to fail in the GitHub actions. I removed a line of code which, as far as I can see, was just an optimization and should not have an impact on functionality.

Would be great to get this merged soon!

Security Implications

  • [none]

System Availability

  • [none]

@a-hat
Copy link
Author

a-hat commented May 15, 2023

Hey @dmattia, since there have been no updates to this project for a while and our team would very much like to continue using the tool, could we do anything to support the continued development of terragrunt-atlantis-config?

@blv-mateucanaves
Copy link

@a-hat I read from this issue that the project still live. Do you mind fixing the failing step? This way things will be easier for @dmattia .
Eager to see this merged because there is a bug in the old terragrunt version that makes the project unusable for us

@a-hat
Copy link
Author

a-hat commented May 30, 2023

@a-hat I read from this issue that the project still live. Do you mind fixing the failing step? This way things will be easier for @dmattia . Eager to see this merged because there is a bug in the old terragrunt version that makes the project unusable for us

@blv-mateucanaves AFAIK the pipeline fails because our fork does not have access to the GitHub secrets of the upstream project. Not sure how the process of getting PRs merged is supposed to work.

We released a new version of terragrunt-atlantis-config with the dependency update in our fork if you want to use that one.

@adamstrawson
Copy link

I think we might have to use your fork here too @a-hat - Disappointing to see the slow progression on this project for what is a minor PR.

@Almenon
Copy link
Collaborator

Almenon commented Jan 27, 2024

Closing, as #297 has been merged in. Now terragrunt-atlantis-config uses v0.54.1.

In regards to the project being alive, the owner recently announced a update in #290 (comment)

@Almenon Almenon closed this Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants