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

Allow customization of build tool tasks #310

Merged
merged 6 commits into from
Oct 26, 2020

Conversation

anton-matosov
Copy link

@anton-matosov anton-matosov commented Sep 29, 2020

Implements #305

Add args parameter to all build tool tasks that allows customization in the workspaces's tasks.json by users
Implement generic task named ros
Add roscore and roslaunch commands using ros task that allows user to run and customize them in tasks.json

@anton-matosov anton-matosov requested a review from a team as a code owner September 29, 2020 17:28
@seanyen
Copy link

seanyen commented Sep 29, 2020

Thank @anton-matosov! Can you rebase to reduce the noise on the diff view?

@anton-matosov
Copy link
Author

Definitely. my bad

Add `args` parameter to all build tool tasks that allows customization in the workspaces's tasks.json by users
Implement generic task of `ros` type
Add `roscore` and `roslaunch` commands using `ros` task that allows user to run and customize them in `tasks.json`
Add `roscore and  `roslaunch` problem matchers that allow corresponding tasks to run in background
@@ -11,6 +11,5 @@
"search.exclude": {
"out": true // set this to false to include "out" folder in search results
},
"typescript.tsdk": "./node_modules/typescript/lib", // we want to use the TS server from our node_modules folder to control its version
"python.pythonPath": "C:\\opt\\python27amd64\\python.exe"
"typescript.tsdk": "./node_modules/typescript/lib"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the automatic upgrade of settings by corresponding vscode extensions

@seanyen
Copy link

seanyen commented Oct 10, 2020

@anton-matosov, LGTM. Is it ready for merge too?

doc/debug-support.md Outdated Show resolved Hide resolved
@ooeygui
Copy link
Member

ooeygui commented Oct 19, 2020

I'm testing this on Windows. @anton-matosov can you update your branch so that the PR is in sync?

@anton-matosov
Copy link
Author

Merged in latest master and updated docs

@ooeygui ooeygui merged commit 92f0aec into ms-iot:master Oct 26, 2020
@anton-matosov anton-matosov deleted the catkinArgs branch February 14, 2024 05:16
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

3 participants