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

docs: add terraform-docs #86

Merged
merged 8 commits into from
Sep 25, 2024
Merged

docs: add terraform-docs #86

merged 8 commits into from
Sep 25, 2024

Conversation

Signed-off-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
Signed-off-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
@nitrocode nitrocode requested review from a team as code owners September 22, 2024 16:25
Copy link
Contributor

github-actions bot commented Sep 22, 2024

Integration Test Results

20 tests  ±0   20 ✅ ±0   6s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 23b5fc0. ± Comparison against base commit e168234.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Sep 22, 2024

Test Results

79 tests  ±0   79 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 23b5fc0. ± Comparison against base commit e168234.

♻️ This comment has been updated with latest results.

Signed-off-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
Signed-off-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
@nitrocode nitrocode enabled auto-merge September 22, 2024 16:36
Signed-off-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
.python-version Outdated Show resolved Hide resolved
@paulschwarzenberger
Copy link
Member

In my other open-source project, I implemented a Terraform docs GitHub Action which automatically updates every time a PR is submitted, and writes to a separate file, rather than the top-level README, so as to stop the README becoming overly long.
I quite like this approach as it avoids separate PRs for Terraform Docs - what do you think?
See example workflow

Copy link
Member

@paulschwarzenberger paulschwarzenberger left a comment

Choose a reason for hiding this comment

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

LGTM

@nitrocode nitrocode merged commit f3d07e7 into main Sep 25, 2024
6 checks passed
@nitrocode nitrocode deleted the terraform-docs branch September 25, 2024 07:27
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.

Use terraform-docs to document inputs/outputs
2 participants