Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VS Code Package Setup #6

Merged
merged 10 commits into from
Mar 15, 2023
Merged

VS Code Package Setup #6

merged 10 commits into from
Mar 15, 2023

Conversation

rudyflores
Copy link
Collaborator

@rudyflores rudyflores commented Mar 10, 2023

Setup for VS Code package in the monorepo, currently this extension when installed only displays an information message for mentioning the extension has activated successfully, but is a good start to the upcoming work.

The VS Code package is handled with the following for bundling & minimizing:

  • tsup minimizer
  • turbopack

The use of turbopack can build packages in the monorepo using caching now!

For testing try running the following:

  • npm install
  • npm run build
  • npm run package

if you run npm run build several times you'll notice builds become faster and if changes are made it will dynamically bundle the necessary packages!

Try testing installing both packages in the /dist folder of the root directory, when installing the vsix it should display a message on the bottom right saying Extension has activated once the extension activates (this is only for now for testing purposes).

Extra: if clicking the play button in the Run & Debug panel in VS Code, it will open the development extension host which will allow testing of the extension and set breakpoints in future development of the repo!

Signed-off-by: Rudy Flores <[email protected]>
Signed-off-by: Rudy Flores <[email protected]>
Signed-off-by: Rudy Flores <[email protected]>
Signed-off-by: Rudy Flores <[email protected]>
@rudyflores rudyflores requested a review from traeok March 13, 2023 13:26
Signed-off-by: Rudy Flores <[email protected]>
Copy link
Member

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Turbopack looks awesome, maybe we can use it in Zowe Explorer v3 😀 Left a few comments

package.json Show resolved Hide resolved
packages/vscode/.vscodeignore Outdated Show resolved Hide resolved
packages/vscode/package.json Show resolved Hide resolved
Signed-off-by: Rudy Flores <[email protected]>
@rudyflores rudyflores requested a review from t1m0thyj March 13, 2023 14:54
Copy link
Member

@traeok traeok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, the caching for turbopack is neat. Thanks @rudyflores !

I noticed some errors when trying to run npm run package, will approve once fixed 😄

npm ERR! code ENOENT
npm ERR! syscall open
npm ERR! path C:\Users\Trae\zowe-cli-secrets-for-kubernetes\dist\zowe-secrets-for-kubernetes-for-zowe-cli-1.0.0.tgz
npm ERR! errno -4058
npm ERR! enoent ENOENT: no such file or directory, open 'C:\Users\Trae\zowe-cli-secrets-for-kubernetes\dist\zowe-secrets-for-kubernetes-for-zowe-cli-1.0.0.tgz'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\Trae\AppData\Local\npm-cache\_logs\2023-03-13T14_55_05_107Z-debug-0.log
npm ERR! Lifecycle script `package` failed with error:
npm ERR! Error: command failed
npm ERR!   in workspace: @zowe/[email protected]
npm ERR!   at location: C:\Users\Trae\zowe-cli-secrets-for-kubernetes\packages\cli
ENOENT: no such file or directory, open 'C:\Users\Trae\zowe-cli-secrets-for-kubernetes\dist\Zowe.secrets-for-kubernetes.vsix'

Signed-off-by: Rudy Flores <[email protected]>
Signed-off-by: Rudy Flores <[email protected]>
@rudyflores rudyflores requested a review from traeok March 13, 2023 15:14
Copy link
Member

@traeok traeok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Was able to install the extension and see the activation message. Thanks for clarifying on the missing dist folder @rudyflores 😄

Signed-off-by: Rudy Flores <[email protected]>
@rudyflores rudyflores requested a review from t1m0thyj March 15, 2023 15:52
Copy link
Member

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @rudyflores!

@rudyflores rudyflores merged commit d494ce0 into main Mar 15, 2023
@rudyflores rudyflores deleted the vsc-integration branch March 15, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants