Skip to content

cdktn-cli: Drop cross-spawn? #85

@commiterate

Description

@commiterate

Description

Tracking issue for #55 (comment).

Not needed in the short to medium term. Might be needed in the long term since cross-spawn seems unmaintained and has some problematic dependency version pins.

Things to do:

  • Re-enable Windows tests in CI (sanity check for when we attempt to drop cross-spawn).
  • Attempt to drop cross-spawn by selectively setting the shell option on node:child_process.spawn() for Windows only (see original comment).

Help Wanted

  • I'm interested in contributing a fix myself

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions