Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Array.isArray to check SAS method #2813

Closed

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