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

feat: adds terraform-docs as a linter 📄 #946

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Gowiem
Copy link

@Gowiem Gowiem commented Dec 31, 2024

Info

  • Adds terraform-docs as a linter plugin to support automatically generating + updating terraform module documentation for terraform projects.
    • Closes Support terraform-docs #681
    • Tests are passing and this looks to be working but could potentially use some more work to make it more configurable. Very open to feedback.

Copy link

trunk-io bot commented Dec 31, 2024

⏱️ 27m total CI duration on this PR
Job Cumulative Duration Recent Runs
Windows Linter Tests 17m 🟩
Linter Tests ubuntu-latest 4m 🟩
Linter Tests macOS 3m 🟩
CodeQL-Build 1m 🟩
Trunk Check runner [linux] 1m 🟩
Repo Tests / Plugin Tests 32s 🟩
Detect changed files 9s 🟩
Aggregate Test Results 1s 🟩

settingsfeedbackdocs ⋅ learn more about trunk.io

output: rewrite
formatter: true
run_from: ${target_directory}
run: terraform-docs markdown --output-file=README.md .
Copy link
Author

Choose a reason for hiding this comment

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

One note here: This is my team and I's (and I would suspect many many others) most common usage when using terraform-docs with terraform child + root modules. However, the tool is fairly configurable and can be run in a lot of different ways. I didn't see an obvious example of how make these run commands configurable, so I figured I'd put up this lightly working version for review + ask for feedback from the trunk team.

Do ya'll have any other configurable plugins that I should check out for inspiration to make sure this doesn't need an immediate update from the 2nd person to use it?

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.

Support terraform-docs
1 participant