Skip to content

Conversation

@Napalys
Copy link
Contributor

@Napalys Napalys commented Nov 13, 2024

Added support for ES2023 feature, Arrray.protype.toSpliced.

The DCA run appears to be fine. The average run time remained consistent, and the meta queries did not detect any new alerts. This was expected due to the novelty of the Arrray.protype.toSpliced function.

@github-actions github-actions bot added the JS label Nov 13, 2024
@Napalys Napalys force-pushed the napalys/toSpliced-support branch from 393ce1c to df4b596 Compare November 13, 2024 12:48
@Napalys Napalys force-pushed the napalys/toSpliced-support branch from 545c340 to 52330e8 Compare November 14, 2024 14:29
@Napalys Napalys force-pushed the napalys/toSpliced-support branch from a369e86 to 6fa3ff3 Compare November 14, 2024 15:56
@Napalys Napalys marked this pull request as ready for review November 15, 2024 07:51
@Napalys Napalys requested a review from a team as a code owner November 15, 2024 07:51
Copy link
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.

Copilot reviewed 5 out of 12 changed files in this pull request and generated 1 suggestion.

Files not reviewed (7)
  • javascript/ql/lib/semmle/javascript/Arrays.qll: Language not supported
  • javascript/ql/lib/semmle/javascript/Regexp.qll: Language not supported
  • javascript/ql/test/library-tests/Arrays/DataFlow.expected: Language not supported
  • javascript/ql/test/library-tests/Arrays/TaintFlow.expected: Language not supported
  • javascript/ql/test/library-tests/RegExp/IsRegex/IsRegex.expected: Language not supported
  • javascript/ql/test/library-tests/RegExp/IsRegex/IsRegex.ql: Language not supported
  • javascript/ql/test/library-tests/TaintTracking/BasicTaintTracking.expected: Language not supported

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@erik-krogh
Copy link
Contributor

change-note? And a comment about the DCA run?
(I usually edit my PR description with a comment about the DCA run when undrafting a PR).

Copy link
Contributor

@erik-krogh erik-krogh left a comment

Choose a reason for hiding this comment

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

Nice 👍

@Napalys Napalys merged commit 63bc1ef into github:main Nov 18, 2024
9 checks passed
@Napalys Napalys deleted the napalys/toSpliced-support branch November 19, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants