Mara

Oh also I put the uncrustify config I used into the repo. To run this what I did was ``` for f in src/*.*; do uncrustify -c uncrustify.cfg --no-backup -l...

Hey @matjack1 thanks for the quick response. I personally find mixing in style changes with function changes make history harder to read because then when looking at a diff there...

What about ``` if (stuff) { ... } ``` vs ``` if (stuff) { ... } ``` ? I thought there was only the former but I just found some...

@elesdoar That doesn't set the attribution. And it also only works with the master branch, which is "not suitable for use at this point", so I'm using the 1.X branch.

Well, still an issue but the above code doesn't run, because the leaflet mapbox thing updated to leaflet 1. So I just redid the dependencies. ``` ui-leaflet mapbox-gl-leaflet body {...

Nice, thank you! I do see it working, I just forgot to reply earlier.

We use the PR template to have a set of manual steps in checkboxes, and use a thing to make all checkboxes a required status check before merging. Thinking about...

@thetic Are you running apple clang? If so I bet it's the same things and I could remove -Wno-suggest-override from this PR and that would be cool. This PR is...

I know I tested this when I made it but now it's not working so I've done something wrong. I"ll try to fix and re-open soon.

It works now. Should we hide the command being run? I don't think seeing ``` $ make test ( rm -f .cpputestBuild/objs/../src/lib.gcda .cpputestBuild/objs/./testrunner.gcda .cpputestBuild/objs/./tests.gcda gcov_output.txt gcov_report.txt gcov_error.txt ; echo "Running...