Michael Chirico

testthat's diagnosis happens at run time, having these things reported at compile time by static analysis is also helpful: - as an early reminder while you're coding - as a...

The style guide doesn't really say much about indentation. Perhaps we could raise that; meanwhile, editing code that _doesn't_ violate anything in the style guide generally goes against styler's intention,...

I think "should"/"shouldn't" can only be ruled by the style guide (for our defaults, at least). If the style guide is silent and we're trying to be consistent, we should...

ah yes, my mistake. edited in the description

@lorenzwalthert please ping again when it's ready, thanks!

Adding to this issue since I found `round_date()` documentation a bit lacking today. I was trying to ascertain what `unit`s get affected by the `week_start` parameter, with the suspicion that...

I should just get rid of the `sp` stuff

Seems Edzer deleted his Twitter account. Anyway, this remains "urgent"... switching away from 'rgdal' was more painful because everything is still 'sp'-based.

WDYT about designing this as `gh_zoom()` instead? And also allowing `precision < nchar(geohashes)` (easier to implement obviously)? `gh_fill()` naming doesn't feel perfect to me, since nothing's being "filled", exactly. "zooming"...

Thanks Garnet! Sorry for the long delay. I no longer work with geohashes very much so this package has lost priority (open to new maintainers!). If I understand right, the...