search

Wumpf / IncludeToolbox

Visual Studio extension to format, prune, and inspect include directives.
https://marketplace.visualstudio.com/items?itemName=Wumpf.IncludeToolbox
MIT License
50 stars 22 forks source link

Merge with Include Minimizer, New Maintainer #82

Open Agrael1 opened 2 years ago

Agrael1 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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.