studio icon indicating copy to clipboard operation
studio copied to clipboard

Use PNPM

Open KhudaDad414 opened this issue 1 year ago • 11 comments

Description

  • Following #1060 this PR migrates studio to PNPM.
  • Some other quality of life improvements.
  • [x] Use PNPM in test workflow
  • [x] Use PNPM in the deploy workflow
  • [x] Remove nodejs tag from the repo after merge

KhudaDad414 avatar Apr 27 '24 20:04 KhudaDad414

🦋 Changeset detected

Latest commit: b5d49c311067cb8f1cd5a4dfe923e65186557950

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Apr 27 '24 20:04 changeset-bot[bot]

Deploy Preview for asyncapi-studio-design-system ready!

Name Link
Latest commit b5d49c311067cb8f1cd5a4dfe923e65186557950
Latest deploy log https://app.netlify.com/sites/asyncapi-studio-design-system/deploys/663a4ab5f959e200093be4ee
Deploy Preview https://deploy-preview-1088--asyncapi-studio-design-system.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 27 '24 20:04 netlify[bot]

Deploy Preview for studio-next ready!

Name Link
Latest commit b5d49c311067cb8f1cd5a4dfe923e65186557950
Latest deploy log https://app.netlify.com/sites/studio-next/deploys/663a4ab5a2de400009e51604
Deploy Preview https://deploy-preview-1088--studio-next.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 27 '24 20:04 netlify[bot]

Deploy Preview for modest-rosalind-098b67 ready!

Name Link
Latest commit b5d49c311067cb8f1cd5a4dfe923e65186557950
Latest deploy log https://app.netlify.com/sites/modest-rosalind-098b67/deploys/663a4ab56af6290008c14a1f
Deploy Preview https://deploy-preview-1088--modest-rosalind-098b67.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Apr 27 '24 20:04 netlify[bot]

@Amzani @Shurtu-gal. PR is ready to review.

PS: Sorry for the long PR. It was like opening a can of bugs (pun intended).

KhudaDad414 avatar May 01 '24 13:05 KhudaDad414

@KhudaDad414 there are some conflicts

Shurtu-gal avatar May 02 '24 11:05 Shurtu-gal

@Shurtu-gal the conflicts has been resolved.

KhudaDad414 avatar May 03 '24 13:05 KhudaDad414

/ptal

KhudaDad414 avatar May 03 '24 13:05 KhudaDad414

@Amzani @magicmatatjahu @princerajpoot20 Please take a look at this PR. Thanks! :wave:

asyncapi-bot avatar May 03 '24 13:05 asyncapi-bot

@Amzani done.

KhudaDad414 avatar May 07 '24 15:05 KhudaDad414

/rtm

KhudaDad414 avatar May 08 '24 08:05 KhudaDad414

/help

derberg avatar Jul 17 '24 13:07 derberg

Hello, @derberg! 👋🏼

    I'm 🧞🧞🧞 Genie 🧞🧞🧞 from the magic lamp. Looks like somebody needs a hand!

    At the moment the following comments are supported in pull requests:

    - `/please-take-a-look` or `/ptal` - This comment will add a comment to the PR asking for attention from the reviewrs who have not reviewed the PR yet.
    - `/ready-to-merge` or `/rtm` - This comment will trigger automerge of PR in case all required checks are green, approvals in place and do-not-merge label is not added
    - `/do-not-merge` or `/dnm` - This comment will block automerging even if all conditions are met and ready-to-merge label is added
    - `/autoupdate` or `/au` - This comment will add `autoupdate` label to the PR and keeps your PR up-to-date to the target branch's future changes. Unless there is a merge conflict or it is a draft PR. (Currently only works for upstream branches.)
    - `/update` or `/u` - This comment will update the PR with the latest changes from the target branch. Unless there is a merge conflict or it is a draft PR. NOTE: this only updates the PR once, so if you need to update again, you need to call the command again.

asyncapi-bot avatar Jul 17 '24 13:07 asyncapi-bot