[R/Analytics.R:2:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:3:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{Analytics} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:3:85:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{Analytics} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/Analytics.R:7:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{Analytics} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:7:85:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{Analytics} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/Analytics.R:11:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:11:81:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/Analytics.R:12:80:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/Analytics.R:16:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L16) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{Analytics} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:25:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L25) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:31:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:34:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L34) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Analytics.R:43:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L43) *style:* **Variable or function name should be snake_case.**
```r
Analytics <- function(sound = 1, expr = NULL) {
^~~~~~~~~
```
[R/Analytics.R:44:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L44) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:49:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L49) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:51:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L51) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/Analytics.R:52:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L52) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/Analytics.R:54:9:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L54) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/Analytics.R:56:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L56) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Analytics.R:58:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L58) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Analytics.R:59:38:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L59) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/Analytics.R:60:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L60) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:60:11:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L60) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/Analytics.R:60:75:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L60) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/Analytics.R:66:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L66) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:70:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L70) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:70:71:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L70) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
                                                                     ~^~
```
[R/Analytics.R:70:84:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L70) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/Analytics.R:72:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:73:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L73) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/Analytics.R:73:27:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L73) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/Analytics.R:74:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L74) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:74:72:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                      ~^~
```
[R/Analytics.R:74:80:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                              ~^~
```
[R/Analytics.R:74:93:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L74) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/AFMS/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/Analytics.R:76:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L76) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Analytics.R:78:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L78) *style:* **Lines should not be more than 80 characters.**
```r
    warning("Analytics() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:83:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L83) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/Analytics.R:87:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L87) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:87:116:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L87) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/Analytics.R:88:45:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L88) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/Analytics.R:88:51:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L88) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/Analytics.R:93:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L93) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:93:70:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L93) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/Analytics.R:93:76:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L93) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/Analytics.R:98:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L98) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Analytics.R:98:94:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L98) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/Analytics.R:98:100:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L98) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/Analytics.R:103:10:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L103) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/Analytics.R:104:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L104) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/Analytics.R:108:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L108) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/Analytics.R:109:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L109) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/Analytics.R:110:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L110) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/Analytics.R:111:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L111) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/Analytics.R:112:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L112) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/Analytics.R:113:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L113) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/Analytics.R:114:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L114) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/Analytics.R:116:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L116) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/Analytics.R:119:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Analytics.R#L119) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/ChronoTrigger.R:2:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:3:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{ChronoTrigger} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:3:89:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{ChronoTrigger} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/ChronoTrigger.R:7:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{ChronoTrigger} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:7:89:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{ChronoTrigger} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/ChronoTrigger.R:11:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:11:81:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/ChronoTrigger.R:12:80:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/ChronoTrigger.R:18:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L18) *style:* **Lines should not be more than 80 characters.**
```r
#'   \code{"black_tyrano"} \item \code{"continuum"} \item \code{"destruction"} \item
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:22:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L22) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{ChronoTrigger} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:31:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L31) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:37:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L37) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:40:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L40) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/ChronoTrigger.R:49:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L49) *style:* **Variable or function name should be snake_case.**
```r
ChronoTrigger <- function(sound = 1, expr = NULL) {
^~~~~~~~~~~~~
```
[R/ChronoTrigger.R:50:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L50) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:72:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L72) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:74:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L74) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/ChronoTrigger.R:75:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L75) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/ChronoTrigger.R:77:9:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L77) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/ChronoTrigger.R:79:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L79) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/ChronoTrigger.R:81:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L81) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/ChronoTrigger.R:82:38:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L82) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/ChronoTrigger.R:83:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L83) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:83:11:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L83) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/ChronoTrigger.R:83:75:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L83) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/ChronoTrigger.R:89:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L89) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:93:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L93) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:93:69:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L93) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
                                                                   ~^~
```
[R/ChronoTrigger.R:93:82:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L93) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/ChronoTrigger.R:95:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L95) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:96:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L96) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/ChronoTrigger.R:96:27:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L96) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/ChronoTrigger.R:97:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L97) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:97:70:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                    ~^~
```
[R/ChronoTrigger.R:97:78:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                            ~^~
```
[R/ChronoTrigger.R:97:91:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L97) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/CT/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/ChronoTrigger.R:99:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L99) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/ChronoTrigger.R:101:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L101) *style:* **Lines should not be more than 80 characters.**
```r
    warning("ChronoTrigger() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:106:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L106) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/ChronoTrigger.R:110:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L110) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:110:116:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L110) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/ChronoTrigger.R:111:45:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L111) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/ChronoTrigger.R:111:51:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L111) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/ChronoTrigger.R:116:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L116) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:116:70:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L116) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/ChronoTrigger.R:116:76:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L116) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/ChronoTrigger.R:121:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L121) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/ChronoTrigger.R:121:94:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L121) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/ChronoTrigger.R:121:100:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L121) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/ChronoTrigger.R:126:10:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L126) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/ChronoTrigger.R:127:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L127) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/ChronoTrigger.R:131:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L131) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/ChronoTrigger.R:132:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L132) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/ChronoTrigger.R:133:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L133) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/ChronoTrigger.R:134:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/ChronoTrigger.R:135:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L135) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/ChronoTrigger.R:136:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L136) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/ChronoTrigger.R:137:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L137) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/ChronoTrigger.R:139:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L139) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/ChronoTrigger.R:142:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/ChronoTrigger.R#L142) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/SuperMario.R:2:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:3:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{SuperMario} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:3:86:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{SuperMario} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/SuperMario.R:7:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{SuperMario} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:7:86:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{SuperMario} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/SuperMario.R:11:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:11:81:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/SuperMario.R:12:80:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/SuperMario.R:23:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L23) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{SuperMario} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:32:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L32) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:38:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L38) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:41:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L41) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/SuperMario.R:50:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L50) *style:* **Variable or function name should be snake_case.**
```r
SuperMario <- function(sound = 1, expr = NULL) {
^~~~~~~~~~
```
[R/SuperMario.R:51:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L51) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:75:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L75) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:77:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L77) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/SuperMario.R:78:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L78) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/SuperMario.R:80:9:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L80) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/SuperMario.R:82:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L82) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/SuperMario.R:84:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L84) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/SuperMario.R:85:38:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L85) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/SuperMario.R:86:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L86) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:86:11:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L86) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/SuperMario.R:86:75:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L86) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/SuperMario.R:92:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L92) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:96:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L96) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:96:70:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L96) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
                                                                    ~^~
```
[R/SuperMario.R:96:83:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L96) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/SuperMario.R:98:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L98) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:99:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L99) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/SuperMario.R:99:27:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L99) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```
[R/SuperMario.R:100:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L100) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:100:71:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                     ~^~
```
[R/SuperMario.R:100:79:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                             ~^~
```
[R/SuperMario.R:100:92:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L100) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/SMB/", sample(sounds, size=1), sep=""), package="geekr")
                                                                                ~^~
```
[R/SuperMario.R:102:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L102) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/SuperMario.R:104:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L104) *style:* **Lines should not be more than 80 characters.**
```r
    warning("SuperMario() could not play the sound due to the following error:\n", ex)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:109:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L109) *warning:* **no visible global function definition for ‘str_detect’**
```r
  str_detect(fname, "\\.wav$")
  ^~~~~~~~~~
```
[R/SuperMario.R:113:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L113) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:113:116:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L113) *style:* **Trailing whitespace is superfluous.**
```r
  system(paste("vlc -Idummy --no-loop --no-repeat --playlist-autostart --no-media-library --play-and-exit", fname),
                                                                                ^
```
[R/SuperMario.R:114:45:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L114) *style:* **Put spaces around all infix operators.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                           ~^~
```
[R/SuperMario.R:114:51:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L114) *style:* **Commas should always have a space after.**
```r
         ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                  ^
```
[R/SuperMario.R:119:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L119) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:119:70:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L119) *style:* **Put spaces around all infix operators.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                    ~^~
```
[R/SuperMario.R:119:76:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L119) *style:* **Commas should always have a space after.**
```r
  system(paste("paplay ", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                           ^
```
[R/SuperMario.R:124:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L124) *style:* **Lines should not be more than 80 characters.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/SuperMario.R:124:94:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L124) *style:* **Put spaces around all infix operators.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ~^~
```
[R/SuperMario.R:124:100:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L124) *style:* **Commas should always have a space after.**
```r
  system(paste("aplay --buffer-time=48000 -N -q", fname), ignore.stdout = TRUE, ignore.stderr=TRUE,wait = FALSE)
                                                                                ^
```
[R/SuperMario.R:129:10:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L129) *warning:* **no visible global function definition for ‘load.wave’**
```r
  sfx <- load.wave(fname)
         ^~~~~~~~~
```
[R/SuperMario.R:130:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L130) *warning:* **no visible global function definition for ‘play’, Did you mean 'lag'?**
```r
  play(sfx)
  ^~~~
```
[R/SuperMario.R:134:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(Sys.info()["sysname"] == "Linux") {
    ^
```
[R/SuperMario.R:135:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L135) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is_wav_fname(fname) && nchar(Sys.which("paplay")) >= 1) {
      ^
```
[R/SuperMario.R:136:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L136) *warning:* **no visible global function definition for ‘play_paplay’**
```r
      play_paplay(fname)
      ^~~~~~~~~~~
```
[R/SuperMario.R:137:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L137) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(is_wav_fname(fname) && nchar(Sys.which("aplay")) >= 1) {
             ^
```
[R/SuperMario.R:138:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L138) *warning:* **no visible global function definition for ‘play_aplay’**
```r
      play_aplay(fname)
      ^~~~~~~~~~
```
[R/SuperMario.R:139:14:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L139) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    } else if(nchar(Sys.which("vlc")) >= 1) {
             ^
```
[R/SuperMario.R:140:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L140) *warning:* **no visible global function definition for ‘play_vlc’**
```r
      play_vlc(fname)
      ^~~~~~~~
```
[R/SuperMario.R:142:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L142) *warning:* **no visible global function definition for ‘play_audio’**
```r
      play_audio(fname)
      ^~~~~~~~~~
```
[R/SuperMario.R:145:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/SuperMario.R#L145) *warning:* **no visible global function definition for ‘play_audio’**
```r
    play_audio(fname)
    ^~~~~~~~~~
```
[R/Warcraft.R:2:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:3:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L3) *style:* **Lines should not be more than 80 characters.**
```r
#' \code{Warcraft} plays a short sound which is useful if you want to get notified,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:3:84:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L3) *style:* **Trailing whitespace is superfluous.**
```r
#' \code{Warcraft} plays a short sound which is useful if you want to get notified,
                                                                                ^
```
[R/Warcraft.R:7:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L7) *style:* **Lines should not be more than 80 characters.**
```r
#' If \code{Warcraft} is not able to play the sound a warning is issued rather than
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:7:84:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L7) *style:* **Trailing whitespace is superfluous.**
```r
#' If \code{Warcraft} is not able to play the sound a warning is issued rather than
                                                                                ^
```
[R/Warcraft.R:11:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L11) *style:* **Lines should not be more than 80 characters.**
```r
#' @param sound character string or number specifying what sound to be played by
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:11:81:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L11) *style:* **Trailing whitespace is superfluous.**
```r
#' @param sound character string or number specifying what sound to be played by
                                                                                ^
```
[R/Warcraft.R:12:80:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L12) *style:* **Trailing whitespace is superfluous.**
```r
#'   either specifying one of the built in sounds, specifying the path to a wav
                                                                               ^
```
[R/Warcraft.R:19:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L19) *style:* **Lines should not be more than 80 characters.**
```r
#'   or url, a random sound will be played. Currently \code{Warcraft} can only handle
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:28:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L28) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:34:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L34) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:37:3:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L37) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/Warcraft.R:46:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L46) *style:* **Variable or function name should be snake_case.**
```r
Warcraft <- function(sound = 1, expr = NULL) {
^~~~~~~~
```
[R/Warcraft.R:47:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L47) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:61:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L61) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:63:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L63) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(sounds[sound]) || length(sounds[sound]) != 1) {
    ^
```
[R/Warcraft.R:64:7:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L64) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(is.character(sound)) {
      ^
```
[R/Warcraft.R:66:9:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L66) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if(file.exists(sound)) {
        ^
```
[R/Warcraft.R:68:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L68) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Warcraft.R:70:16:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L70) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(grepl("^https://", sound)) {
               ^
```
[R/Warcraft.R:71:38:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L71) *style:* **Put spaces around all infix operators.**
```r
        temp_file <- tempfile(pattern="")
                                    ~^~
```
[R/Warcraft.R:72:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L72) *style:* **Lines should not be more than 80 characters.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:72:11:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L72) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
          ^
```
[R/Warcraft.R:72:75:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L72) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if(download.file(sound, destfile = temp_file, quiet = TRUE) == 0) { # The file was successfully downloaded
                                                                          ^
```
[R/Warcraft.R:78:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L78) *style:* **Lines should not be more than 80 characters.**
```r
        warning(paste('"', sound, '" is not a valid sound nor path, playing a random sound instead.', sep = ""))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:82:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L82) *style:* **Lines should not be more than 80 characters.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/Warcraft.R:82:75:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L82) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
                                                                         ~^~
```
[R/Warcraft.R:82:88:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L82) *style:* **Put spaces around all infix operators.**
```r
    sound_path <- system.file(paste("sounds/Warcraft/", sounds[sound], sep=""), package="geekr")
                                                                                ~^~
```
[R/Warcraft.R:84:1:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L84) *style:* **Trailing whitespace is superfluous.**
```r

^~
```
[R/Warcraft.R:85:5:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L85) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.null(sound_path)) { # play a random sound
    ^
```
[R/Warcraft.R:85:27:](https://github.com/mike-decarlo/geekr/blob/e9a61d0a6057648bda615663dd25e394e05c3216/R/Warcraft.R#L85) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  if(is.null(sound_path)) { # play a random sound
                          ^
```


It would. If it breaks out of the for loop, `i+1` will be the number of headers processed. If it never breaks, `i+1` will be `count`, also the number of headers processed.


Если у тебя контейнер называется .blue, то потомки будут .blue__{{name}}, если нужно какой-то отличительный признак добавить, то .blue__{{name}}_{{modififcator}}

В css обращаешься к этим классам.

Если тебе нужно добавить стили к какоммо-то элементу, то не нужно каскадко к нему подбираться, задай ему класс отдельный, и отбращайся к нему.


Also worth noting that using a local variable would be preferred to a property of 'this' since JS has to go traversing the prototype chain or whatever ... you could squeeze microseconds out of this ...


Вот тут .bth не понятно откуда взялся и зачем нужен


@sjperkins Hi Simon. As you can see, I haven't quite started work on programming the GPU, just wanted to get some code running, and to try printing some stuff from the GPU operator. So far, it prints from the test script, but not from the operator itself.


clever


From just looking over the code I find no possible easy performance increases. I'm not sure if all the #ifdef I2_DEBUG precompiler statements are necessary, I think this would make sense in a Release build as well since we are only incrementing counters.


@mhhf Can we not add all of these SafeAdd_fail files


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


Если это у тебя кнопка, то есть специальный тег для кнопки, и называется он <button>
Верстка на одних div это конечно интересно, но так не делают. Если у тебя  интерактивный элемент, то он либо кнопка (button), либо ссылка (a)


Note: it will still verify hostname if `sslmode=verify-full`.

We might want to clarify that:
1) "certificate validation" is performed by a `SLL factory`
2) "hostname verification" is a separate thing

So "I don't care what the server is, let me just connect" mode is activated via `sslmode=require`. The only way `NonValidatingFactory` differs from `LibPQFactory` is `LibPQFactory` sends **client** certificate to the server (if exists), and it might cause server to REJECT the connection (server always validates client certificates no matter what is configured in pg_hba).

So I would suggest to rephrase as follows:
a) If just encryption is desired, one should use `sslmode=require`
b) If "invalid client certificate" causes failures to connect, then `sslcert=` should be added (empty value for `sslcert` caues `LibPQFactory` to ignore client certificate)
c) Alternative to `b`, one can use `NonValidatingFactory` (it never sends client certificates, and it always trusts all the server certificates)


Вот такие разделы можно сделать тегом <section>


Can this selector be made more specific?


[R/dist_all.R:29:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_all.R#L29) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon1 <- to_rad(lon1)
            ^~~~~~
```
[R/dist_all.R:30:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_all.R#L30) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon2 <- to_rad(lon2)
            ^~~~~~
```
[R/dist_all.R:31:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_all.R#L31) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat1 <- to_rad(lat1)
            ^~~~~~
```
[R/dist_all.R:32:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_all.R#L32) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat2 <- to_rad(lat2)
            ^~~~~~
```
[R/dist_haversine.R:34:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_haversine.R#L34) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon1 <- to_rad(lon1)
            ^~~~~~
```
[R/dist_haversine.R:35:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_haversine.R#L35) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon2 <- to_rad(lon2)
            ^~~~~~
```
[R/dist_haversine.R:36:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_haversine.R#L36) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat1 <- to_rad(lat1)
            ^~~~~~
```
[R/dist_haversine.R:37:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_haversine.R#L37) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat2 <- to_rad(lat2)
            ^~~~~~
```
[R/dist_slc.R:34:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_slc.R#L34) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon1 <- to_rad(lon1)
            ^~~~~~
```
[R/dist_slc.R:35:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_slc.R#L35) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon2 <- to_rad(lon2)
            ^~~~~~
```
[R/dist_slc.R:36:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_slc.R#L36) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat1 <- to_rad(lat1)
            ^~~~~~
```
[R/dist_slc.R:37:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_slc.R#L37) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat2 <- to_rad(lat2)
            ^~~~~~
```
[R/dist_vincenty.R:35:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_vincenty.R#L35) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon1 <- to_rad(lon1)
            ^~~~~~
```
[R/dist_vincenty.R:36:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_vincenty.R#L36) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lon2 <- to_rad(lon2)
            ^~~~~~
```
[R/dist_vincenty.R:37:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_vincenty.R#L37) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat1 <- to_rad(lat1)
            ^~~~~~
```
[R/dist_vincenty.R:38:13:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/dist_vincenty.R#L38) *warning:* **no visible global function definition for ‘to_rad’**
```r
    lat2 <- to_rad(lat2)
            ^~~~~~
```
[R/HERE_drivetime.R:36:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L36) *style:* **Variable or function name should be snake_case.**
```r
HERE_drivetime <- function(orgn_lat, orgn_lon, dest_lat, dest_lon
^~~~~~~~~~~~~~
```
[R/HERE_drivetime.R:49:17:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L49) *warning:* **no visible global function definition for ‘to_deg’**
```r
    orgn_lat <- to_deg(orgn_lat)
                ^~~~~~
```
[R/HERE_drivetime.R:50:17:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L50) *warning:* **no visible global function definition for ‘to_deg’**
```r
    orgn_lon <- to_deg(orgn_lon)
                ^~~~~~
```
[R/HERE_drivetime.R:51:17:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L51) *warning:* **no visible global function definition for ‘to_deg’**
```r
    dest_lat <- to_deg(dest_lat)
                ^~~~~~
```
[R/HERE_drivetime.R:52:17:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L52) *warning:* **no visible global function definition for ‘to_deg’**
```r
    dest_lon <- to_deg(dest_lon)
                ^~~~~~
```
[R/HERE_drivetime.R:54:28:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L54) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  id <- paste0("&app_id=", curlEscape(app_id))
                           ^~~~~~~~~~
```
[R/HERE_drivetime.R:55:32:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L55) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  code <- paste0("&app_code=", curlEscape(app_code))
                               ^~~~~~~~~~
```
[R/HERE_drivetime.R:56:34:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L56) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  wypnt0 <- paste0("waypoint0=", curlEscape(paste0(orgn_lat, ",", orgn_lon)))
                                 ^~~~~~~~~~
```
[R/HERE_drivetime.R:57:35:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L57) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  wypnt1 <- paste0("&waypoint1=", curlEscape(paste0(dest_lat, ",", dest_lon)))
                                  ^~~~~~~~~~
```
[R/HERE_drivetime.R:63:38:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L63) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  departure <- paste0("&departure=", curlEscape(departure))
                                     ^~~~~~~~~~
```
[R/HERE_drivetime.R:68:11:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_drivetime.R#L68) *warning:* **no visible global function definition for ‘fromJSON’**
```r
  json <- fromJSON(request_url, flatten = FALSE)
          ^~~~~~~~
```
[R/HERE_geocode.R:22:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L22) *style:* **Variable or function name should be snake_case.**
```r
HERE_geocode <- function(
^~~~~~~~~~~~
```
[R/HERE_geocode.R:38:33:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L38) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  addr <- paste0("searchtext=", curlEscape(address))
                                ^~~~~~~~~~
```
[R/HERE_geocode.R:39:28:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L39) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  id <- paste0("&app_id=", curlEscape(app_id))
                           ^~~~~~~~~~
```
[R/HERE_geocode.R:40:32:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L40) *warning:* **no visible global function definition for ‘curlEscape’**
```r
  code <- paste0("&app_code=", curlEscape(app_code))
                               ^~~~~~~~~~
```
[R/HERE_geocode.R:46:11:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L46) *warning:* **no visible global function definition for ‘fromJSON’**
```r
  json <- fromJSON(request_url, flatten = FALSE)
          ^~~~~~~~
```
[R/HERE_geocode.R:49:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L49) *style:* **Lines should not be more than 80 characters.**
```r
    json$Response$View$Result[[1]]$Location$NavigationPosition[[1]]$Latitude[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/HERE_geocode.R:53:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L53) *style:* **Lines should not be more than 80 characters.**
```r
      , json$Response$View$Result[[1]]$Location$NavigationPosition[[1]]$Latitude[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/HERE_geocode.R:59:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L59) *style:* **Lines should not be more than 80 characters.**
```r
    json$Response$View$Result[[1]]$Location$NavigationPosition[[1]]$Longitude[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/HERE_geocode.R:63:1:](https://github.com/mike-decarlo/gcd/blob/6bb4d59871b5d1fefb6dca4794179a7ff2a27066/R/HERE_geocode.R#L63) *style:* **Lines should not be more than 80 characters.**
```r
      , json$Response$View$Result[[1]]$Location$NavigationPosition[[1]]$Longitude[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


Great finding!

Congrats!


Created multidev environment [riveralive-cms#pr-35](https://dashboard.pantheon.io/sites/ced938b0-bbbf-4ed2-ba1c-98b66d8ba2be#pr-35).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-35-riveralive-cms.pantheonsite.io/)


The chance of interfering with another transferal job is higher with doubleHash than with locHash.


Если у тебя есть стили на этом теге, то не надо вот так писать:

.team h2 {
  //стили
}

Добавь к нему класс .team__title и уже на него навешивай стили.


good ideas here, but idk if i will need it outside later or not.  i may....


Added/changes test case for the above.


3.5.0 was supposed to be the release, but we forgot to update it. I also had to apply a bugfix, so that pushes us into patch territory, ending in 3.5.1.


Looking good ! How is the speed of the table, the same (when the gui requests/fills the initial table, the search is the same)
I vote for this solution!


__apiary.manifest contains reference to non-existent file in repository.__ (Autogenerated message from apiary.io)
+ apiary.apib


Looks legit to me (save for a couple code readability improvements). That issue with stop being called might have something to do with setTimeout and setInterval limits. Seems to me there were race condition and deadlock like things that start happening around the 20ms range when using those. You could go super tedious and adjust the settings manually to prove that to be the case ...


It seems as if your problem is with the installation, which is separate from the software.
My first suggestion would be to first copy all the files, uncompressed, into a new folder on your C drive.
When installing, hopefully the runtime will install automatically. If it does not, make  sure you have the runtime for 2018a installed:https://www.mathworks.com/products/compiler/matlab-runtime.html
You don't need the 2018a version of Matlab, just the runtime.
If that doesn't work, I refer you to thiss: https://www.mathworks.com/matlabcentral/answers/92499-why-do-i-receive-a-message-stating-error-finding-installer-class-when-trying-to-install-matlab-on


It also seems that Hive call the initialize(ObjectInspector[] argOIs) when calling the method, because I tried to override other methods but I still get IllegalArgumentException


actually i doubt it will need to be.  but this.then is a necessity to be outside of this loop.

this.then is just Date.now() upon initialization of this entire file set up in the constructor.


Puzzle `1545-87cf0a16` discovered in   [`src/main/java/com/zerocracy/claims/BatchClaims.java`](  https://github.com/zerocracy/farm/blob/master/src/main/java/com/zerocracy/claims/BatchClaims.java)   and submitted as #1621. Please, remember that the puzzle was not   necessarily added in this particular commit. Maybe it was added earlier, but   we discovered it only now.


Ой, тут тоже беда с Классом, не может быть один модификатор, сначала ты определяешь как блок или как элемент, а уже потом добавляешь модификатор, если он тебе нужен.
В твоем случае вообще не может так называтся этот элемент.
Это у тебя определенно блок, потому что внитри него есть несколько элементов, наилучшее название для него .team-item


@cweithoeft I applied this as a quick fix to get AttackPattern to be STIX compliant. However I think your original code was better from a usability point of view since it prevented NullPointerException when somebody used this without null-checking. I think we should look into configuring Gson to stop serializing empty Lists and then go back to initialized Lists. Should be possible via GsonBuilder#addSerializationExclusionStrategy(ExclusionStrategy).
What do you think?


The problematic function.


Puzzle `1545-78dce506` discovered in   [`src/test/java/com/zerocracy/claims/BatchClaimsTest.java`](  https://github.com/zerocracy/farm/blob/master/src/test/java/com/zerocracy/claims/BatchClaimsTest.java)   and submitted as #1622. Please, remember that the puzzle was not   necessarily added in this particular commit. Maybe it was added earlier, but   we discovered it only now.


Failed to remove a directory: train; Directory not empty


А, не, это у тебя контейнер, тогда вот так будет team-item-container


It says in the comment "gpio14" but why does the code say 12?


Created multidev environment [thf-cms#pr-1](https://dashboard.pantheon.io/sites/25812cb4-ea8c-436b-ac17-09b0c2b7598b#pr-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-1-thf-cms.pantheonsite.io/)


Локальную переменную стоит убрать, и сразу возвращать результат


И так как это список из элементов, то лучше использовать тег ul


Created multidev environment [riveralive-cms#ci-237](https://dashboard.pantheon.io/sites/ced938b0-bbbf-4ed2-ba1c-98b66d8ba2be#ci-237).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-237-riveralive-cms.pantheonsite.io/)


python srez_main.py --run train
.... Failed to remove a directory: train; Directory not empty ?


@ViugiNick It's only for Ruby>=2.5, right?


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


А вот тут юбудет тег li тогда, и название team-list-item


Если на картинке есть стили, то .team-list-item__img


Is it possible there is an issue with the cloud CDN that China uses? I don't know who manages the CRAN cloud but that's the person to talk to.


@marcinsdance Easy enough -- done! https://github.com/astorm/magento2-tutorial-objectmanager-arguments/releases/tag/0.0.2


that last chunck of code is here by mistake. It has been reverted in the previous commit.


Checkmarx scan results 20 Aug 2018 17:51 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a23cad998261c81bf63aa21ac681a2e8b5303e7c	@web-flow


Вот, а тут идеально подойдут модификаторы! определяем, что это ссылка : .team-list-item__link , но нам нужно указать какая это конкретно ссылка, для этого добавляем второй класс  .team-list-item__link_instagram


Candidate for upstreaming?


Another candidate for upstreaming?


Дальше по примеру переделать, использовать не только  div) Не забывай про кнопки!


The problematic function. If we add line 79, it returns normal so that it seems that the `dR` is optimized away.


Commenting here on this code


BRANDON HDEZ CORONA


You cannot just extend it directly, as you will lose the whole module chain.


BRANDON


@damianoneill, I can't debug your Dockerfile given that limited context.


@thomashaselwanter I'm a little uncertain about this. I don't like to get empty lists in JSON strings. You can argue that an empty list is information about a field being empty. Is that more information that being NULL?
My other concern is that I hope we can reduce the use of GSON specific methods. I know that our library is using GSON, based on annotation, so we have picked side.
If we go for addSeriali.... we must provide good documentation for the lib. But let's give it a try.

BTW I like the additions to test and the utils you have provided. Much cleaner code than my first attempt.


fix #16


Breaks the build
```
Pages/Session.cshtml.cs(45,62): error CS1503: Argument 1: cannot convert from 'System.DateTimeOffset' to 'System.DateTime'
```


teste


Hm, I needed to run `sudo glib-compile-schemas /usr/local/share/glib-2.0/schemas/` to get it to work after installing `volctl` with `setup.py` on my Ubuntu. Also, the whole `glib-compile-schemas` step is still missing on the homepage https://buzz.github.io/volctl/.


Oh, wow! Thank you! ([ref discussion](https://groups.google.com/forum/#!topic/tiddlywikidev/P_oTII4Td7w))


My vote would go to the 3rd option as that uses a existing style and it should be very clear what the ZWAVE ID is.
As to the speed of the table: I do not notice any difference and won't expect it as it is client side rendering and have nice fast Laptop. Also tried it on an old IPAD 2 and didn't notice any difference.


Was repräsentiert das `distance`-Field?


Already done. :P


@nobu,

Thanks for reviewing, and, yes, you're correct.

4. What I meant was that, if you look at the standard [path on Appveyor](https://ci.appveyor.com/project/ruby/ruby/build/9390/job/tilcvn7xgpbfr3hg?fullLog=true#L101), it contains a lot of folders, some of which, like `C:\Program Files\Git\usr\bin`, contain dll's that may  conflict with MSYS2/MinGW dll's.

5. There are some RubyGems tests that need `git`, and they may have issues with the space in `Program Files`.  For a long time, I've added a 'no space' symlink to Git. RG will use the value of `ENV['GIT']` for the git .exe binary, so I set that to the `git.exe` in the symlink folder.

Re 4, just this weekend I was adding OpenSSL 1.1.1 to trunk, and Appveyor has the two standard OpenSSL dll's in `C:\Windows\System32` (my Windows 10 system does not).  Needless to say, it caused problems when running `make install`.  Solution was to rename the files...


would be good to name the 'tasks' key in the tasks reducer 'data' instead. This way you don't have to do tasks.tasks


Why???


Sorry brad should have included more.

I'm using dep to manage my dependencies and CMing the vendor folder.  I develop on a mac and build binaries for docker images based off alpine.

Since I raised the issue I've noted that if I .gitignore the vendor/golang.org/x/sys/unix and dep ensure prior to doing a build, it will build correctly on the ci server with the alpine based dockerfile.

Apologies I am not a go expert, is there any chance that go get / dep is downloading or generating some OS specific content with this package?


End of scripts. Down there just cleaning stuff.


Good to write these as implicit returns.

export const deleteTask = task_id => ({
    type: 'DELETE_TASK',
    payload: task_id,
});


I guess. Though I'd assume whoever made it based it on Ralfs work.


This might need to be

```cpp
std::cout << "I am here" << std::endl;
```


Thanks, checking it out now.


Update link to base class in docs


![](https://github.trello.services/images/mini-trello-icon.png) [NC-88 Add defaultGroupId to user payload (frontend and backend)](https://trello.com/c/m8qAcen5/88-nc-88-add-defaultgroupid-to-user-payload-frontend-and-backend)


DELETE_TASK should be a magic string / string constant defined at the top of the actions file

take a look at `insights:app/containers/App/actions.js:1-21`


That is probably true. But I think you just assumed some things - what if there is interruption between process runs (say we call face:analyze and it crashes). Or if we call it only during the night time and stop in the morning? Or we analyzed 10.000 images and after 2 months, we get additional 1000 images from vacation. What are process start and end times in these cases?:)

If we do it all at once, if process is unininterrupted, if there are no new images - what you say is true.

This approach is more of "do tiny bits for 10-15 minutes and bail out and continue later when called again". If you try doing it like that, you will see it is not as easy as using (end_time-start_time)/image_processed 😄. Actually, you **can** do it like that, by you will have to save each:
* process start time,
* process end time,
* images processed in that run

somewhere in DB and you can calculate average velocity from this. But, I didn't want to create another table for estimating this, it seemed easier to just put it as column in Image and use that to get rough estimate:)


![](https://github.trello.services/images/mini-trello-icon.png) [Add middleware for getting user info from token (auth header) and set that info to res.locals.user](https://trello.com/c/vzCK4Dby/98-add-middleware-for-getting-user-info-from-token-auth-header-and-set-that-info-to-reslocaluser)


string constants should be used here too instead of writing a raw string.

take a look at insights:app/containers/App/actions.js:1-21

you can import this value


we should use `datetime` package from `go-common` where possible for both the parsing and the conversion from unix to int64.


all our IDs need to be hashed using `hash.Value` from go-common.


No entiendo que franja, acabo de ver el video nuevamente y no veo esa franja que mencionas.


`state.merge`


We don't have much of a choice when it comes to the JSON we need to produce. `  "kill_chain_phases": [],` is invalid STIX according to the validator. When the list is empty the JSON should not have the kill_chain_phases attribute at all.
I just don't like my solution with the uninitialized `killChainPhase` field for two reasons:

1. More chances for a NullPointerExceptions when a user tries to use it without checking
2. If a user of the library sets it to an empty list we will again produce an invalid STIX file

Number 2 alone is reason enough probably to try this since we library should never produce invalid STIX.
EclusionStrategy might not work since it doesn't have access to the instance of the field only the field attributes. However a custom JsonSerializer for all Java Collection would probably do the trick.


just use `hash.Values`


Было дело. Так и не фиксанули, само исчезло.


You should not try to reduce the likelihood from 1/65k to 1/65million (I made up these numbers), you should try to reduce it to 1/10^74, which is the likelihood of a (true) hash conflict.
And you reach that level by making sure there is nothing there before you write. At that point `locHash` is probably redundant`.


Pues hay que verificar bien porque en este commit solo aparece la clase que borraste (Java) pero no veo la clase nueva (Kotlin). Posiblemente no la agregaste al commit


I'm going to use the existing logic and will add a TODO.


Created multidev environment [lawhelp#pr-124](https://dashboard.pantheon.io/sites/9ec12d94-c804-44b7-968e-166a0b49fdc0#pr-124).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-124-lawhelp.pantheonsite.io/)


For what? This commit is about something not even released yet.


Nerves Hardware Test
Status: Results Received
%{"failures" => 0, "skipped" => 0, "total" => 3}
https://nerves-test-server.herokuapp.com/nerves-project/nerves_system_rpi3/61a7e62c-be1d-4e67-91fa-17d43a3da03c


Sorry, I'm just pushing back a bit on whether `mxe` as an additional CONFIG option is justified. My sense is that it's too general a notion and would be used here essentially as shorthand for a set of existing, more primitive, options such as `mingw`, `qscintilla2`, `cross_compile`, and `static`.

Since my last comment, I have learned from http://pyqt.sourceforge.net/Docs/QScintilla2/:

> To configure qmake to find your QScintilla installation, add the following line to your application's .pro file:
>     CONFIG += qscintilla2
>

Presumably then, the qscintilla installation installs a module (more or less a .prf feature file?) which defines the CONFIG option `qscintilla2`. When qmake runs with this option set, the module will configure the project to use qscintilla. If the option is not set, then the project file (.pro) or qmake invocation can specify explicitly the libraries to link.

Now, it seems to me that this should be decided when building openscad, which will either use `CONFIG += qscintilla2` by default or not. The qmake invocation in openscad.mk can either override the default or not. The openscad installation should provide a way to pass transitive dependencies on to other projects linking to openscad. It could install .prl file or its own module for this purpose. (Is this what openscad.prf is for?) If it does this by transitively including the qscintilla .prf file, then it should also set (or unset) the CONFIG option qsintilla2 as appropriate, rather than leaving this to the application.

When applications depending on openscad are built, neither the application project file nor the qmake invocation should have to specify how to link to qscintilla, since this decision was already made in the openscad build.

My apologies for the long-windedness and risking missing your point.


수정된 파일 push test


Nerves Hardware Test
Status: Results Received
%{"failures" => 0, "skipped" => 0, "total" => 3}
https://nerves-test-server.herokuapp.com/nerves-project/nerves_system_bbb/244b0402-11ef-4288-a144-ee9c050c429b


Do the same as `case 3`. We don't have a 'reference' to a dictionary at this point.


# Specs execution successful
:scroll: [View logs](http://bamboo.comnull)
### Plans updated

Bamboo specs have been successfully imported and the following plans have been updated.

Project | Plan | Build
---|---|---
[Test](http://bamboo.com/browse/TEST) | [Comment test](http://bamboo.com/browse/TEST-CTEST)  | :white_check_mark: [\#1](http://bamboo.com/browse/TEST-CTEST-1)
[New project](http://bamboo.com/browse/NEW) | [Disabled plan](http://bamboo.com/browse/NEW-DISABLED) **disabled** | :white_check_mark: [\#1](http://bamboo.com/browse/NEW-DISABLED-1)

### Deployments updated

Bamboo specs have been successfully imported and the following deployment projects have been updated.

|Deployment project |
----|
[Deployment test](http://bamboo.com/deploy/viewDeploymentProjectVersions.action?id=2162689)|


Oh, I add it accidentally, sorry for that...


This is not correct. The `extract_literal_value()` above returns a valid result, that means that this function should return `true` as well.


COMMIT GUIDE
1) Add update profile view model in View Models folder (see code above).
2) Add update profile method to Home Controller (see code above).
3) Add View for update profile (see code above).


Ecks dee


COMMIT GUIDE

1) Add create user view model in View Models folder (see code above).
2) Add create user method to Home Controller (see code above).
3) Add View for create user (see code above).


Thanks :+1:


@cweithoeft I pushed `EmptyCollectionNonSerializer` which should fix this globally.


1. npm install (tao ra node_modules)
2. npm install sequelize pg@6 pg-hstore sequelize-cli
3. nodemon
bat trang locall len . luc nay van chua code gi het. chi la set up thoi


You mean like `18 = Resources\Themes\Lautus` ?


I don't know what dep does, sorry. Or you're being bitten by docker build caching. Let's bring the discussion to golang-nuts.


this is exersise 13, not 14


Mostly this just reads faster/better to me.  What is the action happening in this doseq? We are invoking setOption.  The filter/predicate is a detail of the iteration, not the body.

Since this is a doseq it isn't actually functional, but in `for` its very helpful
```clj
(for [a (range 10)]
  (when (even? a)
    a))
=> (0 nil 2 nil 4 nil 6 nil 8 nil)
(for [a (range 10)
      :when (even? a)]
  a)
=> (0 2 4 6 8)
```


No PR @lazyvar?????


@elenatorro can we add a piece in the description about how the points are placed inside the grid cell? they aren't placed in the center of the grid cell like with torque.


ok


This is done in previous commit, see: https://github.com/vcmi/vcmi/pull/474/commits/158b3c0b80b94f1b5c0a4ea7baf5e0a77795d981#diff-17182f23fecdfcf334d0f03f9bd612c7R428

Unless you mean that I should remove all keys (ultimate goals) that contain this goal in map value, which is not useful IMO


@AntoineCoconut


returning id. If not len(self.env.cr.fetchall()) -> invalidate


and a small edit on wording:

`resolution of the property-aggregation functions, only applicable to points. Default resolution is 1. Custom values must be greater than 0 and lower than 256. A resolution of 4 means points are aggregated to grid cells of 4x4 pixels`


update from upstream


It uses amazon's cloudfront, which currently doesn't have any endpoints in China: https://www.amazonaws.cn/en/content-delivery/. I'll ask around internally.


kk


I suggest aligning them to be more readable. I can't keep idents here for some reason.
An extra Tab and space will do the job.