Skip to content

Implement new spec changes for AsyncGenerator - #3950

Merged
jedel1043 merged 2 commits into
mainfrom
new-async-generator
Aug 18, 2024
Merged

jedel1043 merged 2 commits into
mainfrom
new-async-generator

Conversation

@jedel1043

Copy link
Copy Markdown
Member

Essentially the same as #3879 but supported by the latest specification.

@jedel1043 jedel1043 added A-Bug Something isn't working A-Internal Changes that don't modify execution behaviour labels Aug 14, 2024
@jedel1043 jedel1043 added this to the next-release milestone Aug 14, 2024
@jedel1043
jedel1043 force-pushed the new-async-generator branch from c98c6a7 to c06b882 Compare August 15, 2024 01:28
@jedel1043
jedel1043 force-pushed the new-async-generator branch from c06b882 to 699a842 Compare August 15, 2024 01:29
@github-actions

Copy link
Copy Markdown

Test262 conformance changes

Test result main count PR count difference
Total 48,212 48,476 +264
Passed 43,346 43,446 +100
Ignored 1,413 1,507 +94
Failed 3,453 3,523 +70
Panics 0 0 0
Conformance 89.91% 89.62% -0.28%
Broken tests (2):
test/harness/asyncHelpers-throwsAsync-func-throws-sync.js (previously Passed)
test/harness/asyncHelpers-throwsAsync-no-arg.js (previously Passed)

@jedel1043

jedel1043 commented Aug 15, 2024

Copy link
Copy Markdown
Member Author

Checked and the regressions are bugs in the test suite caused by recent changes to the asyncHelpers.js harness.

@jedel1043
jedel1043 marked this pull request as ready for review August 15, 2024 03:27
@jedel1043
jedel1043 requested a review from a team August 15, 2024 03:27

@nekevss nekevss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. Couple nits but they're non-blocking.

Comment thread core/engine/src/builtins/async_generator/mod.rs
Comment thread core/engine/src/builtins/async_generator/mod.rs
Comment thread core/engine/src/builtins/async_generator/mod.rs
Comment thread core/engine/src/builtins/async_generator/mod.rs
@nekevss
nekevss requested a review from a team August 16, 2024 17:43

@HalidOdat HalidOdat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work! Looks good to me! :)

@jedel1043
jedel1043 enabled auto-merge August 18, 2024 01:09
@jedel1043
jedel1043 added this pull request to the merge queue Aug 18, 2024
Merged via the queue into main with commit c22f39b Aug 18, 2024
@jedel1043
jedel1043 deleted the new-async-generator branch August 18, 2024 04:45
jedel1043 added a commit that referenced this pull request Sep 11, 2024
* Implement new spec changes for `AsyncGenerator`

* Add panic docs
raskad pushed a commit that referenced this pull request Sep 11, 2024
* Implement new spec changes for `AsyncGenerator` (#3950)

* Implement new spec changes for `AsyncGenerator`

* Add panic docs

* Add spec edition 15 to the tester (#3957)

* Allow dead code for code that is newly detected as unused (#3984)

* Allow dead code for code that is newly detected as unused

* Fix compile errors with nightly rust

* Add missing SAFETY section

* Increase safety of `FutexWaiters`

---------

Co-authored-by: Theo Paris <theo@tinted.dev>
Co-authored-by: José Julián Espina <julian.espina@canonical.com>

* Enable CI for release branches (#3987)

* Allow warnings when running CI on release branches (#3990)

* Bump crates version

---------

Co-authored-by: Hans Larsen <hans@larsen.online>
Co-authored-by: Theo Paris <theo@tinted.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Bug Something isn't working A-Internal Changes that don't modify execution behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants