Skip to content

JS: Fix typo (occured -> occurred) - #22819

Merged
owen-mc merged 1 commit into
github:mainfrom
owen-mc:js/typo-occured
Oct 14, 2026
Merged

owen-mc merged 1 commit into
github:mainfrom
owen-mc:js/typo-occured

Conversation

@owen-mc

@owen-mc owen-mc commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Trivial typo.

Supersedes #19485.

@owen-mc
owen-mc requested review from a team and a balanced review from Copilot October 14, 2026 12:46
@owen-mc owen-mc added the no-change-note-required This PR does not need a change note label Oct 14, 2026
@owen-mc
owen-mc requested a review from a team as a code owner October 14, 2026 12:46

Copilot AI left a comment

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.

🟡 Changes recommended

Reindentation changes result columns without updating the expected test snapshot.

1 open finding
What changed in this PR

Corrects “occured” to “occurred” in a JavaScript security test.

Changes:

  • Fixes two typo instances.
  • Reformats the surrounding callback code.
File Description
javascript/​ql/​test/​query-tests/​Security/​CWE-912/​tst.js Corrects typo and reindents callbacks.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread javascript/ql/test/query-tests/Security/CWE-912/tst.js Outdated
@github-actions github-actions Bot added the JS label Oct 14, 2026
@owen-mc
owen-mc merged commit cf4d0e1 into github:main Oct 14, 2026
13 checks passed
@owen-mc
owen-mc deleted the js/typo-occured branch October 14, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

JS no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants