Skip to content

build: switch docs site to application builder #30988

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

Merged
merged 1 commit into from
Apr 30, 2025

Conversation

crisbeto
Copy link
Member

Switches the docs site to use the application builder which should be faster than the existing one.

@crisbeto crisbeto added the target: major This PR is targeted for the next major release label Apr 30, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Apr 30, 2025
@crisbeto crisbeto force-pushed the docs-application-builder branch 2 times, most recently from 3714ac4 to 44e8206 Compare April 30, 2025 15:33
Switches the docs site to use the `application` builder which should be faster than the existing one.
@crisbeto crisbeto force-pushed the docs-application-builder branch from 44e8206 to ad79013 Compare April 30, 2025 16:19
@crisbeto crisbeto marked this pull request as ready for review April 30, 2025 16:25
@crisbeto crisbeto requested review from a team as code owners April 30, 2025 16:25
@crisbeto crisbeto requested review from mmalerba and andrewseguin and removed request for a team April 30, 2025 16:25
@@ -55,6 +55,5 @@ export async function buildDocsSite() {
await $`pnpm bazel build --config=snapshot-build //docs:build.production`;
await $`rm -Rf docs/dist`;
await $`mkdir -p docs/dist`;
await $`cp -R dist/bin/docs/material-angular-io.production docs/dist`;
await $`chmod -R u+w docs/dist/material-angular-io.production`;
Copy link
Member

Choose a reason for hiding this comment

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

We probably still need the chmod? I think there was a reason it's needed (e.g. when dist is cleared afterwards again)

@devversion devversion added the action: merge The PR is ready for merge by the caretaker label Apr 30, 2025
@devversion devversion merged commit f4b7afe into angular:main Apr 30, 2025
24 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants