-
Notifications
You must be signed in to change notification settings - Fork 449
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 pinned cmake commands view #3542
Conversation
…into dev/mookerem/pin-cmd-pallete
…pinned list in UI
Nbd, but for UI changes it would be easier to review if you add a screenshot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, the changes of implementation look good and I've tested the feature and it seems pretty cool and works well! Just some code and design changes I'd like to see if possible before merging.
One other thing, when you test, when you open VS Code for the first time, does the Pinned Commands welcome view appear within the same CMake Activity bar view with the Project Status and the Project Outline? If not, I think that it should.
Great job 👍
Also, there seem to be some build failures that may be due to flaky issues with the windows test, but I requeued to try again. |
…rosoft/vscode-cmake-tools into dev/mookerem/pin-cmd-pallete
…into dev/mookerem/pin-cmd-pallete
This change addresses item #3296
This changes adds a new view in sidebar where users can add cmake commands to pin