swagger-api / swagger-editor

Swagger Editor
https://editor.swagger.io
Apache License 2.0
8.87k stars 2.24k forks source link

Switch repo from NPM to yarn for improved dependency management #5084

Closed michaeljwiebe closed 3 months ago

michaeljwiebe commented 3 months ago

Description

I was attempting to use a customized version of the Swagger-UI package in my Swagger Editor but was unable to do so using npm. Switching over to yarn resolved the issue for me. Yarn doesn't require special flags to install locally like npm does (see --legacy-peer-deps).

Motivation and Context

Fixes #5000 - This PR allows me to link my local swagger-editor to a customized version of swagger-ui that I have locally using the local relative path to the repo as the version in my package.json. See package.json: Screen Shot 2024-05-24 at 5 38 38 AM

How Has This Been Tested?

I ran yarn test and have built and ran locally and verified that the swagger-editor works and swagger-ui is importing correctly.

Screenshots (if appropriate):

Checklist

My PR contains...

My changes...

Documentation

Automated tests

char0n commented 3 months ago

Hi @michaeljwiebe,

Thanks for the effort but we're not be changing the master branch to yarn. master branch contains SwaggerEditor v4 implementation which is considered (unofficially) obsolete at this point in time.

Please use https://editor-next.swagger.io/ and next branch of this repo to access SwaggerEditor v5.