MattCowgill / readabs

Download and tidy time series data from the Australian Bureau of Statistics in R
https://mattcowgill.github.io/readabs/
Other
101 stars 22 forks source link

Refactor README and Vignette(s) #227

Open MattCowgill opened 1 year ago

MattCowgill commented 1 year ago

We now have three broad groups of functions:

The README should make it clear what each of these groups of functions are, what they do, and when you would use one over the other. Each group of functions should have a separate vignette OR a clearly-delineated separate section in a single vignette.

palesl commented 1 year ago

functions work but help files are (apparently) corrupted (see below), not sure if me or the documentation. probably me honestly.

`Downloading GitHub repo mattcowgill/readabs@HEAD ── R CMD build ────────────────────────────────────────────────── ✔ checking for file ‘/private/var/folders/fg/5j7kqkrs16n_ybq3rhkzlqnswp4y/T/Rtmpp5NBaO/remotes68e43c892ebc/MattCowgill-readabs-82387c2/DESCRIPTION’ ... ─ preparing ‘readabs’: ✔ checking DESCRIPTION meta-information ... ─ checking for LF line-endings in source and make files and shell scripts ─ checking for empty or unneeded directories ─ looking to see if a ‘data/datalist’ file should be added ─ building ‘readabs_0.4.13.901.tar.gz’ Warning: invalid uid value replaced by that for user 'nobody' Warning: invalid gid value replaced by that for user 'nobody'

Installing package into ‘/Users/u1040068/Library/R/arm64/4.2/library’ (as ‘lib’ is unspecified)

MattCowgill commented 1 year ago

Very weird @palesl, I’ll check it out

MattCowgill commented 1 year ago

Hi @palesl, I can't reproduce this, which is weird! We're both on arm64 (Apple Silicon) machines, too.

Could you please try restarting R, then doing this again?

remotes::install_github("MattCowgill/readabs") 
library(readabs)
palesl commented 1 year ago

HI Matt, yes it seems to work now, thanks!