microsoft / microsoft/nodejstools

Define NodeExePath within MSBuild Target

Open
#1,598 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement fun-size up-for-grabs
Dominant language
C#
Stars
1.8k
Forks
355
PR merge metrics
No merged PRs in 30d

Description

When the NodeExePath property is left undefined (and so relying on the default installed path) there doesn't appear to be any access for custom MSBuild rules to use the "default" value.
Could this default value be set on the property (preferably somewhere before the Build target) so that it is possible for MSBuild targets to make use of it?
For our own case - in brief - we import a custom MSBuild .targets file to run the locally installed webpack, parse the output, fill out the VS Error List and "npm pack" the package if the build was successful.
Currently this is possible but it is made flimsy by implicit calls to node/npm (relying on PATH to be setup in the right order) or explicit calls (not good for source control).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no files or tests; start by locating the MSBuild target that defines or consumes NodeExePath and inspect how the default installed path is resolved. Make that default available before the Build target, then verify that custom MSBuild targets can use it without relying on PATH or implicit node/npm calls.

Written by the indexing model from the issue text.

Assessment

Tech stack
node.js, webpack
Domain
build-system
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.