search

bedatadriven / activityinfo-R

ActivityInfo R Language Client
https://www.activityinfo.org/support/docs/R/
17 stars 12 forks source link

Rename importTable() to importRecords() #78

Closed akbertram closed 1 year ago

akbertram commented 1 year ago

I would like to rename importTable() to importRecords() so that the naming is consistent with the other methods: addRecord(), updateRecord(), etc.

I can deprecate importTable and keep it as an alias.

What do you think @Ryo-N7 @nickdickinson

Ryo-N7 commented 1 year ago

yep, I agree! this definitely makes things consistent with the other _Record() functions

nickdickinson commented 1 year ago

I've created the alias and deprecated importTable(). Also updated NEWS.