Skip to content

Add support for custom task location or command#63

Merged
robgolding merged 4 commits intodevelopfrom
custom-task-command-location
May 13, 2019
Merged

Add support for custom task location or command#63
robgolding merged 4 commits intodevelopfrom
custom-task-command-location

Conversation

@robgolding
Copy link
Copy Markdown
Collaborator

Thanks @xarthurx for proposing WSL support in #62. This commit adds support in a different way, by allowing the "task" command to be customised (to wsl task, for example).

@robgolding robgolding force-pushed the custom-task-command-location branch from 3df0653 to 7e5407b Compare May 6, 2019 17:34
robgolding added 2 commits May 6, 2019 14:45
Thanks @xarthurx for proposing WSL support in #62. This commit adds
support in a different way, by allowing the "task" command to be
customised (to `wsl task`, for example).
@robgolding robgolding force-pushed the custom-task-command-location branch from 7e5407b to ee7b486 Compare May 6, 2019 18:46
@xarthurx
Copy link
Copy Markdown

xarthurx commented May 7, 2019

@robgolding is this PR done?
I'm a bit busy this week but will give it a run during weekends if it is done.

@robgolding
Copy link
Copy Markdown
Collaborator Author

The tests have failed here but only because wsl isn't present on CI—I'll need to figure out a way around that. It should be ready to give this a go locally @xarthurx!

Should be more compatible with Windows platforms.
@robgolding
Copy link
Copy Markdown
Collaborator Author

@xarthurx OK, I switched to using TASKRC env var. Can you give this another go?

@xarthurx
Copy link
Copy Markdown

@robgolding This works fine! :)

`wsl` doesn't exist on Travis CI!
@robgolding
Copy link
Copy Markdown
Collaborator Author

Great, thanks @xarthurx. In that case since CI is now passing I'll get this merged and release a new version. Thanks again for the help with this!

@robgolding robgolding merged commit 8280d24 into develop May 13, 2019
@robgolding robgolding deleted the custom-task-command-location branch May 13, 2019 14:38
jspricke added a commit to jspricke/tasklib that referenced this pull request Jan 15, 2021
The test_custom_command was introduced in GothenburgBitFactory#63 with the TASK_VERSION for
Travis. As TASK_VERSION is not part of a default environment, the test
would still try to run `wsl` on other systems and fail.
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