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

DOP-4247: New Github Action to build docs site artifact #77

Merged
merged 32 commits into from
Feb 6, 2024

Conversation

mmeigs
Copy link
Contributor

@mmeigs mmeigs commented Feb 1, 2024

Ticket

DOP-4247

Notes

This new custom Github Action is created to download build artifacts of a docs site so that later in the pipeline another Action can run a Lighthouse score on the built site.

It fetches the most recent build data from the Snooty Data API per the project designated, parses the data apart into a similar structure as our zipped manifests, and saves them to the process for Snooty to use in a build.

There is a companion PR in Snooty to be opened soon.

Behavior with both PRs

Cloud-docs
Server Docs

^ note above you can download the build artifacts

@seungpark
Copy link
Contributor

minor. theres a few typescript errors from the CI / TS tests. most are fixable with vscode formatter using eslint but some are not automatic

src/build-artifact/run.ts Outdated Show resolved Hide resolved
src/build-artifact/run.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@seungpark seungpark left a comment

Choose a reason for hiding this comment

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

think we would want to promisify the streams, and also return a bundle zip? CMIIW

build-artifact/action.yml Show resolved Hide resolved
src/build-artifact/run.ts Outdated Show resolved Hide resolved
src/build-artifact/run.ts Outdated Show resolved Hide resolved
build-artifact/action.yml Outdated Show resolved Hide resolved
}
});

await once(rl, 'close');

Choose a reason for hiding this comment

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

This is cool. I didn't know you can use an event like this

@mmeigs mmeigs merged commit 3e13cd2 into main Feb 6, 2024
4 checks passed
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.

3 participants