Skip to content

Commit

Permalink
Use SLACKR_CONFIG_FILE_PATH as environment variable namerather than S…
Browse files Browse the repository at this point in the history
…LACKR_CONFIG_FILE
  • Loading branch information
botan committed Aug 31, 2023
1 parent f9c6922 commit 67a2b54
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion R/slackr_setup.r
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ slackr_setup <- function(
icon_emoji = "",
incoming_webhook_url = "",
token = "",
config_file = Sys.getenv("SLACKR_CONFIG_FILE", unset = "~/.slackr"),
config_file = Sys.getenv("SLACKR_CONFIG_FILE_PATH", unset = "~/.slackr"),
echo = FALSE,
cache_dir = ""
) {
Expand Down
2 changes: 1 addition & 1 deletion README.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ The vignettes contain setup instructions and example usage:
### Config File Setup

The `slackr_setup()` function will try to read setup values from a `~/.slackr`
(you can change the default filepath by recording in the SLACKR_CONFIG_FILE
(you can change the default filepath by recording in the SLACKR_CONFIG_FILE_PATH
environment variable or supplying as an argument to the `config_file`
parameter) configuration file, which may be easier and more secure than passing
them in manually (plus, will allow you to have multiple `slackr` configurations
Expand Down
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,10 +88,10 @@ roughly the same, with only slightly differing scopes.

The `slackr_setup()` function will try to read setup values from a
`~/.slackr` (you can change the default filepath by recording in the
SLACKR_CONFIG_FILE environment variable or supplying as an argument to
the `config_file` parameter) configuration file, which may be easier and
more secure than passing them in manually (plus, will allow you to have
multiple `slackr` configurations for multiple Slack.com teams).
SLACKR_CONFIG_FILE_PATH environment variable or supplying as an argument
to the `config_file` parameter) configuration file, which may be easier
and more secure than passing them in manually (plus, will allow you to
have multiple `slackr` configurations for multiple Slack.com teams).

The file is in Debian Control File (DCF) format since it really doesn’t
need to be JSON and R has a handy `read.dcf()` function since that’s
Expand Down

0 comments on commit 67a2b54

Please sign in to comment.