Skip to content

build: disable formatting on styles and templates #32614

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.

Sign up for GitHub

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

Jump to bottom
Merged
crisbeto merged 1 commit into from
Jan 5, 2026
Merged

build: disable formatting on styles and templates #32614

crisbeto merged 1 commit into from
Jan 5, 2026

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Jan 5, 2026 •
edited
Loading

It looks like ng-dev enabled formatting on HTML and CSS/Sass files at some point. These changes disable it for our repo, because reformatting all the files ends up causing build failures.

Fixes #32589.

@crisbeto crisbeto added the target: minor This PR is targeted for the next minor release label Jan 5, 2026
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Jan 5, 2026
@crisbeto
build: disable formatting on styles and templates
7653bfa 
It looks like `ng-dev` enabled formatting on HTML and CSS/Sass files at some point. These changes disable it for our repo, because reformatting all the files ends up causing build failures.

Fixes angular#32589.
@crisbeto crisbeto force-pushed the reformat-templates-styles branch from 0c96870 to 7653bfa Compare January 5, 2026 08:39
@crisbeto crisbeto changed the title build: reformat style and template files build: disable formatting on styles and templates Jan 5, 2026
@crisbeto crisbeto marked this pull request as ready for review January 5, 2026 08:41
@pullapprove pullapprove bot requested review from adolgachev and mmalerba January 5, 2026 08:41
@crisbeto crisbeto added target: patch This PR is targeted for the next patch release and removed target: minor This PR is targeted for the next minor release labels Jan 5, 2026
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Jan 5, 2026
@crisbeto crisbeto merged commit 83e30e2 into angular:main Jan 5, 2026
28 of 30 checks passed
@crisbeto
Copy link
Member Author

crisbeto commented Jan 5, 2026

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

crisbeto added a commit that referenced this pull request Jan 5, 2026
@crisbeto
build: disable formatting on styles and templates (#32614)
1396042 
It looks like `ng-dev` enabled formatting on HTML and CSS/Sass files at some point. These changes disable it for our repo, because reformatting all the files ends up causing build failures.

Fixes #32589.

(cherry picked from commit 83e30e2)
@crisbeto crisbeto mentioned this pull request Jan 5, 2026
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devversion devversion devversion approved these changes

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug(devinfra): Get rid of max-line-length stylelint

2 participants

@crisbeto @devversion