Kristjan ESPERANTO
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...