Skip to content

Conversation

@freaktechnik
Copy link
Contributor

@freaktechnik freaktechnik commented Oct 26, 2022

Fixes #2811

Signed-off-by: Martin Giger martin@humanoids.be

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

This PR currently has none of the required changelog labels.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

Signed-off-by: Martin Giger <martin@humanoids.be>
@freaktechnik freaktechnik requested a review from a team as a code owner October 26, 2022 15:57
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Oct 26, 2022
@t3chguy
Copy link
Member

t3chguy commented Oct 26, 2022

@freaktechnik I'm afraid I beat you to it with #2812 - if you fancy absorbing the other two call sites into your PR I'll happily close my PR in favour of yours :)

@freaktechnik
Copy link
Contributor Author

Just noticed your PR, thank you very much! I think closing this is simpler :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SAS.ts assumes arrays are from the same JS realm

2 participants