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

Complete re-write to move to a Spreadsheet based approach #200

Merged
merged 11 commits into from
Jan 8, 2025
Merged

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Jan 8, 2025

No description provided.

Code that can only run in an appscript runtime was already under an `appscript`
folder. Code that can only run in a Node.js runtime now appears under a `node`
folder, except the main `cli.mjs` entry point (and few other historical CLI
commands). Node.js library modules appear under `node/lib`.

Code that could run in either context now appears under a `common` folder. The
goal is to move code to that folder over time to increase the common surface
between both runtimes.
Need to disable a security setting to run Puppeteer for now.
We should not need to record the project URL as such. The repository is already
linked to the project automatically.
The repository name is now read from the "Event" sheet.
@tidoust tidoust merged commit 073963e into main Jan 8, 2025
1 check passed
@tidoust tidoust deleted the refactor branch January 8, 2025 10:33
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.

1 participant