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.

Sign up for GitHub

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

Jump to bottom

Expose the new terraform commands in JFrog CLI's embedded help #1530

Merged
merged 1 commit into from
May 7, 2022
Merged

Expose the new terraform commands in JFrog CLI's embedded help #1530

merged 1 commit into from
May 7, 2022

Conversation

eyalbe4
Copy link
Contributor

@eyalbe4 eyalbe4 commented May 6, 2022

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

Now that Artifactory supports managing Terraform modues, this PR:

  • Exposes the terraform commands in the embedded help.
  • Sets the minimum Artifactory version for Terraform testing.

@eyalbe4 eyalbe4 requested a review from gailazar300 May 6, 2022 06:25
@eyalbe4 eyalbe4 added safe to test Approve running integration tests on a pull request new feature Automatically generated release notes and removed new feature Automatically generated release notes labels May 6, 2022
@eyalbe4 eyalbe4 added safe to test Approve running integration tests on a pull request 🐸 frogbot scan and removed safe to test Approve running integration tests on a pull request labels May 6, 2022
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2022

What is Frogbot?

@eyalbe4 eyalbe4 merged commit 4dc1ed0 into jfrog:dev May 7, 2022
@eyalbe4 eyalbe4 changed the title Expose the terraform commands in JFrog CLI's embedded help Expose the new terraform commands in JFrog CLI's embedded help May 8, 2022
@eyalbe4 eyalbe4 added the new feature Automatically generated release notes label May 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gailazar300 gailazar300 gailazar300 approved these changes

Assignees
No one assigned
Labels
new feature Automatically generated release notes safe to test Approve running integration tests on a pull request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@eyalbe4 @gailazar300