leaflet-blurred-location icon indicating copy to clipboard operation
leaflet-blurred-location copied to clipboard

Make progress on good code formatting and style with CodeClimate

Open jywarren opened this issue 6 years ago • 6 comments

https://codeclimate.com/github/publiclab/leaflet-blurred-location has a bunch of suggestions to improve the code base -- let's plan for some of this over the summer!

jywarren avatar Apr 26 '18 12:04 jywarren

@mridulnagpal -- this is a great resource for making code more maintainable and readable. I know you're working on a few things already but keep this in mind in your summer schedule! It has a lot of good potential first-timers-only issues you could make as well.

jywarren avatar Apr 26 '18 12:04 jywarren

Awesome, on it!!

mridulnagpal avatar May 03 '18 00:05 mridulnagpal

This is also a great next step. Let's get that score up!

jywarren avatar Jun 06 '18 22:06 jywarren

How's this going, Mridul? Need some next steps?

jywarren avatar Jul 03 '18 23:07 jywarren

Hi @gauravano and @sagarpreet-chadha -- I am thinking we might ask some new contributors interested in JavaScript to try some of these?

https://codeclimate.com/github/publiclab/leaflet-blurred-location/issues?category=complexity&engine_name%5B%5D=structure&engine_name%5B%5D=duplication lists a lot of recommendations and has tips on how to fix them:

image

What do you think?

jywarren avatar Jan 07 '19 14:01 jywarren

And also tests are being coordinated here: https://github.com/publiclab/leaflet-blurred-location/issues/119

jywarren avatar Jan 07 '19 14:01 jywarren