Skip to content

Update for current Google Cloud documentation #9

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

elecnix
Copy link

@elecnix elecnix commented Dec 9, 2023

Update to parse the current version of Google Cloud documentation.

The code assumed an old version of the documentation, which had different CSS styles. Also, there is no sub-product navigation, so it is removed from the CLI options.

Modernized to an ES6 module instead of CommonJS. Tested on Node v20.10.0.

A very old version of Puppeteer was in use, and needed an update.

Removed cheerio which did not seem necessary, since Puppeteer can pass query selectors to the document.

@elecnix elecnix changed the title Es6 Update for current Google Cloud documentation Dec 9, 2023
@elecnix elecnix marked this pull request as draft December 9, 2023 21:51
@elecnix elecnix force-pushed the es6 branch 3 times, most recently from e928c1c to 5da37aa Compare December 9, 2023 22:09
@elecnix elecnix marked this pull request as ready for review December 9, 2023 22:16
@elecnix
Copy link
Author

elecnix commented Dec 9, 2023

@monotosh-k can you please review this PR?

@nikitastetskiy
Copy link

Hey, do you got this working ? I am wondering if you could provide a docker or something because there are too many errors to begin with ...

@elecnix
Copy link
Author

elecnix commented Jan 27, 2024

Hey, do you got this working ? I am wondering if you could provide a docker or something because there are too many errors to begin with ...

Yes, I got it working! Did you try from my elecnix:es6 fork?

@nikitastetskiy
Copy link

Yes, it did, thanks! I forgot to switch the branch haha

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