Skip to content

Conversation

@qaisjp
Copy link
Member

@qaisjp qaisjp commented Jan 11, 2026

Summary

  • Make clang-format.ps1 executable
  • Run ./utils/clang-format.ps1 to format all files
  • Add codemod to .git-blame-ignore-revs

Motivation

Make master green after #4626

Test plan

Changes were generated entirely by ./utils/clang-format.ps1

@qaisjp qaisjp requested a review from a team as a code owner January 11, 2026 03:09
return true;
}
}
} // namespace CefAppAuth
Copy link
Member Author

Choose a reason for hiding this comment

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

This comment is weird

Copy link
Member Author

Choose a reason for hiding this comment

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

This is FixNamespaceComments which we have set to true explicitly:

FixNamespaceComments: true

Keep or no keep?

Copy link
Member

Choose a reason for hiding this comment

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

I think disable that. IDEs are more than capable of tracking namespaces these days.

Comment on lines -121 to +132
if (std::getenv("WINE") || std::getenv("WINEPREFIX")) {
if (std::getenv("WINE") || std::getenv("WINEPREFIX"))
{
Copy link
Member Author

Choose a reason for hiding this comment

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

😢 @botder

@qaisjp qaisjp changed the title Run ./utils/clang-format.ps1 [Ready to review, but don't merge yet] Run ./utils/clang-format.ps1 Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants