ngx-openlayers icon indicating copy to clipboard operation
ngx-openlayers copied to clipboard

Allow using a fork instead of an npm package or patch

Open HarelM opened this issue 6 years ago • 3 comments

Current implementation of this package does not allow contributor to test their fork before submitting a pull request. Since in next version the dist folder is no longer checked-in (which is good) one cannot fork and use npm install git+.... In the npm documentation there's a prepare script that might facilitate for this. However, since the repository has two goal - the library and the example it creates a problem for this. One solution is to split this to two repositories - one for the example and the other for the library, the other option is to force the prepare script to copy/remove files in order to be able to create the library so it will be possible to reference it. I don't have a magical solution, I just want to be able to use my fork without creating an npm package. If you are familiar with other solutions I would love to hear.

HarelM avatar Dec 17 '18 19:12 HarelM

First, thanks for pointing this use case, I never used it and just discovered the prepare script. Unfortunately this project could not be splitted into two distinct repositories for the moment because it is builded with Angular CLI whiches force to generate a main project (demo here) before creating a library.

I tried to install from git using a subfolder with NPM but sadly it is not possible at now :cry: here is the issue : https://github.com/npm/npm/issues/2974

Neonox31 avatar Jan 11 '19 20:01 Neonox31

Thanks for the info. I'm currently using a patch of a local version I created. I would say that if the rate pull requests adoption was decent this use case would not have been something I needed (although someone else might), but I have a pull request that is waiting for more than a month without any comment so I gave up on trying to contribute and I'm now looking for a way to use the local changes I did... I'm planning on using this library in production and would love to help out (much like the two PRs I sent), but unfortunately my changes are not being used...

HarelM avatar Jan 11 '19 20:01 HarelM

Sorry for that, we have a lot of work to do simultaneously of this project. I just mentionned maintainers for your P.R.

Neonox31 avatar Jan 11 '19 21:01 Neonox31