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

Avoid IWYU warnings on other namespaces for <algorithm>. #722

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from

Conversation

pkasting
Copy link

@pkasting pkasting commented Nov 3, 2022

This is an extension of the logic already used for other STL headers.

This is an extension of the logic already used for other STL headers.
@pkasting
Copy link
Author

pkasting commented Jan 3, 2023

I don't have permission to merge the PR; can you either merge it or grant me write access?

@ibrahimmemonn
Copy link

I don't have permission to merge the PR; can you either merge it or grant me write access?

I can't merge :( I don't have access but I'll request for it

@tkruse
Copy link

tkruse commented Jan 30, 2023

Hi, as far as I am aware, google does not accept PRs for cpplint anymore (since the google-internal cpplint supposedly has evolved a lot. However, this MR might be suitable for https://github.com/cpplint/cpplint, the pypi deployed fork which I have been maintaining.

Would this also fix cpplint#200 ?

@pkasting
Copy link
Author

I don't know offhand, but it might. You're welcome to merge this code into your fork if you wish.

abeir pushed a commit to abeir/depot_tools that referenced this pull request Apr 24, 2024
Cherry-pick of google/styleguide#722.

Bug: chromium:1368812
Change-Id: I3a604ce2e617b3f145c02519ec3528bff22f691c
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.

None yet

5 participants