Skip to content

refactor: create an m2 system directory #31322

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 2 commits into from
Jun 12, 2025

Conversation

andrewseguin
Copy link
Contributor

No-op change that moves M2 system values to mimic M3 system values, with its own directory separated into categories

@andrewseguin andrewseguin requested a review from a team as a code owner June 10, 2025 16:42
@andrewseguin andrewseguin requested review from crisbeto and ok7sai and removed request for a team June 10, 2025 16:42
@andrewseguin andrewseguin added the target: minor This PR is targeted for the next minor release label Jun 10, 2025
WORKSPACE Outdated
@@ -159,7 +159,7 @@ load("@aspect_rules_ts//ts:repositories.bzl", "rules_ts_dependencies")

rules_ts_dependencies(
# Obtained by: curl --silent https://registry.npmjs.org/typescript/5.8.2 | jq -r '.dist.integrity'
ts_integrity = "sha512-aJn6wq13/afZp/jT9QZmwEjDqqvSGp1VT5GVg+f/t6/oVyrgXM6BY1h9BRh/O5p3PlUPAe+WuiEZOmb/49RqoQ==",
ts_integrity = "sha512-p1diW6TqL9L07nNxvRMM7hMMw4c5XOo/1ibL4aAIGmSAt9slTE1Xgw5KWuof2uTOvCg9BY7ZRi+GaF+7sfgPeQ==",
Copy link
Member

Choose a reason for hiding this comment

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

Weird that this one changed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I should update that this is 5.8.3, or leave it out. Does your workspace not require this?

Copy link
Member

Choose a reason for hiding this comment

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

I didn't get any changes like this when I was running the dev app earlier today.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hm, seems necessary for my workspace. If it doesnt break you, lets include it

@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Jun 10, 2025
@andrewseguin andrewseguin merged commit 115abee into angular:main Jun 12, 2025
19 of 20 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 target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants