Skip to content
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

clang-format: add rule for pch.h and generated files #2767

Merged
1 commit merged into from Jul 15, 2019

Conversation

marlenecota
Copy link
Contributor

@marlenecota marlenecota commented Jul 13, 2019

This makes it so pch.h is always at the top and cppwinrt generated files get moved to below the "main" header.

File: Foo.cpp

#include pch.h
#include Foo.h
#include Foo.g.cpp
Microsoft Reviewers: Open in CodeFlow

@marlenecota marlenecota requested a review from a team as a code owner July 13, 2019 05:27
@marlenecota marlenecota added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Jul 13, 2019
@ghost
Copy link

ghost commented Jul 13, 2019

Hello @marlenecota!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 60 minutes, a condition that will be fulfilled in about 57 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit ba92a18 into microsoft:master Jul 15, 2019
@marlenecota marlenecota deleted the clang-includes branch September 8, 2020 21:54
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants