Skip to content

Conversation

@jeremymeng
Copy link
Member

@jeremymeng jeremymeng commented Dec 1, 2025

There are some improvements in prettier v3.7 but it brings differences
comparing to v3.6.

This PR fixes formatting checks by applying the result of turbo run format for
the whole repo, after our weekly automation upgrades prettier to v3.7.

NO_CI

There are some improvements in prettier v3.7 but it brings differences
comparing to v3.6.

This PR fixes formatting checks by applying the result of `turbo run format` for
the whole repo, after our weekly automation upgrades prettier to v3.7.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR applies automatic formatting changes from Prettier v3.7 across the Azure SDK for JavaScript repository. The changes are purely cosmetic, adjusting code formatting to comply with the newer version of Prettier that was upgraded by the weekly automation.

The primary change pattern involves reformatting TypeScript interface declarations, particularly how extends clauses are formatted across multiple lines. Prettier v3.7 appears to have different line-breaking rules compared to v3.6, resulting in more compact formatting for shorter declarations and different indentation for longer ones.

Reviewed changes

Copilot reviewed 238 out of 242 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Multiple options.ts files Interface declarations with extends OperationOptions reformatted to single lines
Multiple model files Interface extends clauses reformatted for better readability
Test setup files ProvidedContext interface extends Omit<> reformatted with new line breaks
Identity credential files Credential options interfaces reformatted with new indentation
Various TypeScript files Boolean expressions and import statements reformatted

Copy link
Member

@maorleger maorleger left a comment

Choose a reason for hiding this comment

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

I think the merged commit still needs a ***NO_CI*** text, PR LGTM

@jeremymeng
Copy link
Member Author

/check-enforcer override

@jeremymeng
Copy link
Member Author

I think the merged commit still needs a ***NO_CI*** text,

The CI would populate the turbo build cache. But it's not a big deal as next CI would do that too.

@jeremymeng jeremymeng merged commit 76bc362 into Azure:main Dec 1, 2025
104 of 129 checks passed
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Azure Identity SDK Improvements Dec 1, 2025
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Azure SDK for Key Vault Dec 1, 2025
@jeremymeng jeremymeng deleted the engsys/fix-formatting-dec-2025 branch December 1, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants