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 Cloudsmith Repository Resource Documentation #81

Closed

Conversation

RoseSecurity
Copy link

What and Why

  • Updates documentation by linting the markdown and adding valid values to the argument references

@RoseSecurity RoseSecurity requested a review from a team as a code owner May 17, 2024 14:45
@RoseSecurity
Copy link
Author

@BartoszBlizniak This PR would provide explicit guidance on the available choices for the repository resource. Have you considered implementing tfplugindocs to automate the generation of provider documentation?

@noelgass
Copy link

#87 - Could you also consider this report?

@BartoszBlizniak
Copy link
Contributor

Hey @RoseSecurity + @noelgass thank you for your contributions!

@RoseSecurity - unfortunately our tests are set up in a way that we won't be able to run them from a fork atm. I'm not sure if you have permissions to create a branch directly from this project, however, I'm more than happy to do this on your behalf now.

I will also be taking a look at the suggested plugin doc, looks like something we could certainly utilise!

BartoszBlizniak added a commit that referenced this pull request May 30, 2024
Creating a PR on behalf of RoseSecurity from #81
@BartoszBlizniak
Copy link
Contributor

I've created a new PR for this so we can merge the changes there: #88

Thank you for your contribution!

jmiller-cs pushed a commit that referenced this pull request May 31, 2024
Creating a PR on behalf of RoseSecurity from #81
@RoseSecurity
Copy link
Author

@BartoszBlizniak Thank you for merging this! If there is any way that I can support the team or assist in adding functionality to the provider, just say the word. I am actively developing modules and components that utilize this provider so I may have more potential PRs in the near future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants