APA-Technology-Division / urban-and-regional-planning-resources

Community list of data & technology resources concerning the built environment and communities. πŸ™οΈπŸŒ³πŸšŒπŸš¦πŸ—ΊοΈ
Creative Commons Zero v1.0 Universal
257 stars 50 forks source link

Add 3 R packages #38

Closed Bikingman closed 3 years ago

Bikingman commented 3 years ago

This repository is a great resource! I looked it over and thought that I'd offer my 2cents and suggest adding stplanr, tidyverse, and r5r to the list of valuable R packages for transportation planning and modeling. I took a quick look around the Other Technology Resource Aggregators pages and didn't see any of them readily listed. This also eliminates a number of trailing spaces.

d-wasserman commented 3 years ago

Our lint checks can't run with the existing conflicts. Can you resolve them? Also can you break up the contributions so that you only contribute to one package per commit?

Bikingman commented 3 years ago

Thanks, @d-wasserman - Resolved the conflicts - a few resources were added since I forked.

Can you clarify what you mean by, 'break up the contributions so that you only contribute to one package per commit?' Do you mean one resource per commit? Or in other words, should I commit all three R packages over three separate commits?

d-wasserman commented 3 years ago

You have format changes entangled with your contributions, and commit titles that don't specify which did what. We have been trying to limit contributions to one commit at a time. #37 is an example.

If the three r package additions can have a clear commit message , that would help.

Bikingman commented 3 years ago

Ah, sounds good. I thought those format changes only removed trailing spaces (done automatically by my IDE). I'll update the commits

d-wasserman commented 3 years ago

I might suggest opening a new PR.

I appreciate the the format updates BTW, but I just want to isolate contributions and format changes.

It also boosts your contribution count! For what that is worth.

Bikingman commented 3 years ago

Will do!