chore: Expanding lll coverage to tflint

[yhakbar]

Description

Addressed lll findings in tflint.

TODOs

Read the Gruntwork contribution guidelines.

• I authored this code entirely myself
• I am submitting code based on open source software (e.g. MIT, MPL-2.0, Apache)
• I am adding or upgrading a dependency or adapted code and confirm it has a compatible open source license
• Update the docs.
• Run the relevant tests successfully, including pre-commit checks.
• Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated lll linter coverage to include tflint.

Summary by CodeRabbit

• Chores
  • Updated internal code formatting standards and linter configuration to maintain consistent code quality.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
terragrunt-docs	Ready	Preview, Comment	Apr 29, 2026 5:09pm

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request includes a linter configuration update to exclude an additional directory from line length checks and reformatting of function signatures and debug calls in the tflint module. No functional logic changes were made.

Changes

Cohort / File(s)	Summary
Linter Configuration .golangci.yml	Updated the lll linter's path-except exclusion regex to additionally exempt files under internal/tflint/ from the line length check.
Code Formatting internal/tflint/tflint.go	Reformatted function signatures and debug logging calls from single-line to multi-line format; rewrapped doc comment for improved readability. No control flow or runtime logic changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately captures the main change: expanding lll linter coverage to the tflint module by modifying exclusion patterns.
Description check	✅ Passed	The description adequately explains the changes and includes most required template sections with appropriate checkmarks, though it lacks a 'Fixes' reference.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/expanding-lll-coverage-to-tflint

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

The base branch was changed.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@internal/tflint/tflint.go`:
- Around line 245-247: Update the doc comment for findTflintConfigInProject to
correct the possessive typo: replace "it's" with "its" in the sentence "When
running from cache, we start searching from the original config directory to
find config in the source directory." so the comment reads "its parents" instead
of "it's parents" to fix the grammar in the public-facing comment.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fc1628d0-d7c6-404d-83c2-aeb9d5af0bfb

📥 Commits

Reviewing files that changed from the base of the PR and between 1efbff9 and 2451cb4.

📒 Files selected for processing (2)

• .golangci.yml
• internal/tflint/tflint.go

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Fix doc comment grammar in Line 246 (it's → its).

Small typo in a public-facing comment within the changed block.

✏️ Proposed fix

-// folder or it's parents. When running from cache, we start searching
+// folder or its parents. When running from cache, we start searching

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	// findTflintConfigInProject looks for a .tflint.hcl file in the current
	// folder or it's parents. When running from cache, we start searching
	// from the original config directory to find config in the source directory.
	// findTflintConfigInProject looks for a .tflint.hcl file in the current
	// folder or its parents. When running from cache, we start searching
	// from the original config directory to find config in the source directory.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@internal/tflint/tflint.go` around lines 245 - 247, Update the doc comment for
findTflintConfigInProject to correct the possessive typo: replace "it's" with
"its" in the sentence "When running from cache, we start searching from the
original config directory to find config in the source directory." so the
comment reads "its parents" instead of "it's parents" to fix the grammar in the
public-facing comment.