Skip to content

Add support for parsing async iterable<T> type#85

Closed
lucacasonato wants to merge 3 commits into
plinss:mainfrom
lucacasonato:async_iterable_type
Closed

Add support for parsing async iterable<T> type#85
lucacasonato wants to merge 3 commits into
plinss:mainfrom
lucacasonato:async_iterable_type

Conversation

@lucacasonato

@lucacasonato lucacasonato commented Mar 25, 2024

Copy link
Copy Markdown
Contributor

Ref whatwg/webidl#1397

DO NOT LAND UNTIL SPEC CHANGE IS LANDED

Comment thread widlparser/productions.py Outdated
self.async_iterable[0].define_markup(generator)
self.async_iterable[1].define_markup(generator)
generator.add_text(self._open_type)
self.type.define_markup(generator)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Suggested change
self.type.define_markup(generator)
generator.add_type(self.type)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. I had copied this from the self.record branch.

@plinss

plinss commented Mar 25, 2024

Copy link
Copy Markdown
Owner

Thanks for the PR, can you also update the test-expected.txt file? (See CONTRIBUTING.md)

@plinss
plinss marked this pull request as ready for review July 9, 2025 06:10
@plinss

plinss commented Jul 9, 2025

Copy link
Copy Markdown
Owner

This has been merged, not sure why GH isn't marking it as such

@plinss plinss closed this Jul 9, 2025
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