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

Merge with Include Minimizer, New Maintainer #82

Open
Agrael1 opened this issue Jul 5, 2022 · 2 comments
Open

Merge with Include Minimizer, New Maintainer #82

Agrael1 opened this issue Jul 5, 2022 · 2 comments

Comments

@Agrael1
Copy link
Collaborator

Agrael1 commented Jul 5, 2022

Hello,

I'd like to propose a new update to the extension. I'm currently working on a separate extension with IWYU integration.
Its capabilities are the same, but it has a built-in map generator for IWYU, also it solves IWYU includes asynchronously.
The extension is tested on commercial project, and it has shown great results if used correctly. Also it supports VS2022

I'd like to become a maintainer and try merging projects together, also updating IWYU part.
Here is my project:
https://github.com/Agrael1/IncludeMinimizer

@Wumpf
Copy link
Owner

Wumpf commented Jul 5, 2022

I'm happy to have a new maintainer for this project who can look after all the various issues that crop up from time to time.
But it's a bit unclear to me why you need maintainer status for what you're planning - if you're looking to get features over to your project you can do so within the bounds IncludeToolbox license (MIT, so pretty the only restriction is mentioning :)).
Also, I'd hope that a potential future maintainer would first show up with a few PRs before I toss over the keys 😄

@Agrael1
Copy link
Collaborator Author

Agrael1 commented Jul 6, 2022

Well, I was unsure if the project owner was still here, and my PRs would not go unnoticed. Also I have a revoke plan, I'd like to discuss, because IWYU has moved a bit and some repo changes has to be made, also a test pipeline would be great to ensure functionality. Those thing go beyond PR, so I asked for maintenance.

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

No branches or pull requests

2 participants