Closed amovar18 closed 3 months ago
Please mention a test instruction stating to install the command globally and test that
npm I psat -g
psat ...
This PR may also fix the spawn error issue mentioned in issue #677 's in newer versions of Node since we are not spawning a new server to serve CLI dashboard. Let's test that too.
@amovar18 Please resolve conflicts
Description
This PR aims to publish the following packages on NPM registry:
analysis-utils
cli-dashboard
cli
common
design-system
i18n
library-detection
Relevant Technical Choices
packagename:build
Testing Instructions
This PR has to be tested in 2 different ways: 1) When
PSAT
is installed in a project:npm i
.coreutils
which is not available on OSX so its better to install coreutils usingbrew install coreutils
.npm run local-registry:start
.npm run publish:all:local
this will build all the packages that need to be published.npm run cli
command and check the extension if it works properly.http://localhost:4873
you will see all the packages that have been published.npm init
.npm install @ps-analysis-tool/cli --registry=http://localhost:4873
.npx psat
command.npm run local-registry:start
runnpm run local-registry:stop
.2) When
PSAT
is installed globally:npm i @ps-analysis-tool/cli @ps-analysis-tool/cli-dashboard -g --registry=http://localhost:4873
.psat
command.Screenshot/Screencast
Checklist
[ ] This code is covered by unit tests to verify that it works as intended.NAFixes #589