Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update breached_site.R #30

Closed
wants to merge 2 commits into from
Closed

Update breached_site.R #30

wants to merge 2 commits into from

Conversation

@maelle
Copy link
Member

@maelle maelle commented Aug 23, 2018

No description provided.

maelle added 2 commits Aug 23, 2018
@lintr-bot

This comment has been minimized.

Copy link

@lintr-bot lintr-bot commented on ec2e3e4 Aug 23, 2018

R/breached_site.R:14:1: style: lines should not be more than 80 characters.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

R/breached_site.R:14:5: style: Place a space before left parenthesis, except in a function call.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
    ^

R/breached_site.R:14:19: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                 ~^~

R/breached_site.R:14:32: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                              ~^

R/breached_site.R:14:36: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                  ~^~

R/breached_site.R:14:52: style: Commas should always have a space after.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                   ^

R/breached_site.R:14:65: style: Opening curly braces should never go on their own line and should always be followed by a new line.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                ^

R/breached_site.R:14:97: style: Closing curly-braces should always be on their own line, unless it's followed by an else.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                                                ^

R/utils.R:11:13: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
            ^~~~~~~~~~~~~

R/utils.R:24:15: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
              ^~~~~~~~~~~~~

This comment has been minimized.

Copy link
Member Author

@maelle maelle replied Aug 23, 2018

thank you @lintr-bot!

@maelle maelle closed this Aug 23, 2018
@maelle maelle deleted the lintr-bot branch Aug 23, 2018
@lintr-bot
Copy link

@lintr-bot lintr-bot commented Aug 23, 2018

R/breached_site.R:14:1: style: lines should not be more than 80 characters.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

R/breached_site.R:14:5: style: Place a space before left parenthesis, except in a function call.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
    ^

R/breached_site.R:14:19: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                 ~^~

R/breached_site.R:14:32: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                              ~^

R/breached_site.R:14:36: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                  ~^~

R/breached_site.R:14:52: style: Commas should always have a space after.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                   ^

R/breached_site.R:14:65: style: Opening curly braces should never go on their own line and should always be followed by a new line.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                ^

R/breached_site.R:14:97: style: Closing curly-braces should always be on their own line, unless it's followed by an else.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                                                ^

R/utils.R:11:13: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
            ^~~~~~~~~~~~~

R/utils.R:24:15: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
              ^~~~~~~~~~~~~
3 similar comments
@lintr-bot
Copy link

@lintr-bot lintr-bot commented Aug 23, 2018

R/breached_site.R:14:1: style: lines should not be more than 80 characters.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

R/breached_site.R:14:5: style: Place a space before left parenthesis, except in a function call.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
    ^

R/breached_site.R:14:19: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                 ~^~

R/breached_site.R:14:32: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                              ~^

R/breached_site.R:14:36: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                  ~^~

R/breached_site.R:14:52: style: Commas should always have a space after.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                   ^

R/breached_site.R:14:65: style: Opening curly braces should never go on their own line and should always be followed by a new line.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                ^

R/breached_site.R:14:97: style: Closing curly-braces should always be on their own line, unless it's followed by an else.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                                                ^

R/utils.R:11:13: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
            ^~~~~~~~~~~~~

R/utils.R:24:15: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
              ^~~~~~~~~~~~~
@lintr-bot
Copy link

@lintr-bot lintr-bot commented Aug 23, 2018

R/breached_site.R:14:1: style: lines should not be more than 80 characters.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

R/breached_site.R:14:5: style: Place a space before left parenthesis, except in a function call.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
    ^

R/breached_site.R:14:19: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                 ~^~

R/breached_site.R:14:32: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                              ~^

R/breached_site.R:14:36: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                  ~^~

R/breached_site.R:14:52: style: Commas should always have a space after.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                   ^

R/breached_site.R:14:65: style: Opening curly braces should never go on their own line and should always be followed by a new line.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                ^

R/breached_site.R:14:97: style: Closing curly-braces should always be on their own line, unless it's followed by an else.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                                                ^

R/utils.R:11:13: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
            ^~~~~~~~~~~~~

R/utils.R:24:15: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
              ^~~~~~~~~~~~~
@lintr-bot
Copy link

@lintr-bot lintr-bot commented Aug 23, 2018

R/breached_site.R:14:1: style: lines should not be more than 80 characters.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

R/breached_site.R:14:5: style: Place a space before left parenthesis, except in a function call.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
    ^

R/breached_site.R:14:19: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                 ~^~

R/breached_site.R:14:32: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                              ~^

R/breached_site.R:14:36: style: Put spaces around all infix operators.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                  ~^~

R/breached_site.R:14:52: style: Commas should always have a space after.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                   ^

R/breached_site.R:14:65: style: Opening curly braces should never go on their own line and should always be followed by a new line.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                ^

R/breached_site.R:14:97: style: Closing curly-braces should always be on their own line, unless it's followed by an else.

if(is.null(name)|length(name)!= 1|!inherits(name,"character")){stop("Problematic breach name")}
                                                                                                ^

R/utils.R:11:13: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
            ^~~~~~~~~~~~~

R/utils.R:24:15: warning: no visible global function definition for ‘get_and_retry’

output <- get_and_retry(URLS, headers, verbose)
              ^~~~~~~~~~~~~
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.