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

Add support for 'Command' #161 #162

Closed
wants to merge 1 commit into from

Conversation

abizeauMaxon
Copy link

To support by feature request with issue #161

Command/Application to launch when debugging
@Irame
Copy link
Collaborator

Irame commented Nov 16, 2023

Thank you for the contribution and the work you put in!
Sorry for the late response but I haven't got time recently to maintain this extension properly. I will try to look into you PR this weekend. And thank you again, every contribution is greatly appreciated.

@abizeauMaxon
Copy link
Author

No problem, must of it is just a replicate of your work over the "Working directory".

What I realize that I would like, is to have some variable defines at the solution level, because I have to define them per project... I have over 120 plugins of a 3rd party application that I need to use, already being able to specify different version of the tools is already magic!

Thanks for this project!

@Irame
Copy link
Collaborator

Irame commented Nov 18, 2023

Hi, I just skimmed through the changes and noticed, that you chose to implement it on the master branch. It would be awesome if you could rebase this PR onto dev/main (or create a new one if that is easier). There are a lot of refactoring changes there which makes the rebase quite tedious, I think.
I also would very much appreciate someone who could test the changes I made there, because currently I don't use the extension that much.

@abizeauMaxon
Copy link
Author

I'm currently using the version I build daily right now and it works well. It does what I need.

I currently don't have much time to rebase this. But I will try to do it at somepoint.

Irame added a commit that referenced this pull request Feb 25, 2024
@Irame
Copy link
Collaborator

Irame commented Feb 25, 2024

Rebasing this onto the new project structure is very tedious so I just re-implemented it based on the implementation you provided.

@Irame Irame closed this Feb 25, 2024
@abizeauMaxon
Copy link
Author

Thanks a lot! Sorry for the burden! But this tool save my life now!! With 300 projects and having multiple app to debug, super easy to define multiple and test on demand! Thanks a lot! Will look into your build instead of my custom fork version

@Irame
Copy link
Collaborator

Irame commented Apr 19, 2024

It would be great if you could test v3.0.3. It works great for me but it seems that some people could have issues with this version (see the last few comments in issue #177).

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.

2 participants