Skip to content

query: add -query flag to validate command#37671

Merged
dsa0x merged 14 commits intomainfrom
sams/validate-query-command
Oct 1, 2025
Merged

query: add -query flag to validate command#37671
dsa0x merged 14 commits intomainfrom
sams/validate-query-command

Conversation

@dsa0x
Copy link
Copy Markdown
Member

@dsa0x dsa0x commented Sep 25, 2025

This adds a -query flag to the validate command, enabling offline validation of the query files. With this flag, all other terraform-related files are also validated.

image

Target Release

1.14.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@dsa0x dsa0x added the 1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged label Sep 25, 2025
@dsa0x dsa0x force-pushed the sams/validate-query-command branch from 9e9dbbc to 5f5b2e4 Compare September 25, 2025 10:32
Base automatically changed from sams/list-remove-excluded-deps to main September 25, 2025 11:50
@dsa0x dsa0x force-pushed the sams/validate-query-command branch from 95ab99a to 0f6f3d8 Compare September 25, 2025 13:11
@dsa0x dsa0x marked this pull request as ready for review September 25, 2025 13:25
@dsa0x dsa0x requested a review from a team as a code owner September 25, 2025 13:25
jbardin
jbardin previously approved these changes Sep 30, 2025
dbanck
dbanck previously approved these changes Oct 1, 2025
Copy link
Copy Markdown
Member

@dbanck dbanck left a comment

Choose a reason for hiding this comment

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

LGTM, one small typo

Co-authored-by: Daniel Banck <dbanck@users.noreply.github.com>
@dsa0x dsa0x merged commit 2274026 into main Oct 1, 2025
7 checks passed
@dsa0x dsa0x deleted the sams/validate-query-command branch October 1, 2025 09:33
dsa0x added a commit that referenced this pull request Oct 1, 2025
Co-authored-by: Samsondeen <40821565+dsa0x@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 1, 2025

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants