Skip to content

Conversation

jessehouwing
Copy link
Contributor

@jessehouwing jessehouwing commented Jan 8, 2025

Fixes: #480

For the supported task runners:

  • ["Node", "Node10", "Node16", "Node20_1", "PowerShell", "PowerShell3", "Process"]
  • Checks the target has been supplied.
  • Checks the target to ensure the file exists on disk.

Has support for $(currentfolder) and skips validation when the target contains any other variable (such as $(filename)).

@jessehouwing jessehouwing requested review from a team as code owners January 8, 2025 15:04
@jessehouwing jessehouwing force-pushed the features/validateExecutionTarget branch from b063078 to 10fff44 Compare February 28, 2025 12:38
@microsoft microsoft deleted a comment from azure-pipelines bot Mar 6, 2025
@jessehouwing
Copy link
Contributor Author

@tarunramsinghani Any changes required to get this merged?

@tarunramsinghani
Copy link
Contributor

Changes looks good. Approved.

@tarunramsinghani
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tarunramsinghani tarunramsinghani merged commit 833fcb2 into microsoft:master Apr 10, 2025
4 checks passed
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.

Would be nice if the task.json validation would check for the existence of the execution.*.target
3 participants