Skip to content

fix(scripts): fix jq syntax error#20593

Open
StaberindeZA wants to merge 1 commit into
mainfrom
fix-jq-syntax
Open

fix(scripts): fix jq syntax error#20593
StaberindeZA wants to merge 1 commit into
mainfrom
fix-jq-syntax

Conversation

@StaberindeZA
Copy link
Copy Markdown
Contributor

Because

  • Running the script resulted in a jq related syntax error.

This pull request

  • Resolves the syntax error.

Issue that this pull request solves

Closes: (issue number)

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).
  • I have manually reviewed all AI generated code.

How to review (Optional)

  • Key files/areas to focus on:
  • Suggested review order:
  • Risky or complex parts:

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

Because:

* Running the script resulted in a jq related syntax error.

This commit:

* Resolves the syntax error.

Closes #
@StaberindeZA StaberindeZA requested a review from a team as a code owner May 13, 2026 18:16
Copilot AI review requested due to automatic review settings May 13, 2026 18:16
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a jq expression in the smoke-test trigger script where the pipe inside an object literal value was not properly scoped, causing a syntax/semantic error. Wrapping the inner expression in parentheses ensures the | .[$param] = true mutation applies to the parameters sub-object instead of the outer object.

Changes:

  • Wrap the parameters object value expression in parentheses so the pipe operates on the inner object only.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@dschom dschom left a comment

Choose a reason for hiding this comment

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

Thanks! I didn't have this issue, but looks correct.

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.

3 participants