Skip to content

Conversation

@meptrsn
Copy link
Contributor

@meptrsn meptrsn commented Apr 17, 2025

… route_fun

Added wt_profile passthrough so that dodgr is able to accept other values for wt_profile to override the default "bicycle" value for wt_profile.

… route_fun

Added wt_profile passthrough so that dodgr is able to accept other values for wt_profile to override the default "bicycle" value for wt_profile.
@Robinlovelace Robinlovelace marked this pull request as ready for review April 17, 2025 14:54
@meptrsn
Copy link
Contributor Author

meptrsn commented Apr 17, 2025

It seems like roxygen needs to be rerun to match up the man pages to the functions, but I don't have a local environment set up to do that. I'm working on getting situated so I can run roxygenize() on my local machine to match up the man pages. Once that's done, it should be good to merge.

@meptrsn
Copy link
Contributor Author

meptrsn commented Apr 17, 2025

It's passing the ubuntu checks, so I am confident I at least haven't broken anything inadvertently. I'm good to merge.

@Robinlovelace Robinlovelace merged commit 7f7e146 into ropensci:master Apr 17, 2025
4 of 5 checks passed
@Robinlovelace
Copy link
Member

Great job, please put the word out there if you get a chance, very happy about this!

@meptrsn meptrsn deleted the patch-1 branch April 17, 2025 21:50
@meptrsn
Copy link
Contributor Author

meptrsn commented Apr 17, 2025

Great! Thanks, I'm glad it worked out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants