Skip to content

Conversation

@bhearsum
Copy link
Contributor

Taskgraph's version of run-task does not currently support Windows. After some investigation, it looks like much of https://bugzilla.mozilla.org/show_bug.cgi?id=1459737 was already ported to it - the only thing that we need to change is to relocate the POSIX-specific imports into a POSIX-specific function.

@bhearsum bhearsum requested a review from a team as a code owner October 30, 2025 17:27
@bhearsum bhearsum requested a review from jcristau October 30, 2025 17:27
Copy link
Contributor

@Eijebong Eijebong left a comment

Choose a reason for hiding this comment

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

LGTM, ruff isn't pleased with this though, you'll have to annotate these to appease it.

Taskgraph's version of `run-task` does not currently support Windows. After some investigation, it looks like much of https://bugzilla.mozilla.org/show_bug.cgi?id=1459737 was already ported to it - the only thing that we need to change is to relocate the POSIX-specific imports into a POSIX-specific function.
@bhearsum bhearsum enabled auto-merge (squash) October 30, 2025 18:32
@bhearsum bhearsum merged commit 309413e into taskcluster:main Oct 30, 2025
21 checks passed
bhearsum added a commit to bhearsum/taskgraph that referenced this pull request Oct 31, 2025
Taskgraph's version of `run-task` does not currently support Windows. After some investigation, it looks like much of https://bugzilla.mozilla.org/show_bug.cgi?id=1459737 was already ported to it - the only thing that we need to change is to relocate the POSIX-specific imports into a POSIX-specific function.
abhishekmadan30 pushed a commit to abhishekmadan30/taskgraph that referenced this pull request Dec 24, 2025
Taskgraph's version of `run-task` does not currently support Windows. After some investigation, it looks like much of https://bugzilla.mozilla.org/show_bug.cgi?id=1459737 was already ported to it - the only thing that we need to change is to relocate the POSIX-specific imports into a POSIX-specific function.
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