-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eurostat 4.0.0 #283
eurostat 4.0.0 #283
Conversation
Reading error message (http status, id and label) from the response and return the status, id and label that were saved in the memory.
Update get_eurostat_json.R
Require always sf for `get_eurostat_geospatial()`
Implement data.table functions for handling big objects in memory
Update package vignettes and articles on eurostat 4.0.0
Merge branch 'master' into v4 # Conflicts: # revdep/library.noindex/eurostat/new/lubridate/NEWS.md
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #283 +/- ##
==========================================
+ Coverage 67.18% 76.41% +9.22%
==========================================
Files 24 23 -1
Lines 1024 1132 +108
==========================================
+ Hits 688 865 +177
+ Misses 336 267 -69 ☔ View full report in Codecov by Sentry. |
Checked on devtools::check_win_devel, devtools::check_win_release and devtools::check_win_oldrelease, all ok. Checked locally under
All ok except for 1 note:
This is due to Twitter API changes, the URL works ok |
Really cool |
I suggest this version is uploaded to CRAN. I will run some final tests and post the results here before this is merged into master and submitted to CRAN