Remove homegrown retina detection

[jieter]

and

• some indentation cleanup.
• add {r} to Stamen and CartoDB urls (replaces Ref #251 : Handle retina for Stamen & CartoDB tiles #257 and Stamen retina works now #251)

[jieter]

@brunob, can you have a look?

[brunob]

@jieter ok i'll do that today

[brunob]

GTM for me :)

[brunob]

@jieter can i remove the topic branch safely ?

[jieter]

Yes!

[jieter]

maybe also a good time to do new release?

[brunob]

Branch deleted and gogogo for the release :)

[brunob]

@jieter i'm ready to push a new release, what do you think about a 1.2.0 version since the retina change is not really a "small one" ?

[jieter]

👍

[brunob]

@jieter do we have a script to generate the changelog or do you make it manually ?

[jieter]

Manually, unfortunately

…

On June 5, 2018 8:16:05 PM GMT+02:00, b_b ***@***.***> wrote: @jieter do we have a script to generate the changelog or do you make it manually ? -- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: #282 (comment)

[brunob]

'k, i'm on it

[brunob]

@jieter here is the changelog for review 176a896

Feel free to push the release, i'm not sure if https://github.com/leaflet-extras/leaflet-providers/wiki is up to date about it :)

[jieter]

yay: https://www.npmjs.com/package/leaflet-providers/v/1.2.0

the wiki was still the correct procedure, except for the last step, which isn't necessary anymore, so I removed it.

Thanks for doing the changelog work!

[brunob]

@jieter maybe we could use a tool like https://github.com/conventional-changelog/standard-version for changelog+release ?

[jieter]

I never looked into it, but more automation in the deployment process would be nice