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

fix: Fix exception on Tizen due to unsupported Array method #5429

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

joeyparrish
Copy link
Member

Tizen 3 does not support Array.flat(). This fixes a runtime exception on Tizen 3 by replacing flat() with concat() and the spread operator. This also bans the use of flat().

flat() was originally introduced in PR #5422

Tizen 3 does not support Array.flat().  This fixes a runtime exception
on Tizen 3 by replacing flat() with concat() and the spread operator.
This also bans the use of flat().

flat() was originally introduced in PR shaka-project#5422
@github-actions
Copy link
Contributor

Incremental code coverage: 100.00%

@joeyparrish joeyparrish merged commit 8ff2917 into shaka-project:main Jul 21, 2023
@joeyparrish joeyparrish deleted the fix-tizen-exception branch July 21, 2023 13:53
joeyparrish added a commit that referenced this pull request Jul 21, 2023
Tizen 3 does not support Array.flat(). This fixes a runtime exception on
Tizen 3 by replacing flat() with concat() and the spread operator. This
also bans the use of flat().

flat() was originally introduced in PR #5422
joeyparrish added a commit that referenced this pull request Jul 21, 2023
Tizen 3 does not support Array.flat(). This fixes a runtime exception on
Tizen 3 by replacing flat() with concat() and the spread operator. This
also bans the use of flat().

flat() was originally introduced in PR #5422
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Sep 19, 2023
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Sep 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants