Skip to content

Conversation

@AlejandraPedroza
Copy link
Contributor

No description provided.

@AlejandraPedroza AlejandraPedroza requested a review from a team as a code owner November 20, 2025 18:05
@zamulla zamulla self-assigned this Nov 21, 2025
Copy link
Contributor

@zamulla zamulla left a comment

Choose a reason for hiding this comment

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

A couple of things I had questions about, otherwise all good 👍

}
```

2. Enable the `BigInt` type. See how to enable it in [Use of `BigInt` type to represent Kotlin's `Long` type](#use-bigint-type-to-represent-kotlin-s-long-type).
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not about the change, but for consistency: The link text does not match the corresponding heading, should it? Both are grammatically correct, it just looks odd that they are different.

### Use `BigInt64Array` type to represent Kotlin's `LongArray` type
<primary-label ref="experimental-general"/>

Kotlin/JS uses JavaScript's built-in `BigInt64Array` type
Copy link
Contributor

Choose a reason for hiding this comment

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

If I need to enable this feature, it should probably be "can use", not "uses"?

Kotlin/JS uses JavaScript's built-in `BigInt64Array` type
to represent Kotlin's `LongArray` values when compiling to JavaScript.

To enable support for the `BigInt64Array` type, you need to add the following compiler option to your
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To enable support for the `BigInt64Array` type, you need to add the following compiler option to your
To enable support for the `BigInt64Array` type, add the following compiler option to your

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants