Skip to content

[js] add js.lib.NativeString #12127

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

Draft
wants to merge 3 commits into
base: development
Choose a base branch
from
Draft

[js] add js.lib.NativeString #12127

wants to merge 3 commits into from

Conversation

kLabz
Copy link
Contributor

@kLabz kLabz commented Apr 4, 2025

Closes #11904

This is not a solution for the general case; removing special handling for untyped cast in null safety would be the way to go to unify behavior between first compilation and compilation with hxb roundtrip.

**/
@:overload(function(pattern:RegExp, replacement:String):String {})
@:overload(function(pattern:RegExp, replacement:String->String):String {})
@:overload(function(pattern:String, replacement:String->String):String {})
Copy link
Member

Choose a reason for hiding this comment

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

I would like to avoid this syntax for new additions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there a way to have "real overloads" share documentation?

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, I guess that's what @see is for, though I'm not sure how you'd point to the right thing here.

@skial skial mentioned this pull request Apr 9, 2025
1 task
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.

[js] Regression with null safety diagnostics
2 participants