Skip to content

Fix OSM vignette#304

Merged
cforgaci merged 5 commits into
mainfrom
osm-vignette-cf
Jul 23, 2025
Merged

Fix OSM vignette#304
cforgaci merged 5 commits into
mainfrom
osm-vignette-cf

Conversation

@cforgaci
Copy link
Copy Markdown
Contributor

No description provided.

@cforgaci cforgaci added the documentation Improvements or additions to documentation label Jul 22, 2025
@cforgaci cforgaci added this to the 0.2.0 milestone Jul 22, 2025
@cforgaci cforgaci requested a review from fnattino July 22, 2025 12:26
@cforgaci cforgaci linked an issue Jul 22, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@fnattino fnattino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, building this vignette using OSM data seems not to create issues! Just two small comments, feel free to merge afterwards.

Comment thread vignettes/vig_02-getting-osm-data.Rmd Outdated
Comment thread vignettes/vig_02-getting-osm-data.Rmd Outdated
cforgaci and others added 3 commits July 23, 2025 17:16
Co-authored-by: Francesco Nattino <49899980+fnattino@users.noreply.github.com>
@cforgaci cforgaci requested a review from fnattino July 23, 2025 15:23
@cforgaci cforgaci merged commit be5887b into main Jul 23, 2025
8 checks passed
@cforgaci cforgaci deleted the osm-vignette-cf branch July 23, 2025 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Getting OSM data vignette

2 participants