Skip to content

Conversation

@alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Nov 19, 2025

This commit migrates the schema generation process to utilize Bazel's tagging system instead of a custom build script.

  • The scripts/build-schema.mts script has been removed.
  • A new build-schema script has been added to package.json that executes bazel build for targets tagged with "schema".
  • The CI/PR workflows (.github/workflows/ci.yml and pr.yml) have been updated to use this new pnpm run build-schema command.
  • The ng_cli_schema_generator.bzl and ts_json_schema.bzl rules have been updated to include the "schema" tag, allowing them to be picked up by the new build process.
  • This also reduces the schema generation by around 20s.

This commit migrates the schema generation process to utilize Bazel's tagging system instead of a custom build script.

- The `scripts/build-schema.mts` script has been removed.
- A new `build-schema` script has been added to `package.json` that executes `bazel build` for targets tagged with "schema".
- The CI/PR workflows (`.github/workflows/ci.yml` and `pr.yml`) have been updated to use this new `pnpm run build-schema` command.
- The `ng_cli_schema_generator.bzl` and `ts_json_schema.bzl` rules have been updated to include the "schema" tag, allowing them to be picked up by the new build process.
@alan-agius4 alan-agius4 requested a review from clydin November 19, 2025 13:52
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Nov 19, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Nov 19, 2025
@alan-agius4
Copy link
Collaborator Author

FYI: @hybrist & @dgp1130

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 19, 2025
@dgp1130 dgp1130 merged commit dfe021b into angular:main Nov 19, 2025
63 checks passed
@dgp1130
Copy link
Collaborator

dgp1130 commented Nov 19, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants