Skip to content

Add script for canonicalization of includes#633

Open
metrapoliten wants to merge 1 commit intoDesbordante:mainfrom
metrapoliten:refactor/includes/script
Open

Add script for canonicalization of includes#633
metrapoliten wants to merge 1 commit intoDesbordante:mainfrom
metrapoliten:refactor/includes/script

Conversation

@metrapoliten
Copy link
Collaborator

No description provided.

@chernishev chernishev mentioned this pull request Dec 7, 2025
@BUYT-1
Copy link
Collaborator

BUYT-1 commented Dec 7, 2025

I'm not entirely sure it should be in the repository. Just drop it in the comments of one of the includes PR.

@chernishev
Copy link
Collaborator

chernishev commented Dec 7, 2025

I'm not entirely sure it should be in the repository. Just drop it in the comments of one of the includes PR.

There are several such scripts, e.g. script for IWYU. May be we should start storing them in a centralized way (e.g. in the repo), in case if we need them?

@BUYT-1
Copy link
Collaborator

BUYT-1 commented Dec 7, 2025

I'm not entirely sure it should be in the repository. Just drop it in the comments of one of the includes PR.

There are several such scripts, e.g. script for IWYU. May be we should start storing them in a centralized way (e.g. in the repo), in case if we need them?

Can't this particular constraint be enforced with a git grep?
As for in general, maybe a separate repo in the organization can work for that?

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