-
Notifications
You must be signed in to change notification settings - Fork 717
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
link float16array tests PR #492
base: main
Are you sure you want to change the base?
Conversation
@@ -34,7 +34,7 @@ Stage 2 indicates that the committee expects these features to be developed and | |||
| [Sync Iterator helpers][iterator-helpers] | Gus Caplan | Michael Ficarra<br />Jonathan Keslin<br />Kevin Gibbons | <sub>[iterator-helpers][iterator-helpers-tests]</sub> | <sub>[November 2022][iterator-helpers-notes]</sub> | | |||
| [Explicit Resource Management][resource-management] | Ron Buckton | Ron Buckton | <sub>[explicit-resource-management][resource-management-tests] | <sub>[March 2023][resource-management-notes]</sub> | | |||
| [ArrayBuffer transfer][arraybuffer-transfer] | Shu-yu Guo<br />Jordan Harband<br />Yagiz Nizipli | Shu-yu Guo<br />Jordan Harband<br />Yagiz Nizipli | <sub>[arraybuffer-transfer][arraybuffer-transfer-tests]</sub> | <sub>[January 2023][arraybuffer-transfer-notes]</sub> | | |||
| [Float16 on TypedArrays, DataView, `Math.f16round`][float16s] | Leo Balter | Leo Balter<br />Kevin Gibbons | :question: | <sub>[May 2023][float16s-notes]</sub> | | |||
| [Float16 on TypedArrays, DataView, `Math.f16round`][float16s] | Leo Balter | Leo Balter<br />Kevin Gibbons | N/A but [relevant PR](https://github.com/tc39/test262/pull/3849) | <sub>[May 2023][float16s-notes]</sub> | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why would it be N/A?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The linked PR doesn't add a feature flag. I'll follow up with more tests which add one later.
I guess "none yet" is maybe more precise?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahh k got it. what about leaving the question mark, but adding like see [PR](…)
or something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, that's good too, no preference.
No description provided.