Kristjan ESPERANTO

Results 35 comments of Kristjan ESPERANTO

Oh, I missed that @khassel already tested the alternative syntax 😑 So this is not a step forward, sorry.

Okay, I don't think there is an urgent need for us to upgrade now. Should I close my PR?

`digest-fetch` still depends on `node-fetch` 2.x. So it makes sense that you replaced it too. I'll leave my PR open, if it bothers someone, we can of course close it....

What do you think about using `node-fetch-commonjs`? I have put it in #2750 as a test. I think it is better than continuing to use `node-fetch 2.x`, but technically it...

That fetch becomes a native module in nodejs was news to me. Thanks! I think we can wait for that and as you suggested then switching to the native module...

@MajorC1983 Thanks for reporting this. In the meantime, jimp is in the package.json and is installed with the modul installation. This issue can therefore be closed :slightly_smiling_face:

Just try a PR and you'll see if darickc accepts it :slightly_smiling_face:

Thanks! I think this is a useful addition. Can you eliminate the merge conflicts?

With Leaflet 1.8.0 I can also see this, but with version 1.9.2 it no longer occurs :slightly_smiling_face:

In my understanding of MIT, it is not necessary to mention the license in all files. But I think it's fine to do it. As a basis for discussion, here...