microsoft / tfs-cli

Cross-platform CLI for Microsoft Team Foundation Server and Visual Studio Team Services
MIT License
372 stars 132 forks source link

Fix --rev-version bug #359

Closed xinyi-joffre closed 4 years ago

xinyi-joffre commented 4 years ago

Addressing the --rev-version bug in #358.

The json-in-place set function returns an Inplace type object, which was causing issues. This adds a toString so that it can be properly written back to the json file.

jpiquot commented 4 years ago

Hi,

I have the same issue with the --rev-version argument.

PS D:\dev\DevOps\DevOpsExt> tfx extension create --output-path './bin' --manifests-globs 'vss-extension.json' --rev-version (node:28916) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency (Use node --trace-warnings ... to show where the warning was created) TFS Cross Platform Command Line Interface v0.8.3 Copyright Microsoft Corporation

error: TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of Inplace

VAllens commented 4 years ago
TFS Cross Platform Command Line Interface v0.8.3
Copyright Microsoft Corporation
error: TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received an instance of Inplace
npm ERR! code ELIFECYCLE

This error only occurs in node v14.x or above

VAllens commented 4 years ago

Hello, Any progress? We need to use the --rev-version parameter in Azure DevOps Pipeline to automatically increment the version number

VAllens commented 4 years ago

Tks!!!

When can new npm packages be released?

ChrisGibson1982 commented 3 years ago

What version should this fix be available in?

VAllens commented 3 years ago

Wait indefinitely....

michaelper22 commented 3 years ago

Is there a release planned? Last build in NPM was 5 months ago.

Kelly-CLIU commented 3 years ago

This issue has been fixed in the latest version 0.9.0

VAllens commented 3 years ago

Good!!!!

abhi-msft commented 3 years ago

Issue still exists for tfx cli v0.9.2 when using node v14.17.1