search

bedatadriven / activityinfo-R

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

Respect standard interactive() session flag #86

Closed akbertram closed 1 year ago

akbertram commented 1 year ago

@nickdickinson I'm not sure I see the rationale for have a seperate ActivityInfo interactive flag, especially for authentication. If we are running from Rscript or other non-interactive context, we should not be prompting to save tokens.

nickdickinson commented 1 year ago

It should be respected. I believe you may be looking at the code required to run tests emulating an interactive session. interactive2() is internal to ensure we can run tests but otherwise, unless you use the undocumented option, it is only there for tests.

akbertram commented 1 year ago

Ah ok got it!