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 to configure default ActiveProject #3730

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sanore
Copy link

@sanore sanore commented Apr 30, 2024

This change addresses item #1078

The purpose of this change

The following changes are proposed:
When the user configured a default project in workspace configuration, during initialization it selects automatically the selected project.

  • Add a workspace configuration option: "cmake.defaultActiveProject"
  • Selects the configured project during initialization if configuration contains a valid project name

Other Notes/Information

I've never used typescript before and therefore this is also my first pull request to vscode-cmake-tools. So please take a look and comment if I have forgot something. Thanks for your time! I'm open to suggestions for improvement

Open Questions

  • Who is responsible for all translations?
  • Are there any recommendations how to setup unit tests for this feature?
  • Do I have forgot something?

@sanore
Copy link
Author

sanore commented Apr 30, 2024

@microsoft-github-policy-service agree

@sanore sanore marked this pull request as ready for review April 30, 2024 16:57
@sanore
Copy link
Author

sanore commented May 27, 2024

Any missing information for this PR?

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.

None yet

1 participant