Skip to content

Conversation

@sjrd
Copy link
Member

@sjrd sjrd commented Dec 8, 2021

No description provided.

@sjrd sjrd requested a review from gzm0 December 8, 2021 11:01
@sjrd
Copy link
Member Author

sjrd commented Dec 10, 2021

Ping @gzm0 :)

@gzm0
Copy link
Contributor

gzm0 commented Dec 10, 2021

Thanks for the ping. This indeed slipped through the cracks.

There seems to be a lot in 1.8.0, so I'll for sure need until the end of the day to review this fully.

@sjrd
Copy link
Member Author

sjrd commented Dec 10, 2021

Sure, no problem. :) There's no rush.

Copy link
Contributor

@gzm0 gzm0 left a comment

Choose a reason for hiding this comment

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

Just minor typos.

^
{% endhighlight %}

As the warning says, the recommended fix is to use the exuctor provided by [`scalajs-macrotask-executor`](https://github.com/scala-js/scala-js-macrotask-executor), as an external dependency.
Copy link
Contributor

Choose a reason for hiding this comment

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

to use the executor.

new Child()
{% endhighlight %}

Attempting to use `js.new.target` anywhere but in the constructor of a non-native JS class will result in compile error.
Copy link
Contributor

Choose a reason for hiding this comment

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

result in a compile error?

@sjrd
Copy link
Member Author

sjrd commented Dec 10, 2021

Thanks :)

@sjrd sjrd merged commit ea9aa99 into scala-js:main Dec 10, 2021
@sjrd sjrd deleted the scalajs-1.8.0 branch December 10, 2021 15:47
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.

2 participants