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

fix: terraform setup added #1665

Merged
merged 1 commit into from
Jan 30, 2025
Merged

Conversation

neoandmatrix
Copy link
Contributor

Description
This PR includes the necessary changes required in the new ubuntu-latest image used for the workflow to setup terraform.

As previously discussed with @Shurtu-gal in the issue #1658 the used action in the PR is hashicorp/setup-terraform@v3 action and i have tested it by creating a slack workspace and running the workflow similar to one mentioned in the issue.
Here is the repository for same.

Related issue(s)
Resolves #1658

@Shurtu-gal please review and provide feedback on the PR.
Thanks.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@neoandmatrix
Copy link
Contributor Author

Hey @Shurtu-gal any reviews or suggestions for these changes?

Copy link
Contributor

@Shurtu-gal Shurtu-gal left a comment

Choose a reason for hiding this comment

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

LGTM

@Shurtu-gal
Copy link
Contributor

/update

@Shurtu-gal
Copy link
Contributor

@neoandmatrix we need for a codeowner to approve it though.
/ptal

@asyncapi-bot
Copy link
Contributor

@derberg @quetzalliwrites @thulieblack Please take a look at this PR. Thanks! 👋

Copy link
Member

@thulieblack thulieblack left a comment

Choose a reason for hiding this comment

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

@Shurtu-gal I'll approve

@Shurtu-gal
Copy link
Contributor

@thulieblack you would have to approve the workflow as well 😓

@thulieblack
Copy link
Member

which workflow?

@derberg
Copy link
Member

derberg commented Jan 30, 2025

some workflows maintainers need to approve if there is a first PR from contributor

I just clicked

Screenshot 2025-01-30 at 11 07 14

@Shurtu-gal
Copy link
Contributor

/rtm

@asyncapi-bot asyncapi-bot merged commit 2be75a6 into asyncapi:master Jan 30, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Slack groups are not updated
5 participants