CreateCommSpeciesTable.R function

To see the styling errors by line do (in Rstudio): fName <- "./WKRDB-EST2/subgroup4/personal/Kasia/CreateCommSpeciesTable.R" lintr::lint(fName,linters=lintr::with_defaults(object_name_linter=lintr::object_name_linter(styles = "camelCase")))

Unique styling issues identified:

[ ] Variable and function name style should be camelCase.
[x] Put spaces around all infix operators.
[x] Lines should not be more than 80 characters.
[ ] Only use double-quotes.
[x] Commas should always have a space after.
[ ] functions should have cyclomatic complexity of less than 15
[x] Remove spaces before the left parenthesis in a function call.
[x] There should be a space between right parenthesis and an opening curly brace.
[ ] no visible binding for global variable 'SAsequenceNumber'
[x] Avoid 1:nrow(...) expressions, use seq_len.
[x] Opening curly braces should never go on their own line and should always be followed by a new line.
[x] Trailing whitespace is superfluous.
[ ] no visible binding for global variable 'SAid'
[ ] no visible binding for global variable 'SAparentSequenceNumbe'
[ ] no visible binding for global variable 'SAparentSequenceNumber'
[x] Place a space before left parenthesis, except in a function call.
[x] Do not place spaces around code in parentheses or square brackets.
[ ] no visible binding for global variable 'SAunitType'
[x] local variable 'cos' assigned but may not be used
[x] Avoid 1:length(...) expressions, use seq_len.
[x] Commas should never have a space before.
[x] local variable 'CommercialSampleNew' assigned but may not be used

You can use also auto styler to fix some. Unique lint issues after running styler::style_file(fName):

[ ] Variable and function name style should be camelCase.
[x] Lines should not be more than 80 characters.
[ ] functions should have cyclomatic complexity of less than 15
[ ] no visible binding for global variable 'SAsequenceNumber'
[x] Avoid 1:nrow(...) expressions, use seq_len.
[x] Opening curly braces should never go on their own line and should always be followed by a new line.
[ ] no visible binding for global variable 'SAid'
[ ] no visible binding for global variable 'SAparentSequenceNumbe'
[ ] no visible binding for global variable 'SAparentSequenceNumber'
[ ] no visible binding for global variable 'SAunitType'
[x] local variable 'cos' assigned but may not be used
[x] Avoid 1:length(...) expressions, use seq_len.
[x] local variable 'CommercialSampleNew' assigned but may not be used

ices-eg / WK_RDBES

CreateCommSpeciesTable.R function #63