monodeploy icon indicating copy to clipboard operation
monodeploy copied to clipboard

Crash when yarn's version plugin is installed

Open jeffrson opened this issue 2 years ago • 6 comments

Describe the bug

yarn monodeploy is crashing, when yarn's version plugin is installed.

To reproduce

yarn plugin import version yarn monodeploy --dry-run

Expected behavior

No crash - should work

  • OS: [e.g. Ubuntu 20.04, Mac OSX 10.14]: Win10, Debian11
  • Monodeploy Version: [e.g. 1.0] 2.8.20
  • Node Version: [e.g. 12.16.1] 17.7.2

jeffrson avatar Mar 22 '22 22:03 jeffrson

What's your use case for using monodeploy alongside the version plugin? They're typically mutually exclusive. Either way, it definitely shouldn't crash. I'll take a look.

What version of yarn are you using? (I need to add this to the issue template :smile:)

I'm working on a v3 release, so I wonder if it works with the v3 pre-release. If it does, I can backport the relevant fixes.

noahnu avatar Mar 23 '22 05:03 noahnu

Sure, the version plugin is not needed. It's been just my repository setup when I tried monodeploy.

I'm not happy with "version" because of what you call "implicit" dependencies. That's why I look into monodeploy, which seems pretty good (although it doesn't really work for me - that's the point of my question "how does it work").

I'm using yarn 3.2.0 (current stable).

jeffrson avatar Mar 23 '22 09:03 jeffrson

I can reproduce on v3:

TypeError: Cannot read properties of undefined (reading 'fetchBase') (when initializing @yarnpkg/plugin-version, defined in /Users/noah/repos/repo/.yarnrc.yml)

noahnu avatar Apr 24 '22 17:04 noahnu

Looks like the same issue as https://github.com/yarnpkg/berry/issues/3857

noahnu avatar Apr 24 '22 17:04 noahnu

I'm getting a similar error with https://github.com/mhassan1/yarn-plugin-aws-codeartifact

TypeError: Cannot read properties of undefined (reading 'RegistryType') (when initializing @yarnpkg/plugin-aws-codeartifact

Comes from line

freshollie avatar May 31 '23 11:05 freshollie

Monodeploy has been forked under a new name Monoweave. I've copied your issue over and will address it in the monoweave project.

https://github.com/monoweave/monoweave/issues/82

noahnu avatar Mar 31 '24 19:03 noahnu