Skip to content

build: remove git validation from default vscode settings #31310

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
Jun 7, 2025

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Jun 7, 2025

Removes the validations for git messages that were introduced at some point. It seems like having these in the project overrides the user settings and is annoying for some team members. If somebody wants to run these validations, they can add them to their own vscode settings.

Removes the validations for git messages that were introduced at some point. It seems like having these in the project overrides the user settings and is annoying for some team members. If somebody wants to run these validations, they can add them to their own vscode settings.
@crisbeto crisbeto requested a review from a team as a code owner June 7, 2025 07:35
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Jun 7, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Jun 7, 2025
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jun 7, 2025
@crisbeto crisbeto merged commit d91374b into angular:main Jun 7, 2025
27 of 29 checks passed
@crisbeto
Copy link
Member Author

crisbeto commented Jun 7, 2025

The changes were merged into the following branches: main, 20.0.x

crisbeto added a commit that referenced this pull request Jun 7, 2025
Removes the validations for git messages that were introduced at some point. It seems like having these in the project overrides the user settings and is annoying for some team members. If somebody wants to run these validations, they can add them to their own vscode settings.

(cherry picked from commit d91374b)
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: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants