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 tf-apply-main.yml #59

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update tf-apply-main.yml #59

wants to merge 5 commits into from

Conversation

ro4i7
Copy link

@ro4i7 ro4i7 commented May 24, 2024

This will resolve the #55 issue for Terraform Deployment error

@ro4i7
Copy link
Author

ro4i7 commented May 24, 2024

Hello @nishantsharmax please look into this

@SSKale1 SSKale1 self-assigned this May 27, 2024
Copy link
Contributor

@Rishappreet-Singh Rishappreet-Singh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing Warnings, Correcting Action Versions as per Issue #55

Thank you for your contribution! It's great to see your engagement in resolving Issue #55. After reviewing the changes made:

  • It's clear that the warning messages related to node version updates are actually advising us to update specific actions rather than the node version itself.
  • For instance, updating actions/checkout@v3 to actions/checkout@v4 and actions/setup-python@v2 to actions/setup-python@v5 directly addresses the warnings mentioned in Issue Terraform Deployment error  #55.

Therefore, the approach of updating the actions is appropriate and directly resolves the identified warnings without the need for adding node versions separately.

Let's proceed with merging these updates to effectively resolve the issues.

@mahhesshh
Copy link

I am facing the same issue how can i solve it easily??

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.

4 participants