Skip to content
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

find-specs broken - no way to extract stage 3 TC39 proposals #1031

Closed
tidoust opened this issue Aug 21, 2023 · 2 comments
Closed

find-specs broken - no way to extract stage 3 TC39 proposals #1031

tidoust opened this issue Aug 21, 2023 · 2 comments

Comments

@tidoust
Copy link
Member

tidoust commented Aug 21, 2023

TC39 stage 3 proposals are extracted from a rendered version of a README.md file fetched from GitHub with JSDOM. GitHub now seems to be making an extensive use of React to load the contents of the page dynamically, and this does not seem to work well with JSDOM. The find-specs script fails accordingly as it cannot find the <table> elements.

We could perhaps to switch to Puppeteer. Or retrieve the markdown itself and parse it or render it ourselves with some dedicated library.

(Also note the default branch name was renamed to main but the URL of the README file is not the cause of the issue).

@dontcallmedom
Copy link
Member

switching to puppeteer would allow to revisit the use of spec-generator we took in #944

tidoust added a commit that referenced this issue Aug 28, 2023
Puppeteer is more heavy-weight but JSDOM cannot easily load specs and GitHub
pages. This update:
- fixes the find-specs script (#1031)
- removes the dependency on the spec-generator service
tidoust added a commit that referenced this issue Aug 28, 2023
Puppeteer is more heavy-weight but JSDOM cannot easily load specs and GitHub
pages. This update:
- fixes the find-specs script (#1031)
- removes the dependency on the spec-generator service
@tidoust
Copy link
Member Author

tidoust commented Aug 28, 2023

Switched to Puppeteer.

@tidoust tidoust closed this as completed Aug 28, 2023
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

No branches or pull requests

2 participants