Skip to content

chore: add error diagnostic to missing required parameters#131

Merged
Emyrk merged 4 commits into
mainfrom
coder/param_required
May 21, 2025
Merged

chore: add error diagnostic to missing required parameters#131
Emyrk merged 4 commits into
mainfrom
coder/param_required

Conversation

@Emyrk
Copy link
Copy Markdown
Member

@Emyrk Emyrk commented May 21, 2025

No description provided.

@Emyrk Emyrk requested a review from jaaydenh May 21, 2025 17:31
Comment thread warnings.go Outdated
)

func warnings(modules terraform.Modules) hcl.Diagnostics {
func parameterDiagnostics(modules terraform.Modules) hcl.Diagnostics {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does it still make sense to name this file warnings.go?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll revert it 👍

@Emyrk Emyrk merged commit 5355be4 into main May 21, 2025
3 checks passed
@Emyrk Emyrk deleted the coder/param_required branch May 21, 2025 18:02
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.

2 participants