[inst/examples/example_experiment.R:91:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_experiment.R#L91) *style:* **Commented code should be removed.**
```r
  # library(raster)
    ^~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:92:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_experiment.R#L92) *style:* **Commented code should be removed.**
```r
  # beginCluster(20) # if you have multiple clusters available, use them here to save time
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:96:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_experiment.R#L96) *style:* **Commented code should be removed.**
```r
  # endCluster() # end the clusters
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_experiment.R:152:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_experiment.R#L152) *style:* **Commented code should be removed.**
```r
  # fires <- lapply(sims, function(x) x$landscape$fires) %>% stack
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:48:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L48) *style:* **Commented code should be removed.**
```r
  # cl <- makeCluster(detectCores() - 1) # not implemented yet in DEoptim
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:83:4:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L83) *style:* **Commented code should be removed.**
```r
  #stopCluster(cl) # not yet implemented, waiting for DEoptim
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:106:34:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L106) *style:* **Variable or function name should be lowerCamelCase.**
```r
    params(sim1)$caribouMovement$N <- pars[2]
                                 ^
```
[inst/examples/example_POM.R:113:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L113) *style:* **Variable or function name should be lowerCamelCase.**
```r
    nPattern_Out <- caribouFn(out$caribou)
    ^~~~~~~~~~~~
```
[inst/examples/example_POM.R:119:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L119) *style:* **Commented code should be removed.**
```r
    # cat(minimizeFn)
      ^~~~~~~~~~~~~~~
```
[inst/examples/example_POM.R:120:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L120) *style:* **Commented code should be removed.**
```r
    # cat(" ")
      ^~~~~~~~
```
[inst/examples/example_POM.R:121:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L121) *style:* **Commented code should be removed.**
```r
    # cat(pars)
      ^~~~~~~~~
```
[inst/examples/example_POM.R:122:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/examples/example_POM.R#L122) *style:* **Commented code should be removed.**
```r
    # cat("\n")
      ^~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:54:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L54) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.caribouMovement <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:78:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L78) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$moveInterval, "caribouMovement", "move")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:82:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L82) *style:* **Lines should not be more than 100 characters.**
```r
      Plot(sim$caribou, addTo = paste("sim", SpaDES.core::P(sim)$stackName, "habitatQuality", sep = "$"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:86:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L86) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$.plotInterval, "caribouMovement", "plot", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:90:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L90) *style:* **Lines should not be more than 100 characters.**
```r
      Plot(sim$caribou, addTo = paste("sim", SpaDES.core::P(sim)$stackName, "habitatQuality", sep = "$"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:95:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L95) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$.plotInterval, "caribouMovement", "plot", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:102:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L102) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$.saveInterval, "caribouMovement", "save", .last() + 1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:114:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L114) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:121:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L121) *style:* **Variable or function name should be lowerCamelCase.**
```r
  N <- SpaDES.core::P(sim)$N
  ^
```
[inst/sampleModules/caribouMovement/caribouMovement.R:122:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L122) *style:* **Variable or function name should be lowerCamelCase.**
```r
  IDs <- as.character(1:N)
  ^~~
```
[inst/sampleModules/caribouMovement/caribouMovement.R:138:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/caribouMovement/caribouMovement.R#L138) *style:* **Variable or function name should be lowerCamelCase.**
```r
Move <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:20:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L20) *style:* **Lines should not be more than 100 characters.**
```r
    person(c("Alex", "M"), "Chubaty", email = "alexander.chubaty@canada.ca", role = c("aut", "cre")),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:21:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L21) *style:* **Lines should not be more than 100 characters.**
```r
    person(c("Eliot", "J", "B"), "McIntire", email = "eliot.mcintire@canada.ca", role = c("aut", "cre")),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:35:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L35) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("persistprob", "numeric", 0.00, 0, 1, "probability of fire persisting in a pixel"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:37:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L37) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("spreadprob", "numeric", 0.225, 0.05, 0.5, "probability of fire spreading into a pixel"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:38:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L38) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("startTime", "numeric", start(sim) + 1, 0, end(sim), "time of initial fire ignition"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:39:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L39) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", start(sim), start(sim), end(sim) + 1, "time to schedule first plot event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:41:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L41) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "time to schedule first save event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:42:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L42) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInterval", "numeric", NA_real_, NA, NA, "time interval between save events")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:45:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L45) *style:* **Lines should not be more than 100 characters.**
```r
    expectsInput(objectName = SpaDES.core::P(sim, "fireSpread")$stackName, objectClass = "RasterStack",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:51:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L51) *style:* **Lines should not be more than 100 characters.**
```r
    createsOutput(objectName = SpaDES.core::P(sim, "fireSpread")$stackName, objectClass = "RasterStack",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:59:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L59) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.fireSpread <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:80:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L80) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, SpaDES.core::P(sim)$.plotInitialTime, "fireSpread", "plot.init", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:87:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L87) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim), "fireSpread", "stats") # do stats immediately following burn
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:88:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L88) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$returnInterval, "fireSpread", "burn")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:108:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L108) *style:* **Lines should not be more than 100 characters.**
```r
      stackName <- SpaDES.core::P(sim)$stackName # Plot doesn't like long names -- create local variable
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:143:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L143) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:147:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L147) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- raster(extent(landscapes), ncol = ncol(landscapes),
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:151:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L151) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- setValues(Fires, 0)
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:154:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L154) *style:* **Variable or function name should be lowerCamelCase.**
```r
  landscapes$Fires <- Fires
             ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:160:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L160) *style:* **Variable or function name should be lowerCamelCase.**
```r
Burn <- function(sim) {
^~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:163:3:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L163) *style:* **Variable or function name should be lowerCamelCase.**
```r
  Fires <- spread(landscapes[[1]],
  ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:175:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L175) *style:* **Variable or function name should be lowerCamelCase.**
```r
  landscapes$Fires <- Fires
             ^~~~~
```
[inst/sampleModules/fireSpread/fireSpread.R:182:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/fireSpread/fireSpread.R#L182) *style:* **Variable or function name should be lowerCamelCase.**
```r
Stats <- function(sim) {
^~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:1:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L1) *style:* **Variable or function name should be lowerCamelCase.**
```r
SpaDES.core.version <- "0.1.0"
^~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:33:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L33) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("inRAM", "logical", FALSE, TRUE, FALSE, "should the raster be stored in memory?"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:34:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L34) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("nx", "numeric", 100L, 10L, 500L, "size of map (number of pixels) in the x dimension"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:35:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L35) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter("ny", "numeric", 100L, 10L, 500L, "size of map (number of pixels) in the y dimension"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:37:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L37) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", start(sim), start(sim), NA, "time to schedule first plot event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:38:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L38) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInterval", "numeric", NA_real_, NA, NA, "time interval between plot events"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:39:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L39) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "time to schedule first save event"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:40:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L40) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInterval", "numeric", NA_real_, NA, NA, "time interval between save events"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:41:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L41) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".useCache", "logical", FALSE, c("init", "plot"), NA, "should the module result be cached for future use")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:49:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L49) *style:* **Lines should not be more than 100 characters.**
```r
    createsOutput(objectName = SpaDES.core::P(sim, "randomLandscapes")$stackName, objectClass = "RasterStack",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:55:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L55) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.randomLandscapes <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:63:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L63) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, SpaDES.core::P(sim)$.plotInitialTime, "randomLandscapes", "plot", .last())
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:64:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L64) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, SpaDES.core::P(sim)$.saveInitialTime, "randomLandscapes", "save", .last() + 1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:76:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L76) *style:* **Lines should not be more than 100 characters.**
```r
      sim <- scheduleEvent(sim, time(sim) + SpaDES.core::P(sim)$.saveInterval, "randomLandscapes", "save", .last() + 1)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:87:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L87) *style:* **Variable or function name should be lowerCamelCase.**
```r
Init <- function(sim) {
^~~~
```
[inst/sampleModules/randomLandscapes/randomLandscapes.R:101:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/randomLandscapes/randomLandscapes.R#L101) *style:* **Variable or function name should be lowerCamelCase.**
```r
    DEM <- gaussMap(template, scale = 300, var = 0.03, speedup = speedup, inMemory = inMemory)
    ^~~
```
[inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R:17:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R#L17) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".plotInitialTime", "numeric", NA_real_, NA, NA, "This describes the simulation time at which the first plot event should occur"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R:18:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/inst/sampleModules/SpaDES_sampleModules/SpaDES_sampleModules.R#L18) *style:* **Lines should not be more than 100 characters.**
```r
    defineParameter(".saveInitialTime", "numeric", NA_real_, NA, NA, "This describes the simulation time at which the first save event should occur")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:130:22:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L130) *style:* **Variable or function name should be lowerCamelCase.**
```r
      obj$.list[[1]]$._startClockTime <- NULL
                     ^~~~~~~~~~~~~~~~
```
[R/cache.R:131:22:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L131) *style:* **Variable or function name should be lowerCamelCase.**
```r
      obj$.list[[1]]$._timestamp <- NULL
                     ^~~~~~~~~~~
```
[R/cache.R:238:50:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L238) *style:* **Commas should always have a space after.**
```r
          cat(crayon::blue("  Using ", fromWhere," copy of", cur$moduleName, "module\n"))
                                                 ^
```
[R/cache.R:249:50:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L249) *style:* **Commas should always have a space after.**
```r
          cat(crayon::blue("  Using ", fromWhere," copy of", cur$eventType, "event in",
                                                 ^
```
[R/cache.R:299:10:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L299) *style:* **Commented code should be removed.**
```r
        #checkPath(cacheRepo, create = TRUE) #SpaDES dependency
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:352:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L352) *style:* **Lines should not be more than 100 characters.**
```r
      isNewObj <- !names(postDigest$.list[[whSimList2]]) %in% names(preDigest[[whSimList]]$.list[[whSimList2]])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:357:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L357) *style:* **Lines should not be more than 100 characters.**
```r
      pre <- lapply(preDigest[[whSimList]]$.list[[whSimList2]][existingObjs], fastdigest::fastdigest)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:436:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L436) *style:* **Lines should not be more than 100 characters.**
```r
        #   makes soft copy of all objects, i.e., they have the identical objects, which are pointers only
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:466:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L466) *style:* **Put spaces around all infix operators.**
```r
        if (length(object2@current)==0) { # means it is not in a spades call
                                  ~^~~
```
[R/cache.R:466:41:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L466) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
        if (length(object2@current)==0) { # means it is not in a spades call
                                        ^
```
[R/cache.R:468:15:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L468) *style:* **Commented code should be removed.**
```r
          #   length(unlist(object2@completed, recursive = FALSE))/(length(object2@completed[[1]]))
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:480:63:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L480) *style:* **Put spaces around all infix operators.**
```r
          eventsAddedByThisModule <- events(object)$moduleName==current(object2)$moduleName
                                                             ~^~~
```
[R/cache.R:484:42:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L484) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
              b <- lapply(b, function(x) {x[["order"]] <- 2; x})
                                         ^
```
[R/cache.R:487:42:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L487) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
              d <- lapply(d, function(x) {x[["order"]] <- 1; x})
                                         ^
```
[R/cache.R:494:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L494) *style:* **Lines should not be more than 100 characters.**
```r
              #              args = list(append(object@events[eventsAddedByThisModule], object2@events)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:497:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L497) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
              object2@events <- lapply(f1, function(f) {f$order <- NULL; f})
                                                       ^
```
[R/cache.R:499:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L499) *style:* **Lines should not be more than 100 characters.**
```r
              #                           args = list(append(object@events[eventsAddedByThisModule], object2@events)))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:502:12:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L502) *style:* **Commented code should be removed.**
```r
          #object2@events <- unique(rbindlist(list(object@events, object2@events)))
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:506:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L506) *style:* **Lines should not be more than 100 characters.**
```r
        # This is for objects that are not in the return environment yet because they are unrelated to the
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:519:10:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L519) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      for(atts in attrsToGrab) {
         ^
```
[R/cache.R:523:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L523) *style:* **Commented code should be removed.**
```r
      # attr(object2, "tags") <- attr(object, "tags")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:524:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L524) *style:* **Commented code should be removed.**
```r
      # attr(object2, "call") <- attr(object, "call")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:525:9:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L525) *style:* **Commented code should be removed.**
```r
      # attr(object2, "function") <- attr(object, "function")
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:558:66:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L558) *style:* **Variable or function name should be lowerCamelCase.**
```r
  setGeneric(".addTagsToOutput", function(object, outputObjects, FUN) {
                                                                 ^~~
```
[R/cache.R:581:48:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L581) *style:* **Variable or function name should be lowerCamelCase.**
```r
  definition = function(object, outputObjects, FUN, preDigestByClass) {
                                               ^~~
```
[R/cache.R:674:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L674) *style:* **Variable or function name should be lowerCamelCase.**
```r
objSize.simList <- function(x, quick = getOption("reproducible.quick", FALSE)) {
^~~~~~~~~~~~~~~
```
[R/cache.R:701:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L701) *style:* **Variable or function name should be lowerCamelCase.**
```r
makeMemoiseable.simList <- function(x) {
^~~~~~~~~~~~~~~~~~~~~~~
```
[R/cache.R:709:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/cache.R#L709) *style:* **Variable or function name should be lowerCamelCase.**
```r
unmakeMemoiseable.simList_ <- function(x) {
^~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/check.R:151:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/check.R#L151) *style:* **Commented code should be removed.**
```r
    # modules <- sim@modules
      ^~~~~~~~~~~~~~~~~~~~~~
```
[R/check.R:152:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/check.R#L152) *style:* **Commented code should be removed.**
```r
    # userModules <- modules[-which(coreModules %in% modules)]
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/checkpoint.R:36:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/checkpoint.R#L36) *style:* **Variable or function name should be lowerCamelCase.**
```r
doEvent.checkpoint <- function(sim, eventTime, eventType, debug = FALSE) {
^~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:86:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L86) *style:* **Lines should not be more than 100 characters.**
```r
                xAsCall <- .isLastLineSim(x = x, xAsString = bb[seq(funStarts[yy], funEnds[yy + 1])])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:90:16:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L90) *style:* **Commented code should be removed.**
```r
              #y = strsplit(bb[funStarts[yy]], split = "\\s+")[[1]][1]
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:91:53:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L91) *style:* **Commas should always have a space after.**
```r
              y <- paste0(cantCodeCheckMessage, "'",bb[funStarts[yy]], "'")
                                                    ^
```
[R/code-checking.R:144:39:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L144) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
    if (identical(type, "returnSim")) { # This is intended for only the last line of a function
                                      ^
```
[R/code-checking.R:182:48:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L182) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      } else if (identical(x[[1]], quote(`<-`)) ) {
                                               ^
```
[R/code-checking.R:185:62:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L185) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
            if (any(grepl(x[[2]][[2]], pattern = ".xData"))) {# i.e., sim@.xData
                                                             ^
```
[R/code-checking.R:202:19:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L202) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          x <- x[-(1:2)]
                  ^
```
[R/code-checking.R:218:29:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L218) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          x[[2]] <- x[[2]][-(1:2)]
                            ^
```
[R/code-checking.R:320:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L320) *style:* **Lines should not be more than 100 characters.**
```r
    allChecks[anyCantCodeCheck] <- lapply(names(cantCodeCheck[anyCantCodeCheck]), function(objName) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:369:51:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L369) *style:* **Commas should always have a space after.**
```r
                                              " ",verb," declared in outputObjects, ",
                                                  ^
```
[R/code-checking.R:369:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L369) *style:* **Commas should always have a space after.**
```r
                                              " ",verb," declared in outputObjects, ",
                                                       ^
```
[R/code-checking.R:385:20:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L385) *style:* **Commas should always have a space after.**
```r
               " ",verb," declared in inputObjects, ",
                   ^
```
[R/code-checking.R:385:25:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L385) *style:* **Commas should always have a space after.**
```r
               " ",verb," declared in inputObjects, ",
                        ^
```
[R/code-checking.R:396:90:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L396) *style:* **Commas should always have a space after.**
```r
                                       paste0(paste(missingFrmMod, collapse = ", "), " ",verb,
                                                                                ^
```
[R/code-checking.R:441:75:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L441) *style:* **Commas should always have a space after.**
```r
             " but only for the 'get' functions, not the 'set' function ","
                                                                          ^
```
[R/code-checking.R:455:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L455) *style:* **Lines should not be more than 100 characters.**
```r
    missingInMetadata <- simAssignsNotInDotInputObjects[!(simAssignsNotInDotInputObjects %in% outputObjNames)]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:464:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L464) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                       ^
```
[R/code-checking.R:464:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L464) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                                   ^
```
[R/code-checking.R:465:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L465) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in outputObjects")
                                ^
```
[R/code-checking.R:465:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L465) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in outputObjects")
                                            ^
```
[R/code-checking.R:472:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L472) *style:* **Lines should not be more than 100 characters.**
```r
    missingInMetadata <- simAssignsInDotInputObjects[!(simAssignsInDotInputObjects %in% inputObjNames)]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:481:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L481) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                       ^
```
[R/code-checking.R:481:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L481) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," assigned to sim inside ",
                                   ^
```
[R/code-checking.R:482:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L482) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                ^
```
[R/code-checking.R:482:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L482) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                            ^
```
[R/code-checking.R:504:24:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L504) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," used from sim inside ",
                       ^
```
[R/code-checking.R:504:36:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L504) *style:* **Commas should always have a space after.**
```r
                   " ",verbs[[fn]]," used from sim inside ",
                                   ^
```
[R/code-checking.R:505:33:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L505) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                ^
```
[R/code-checking.R:505:45:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L505) *style:* **Commas should always have a space after.**
```r
                   fn, ", but ",verbs[[fn]]," not declared in inputObjects")
                                            ^
```
[R/code-checking.R:517:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L517) *style:* **Commas should always have a space after.**
```r
        " ",verb," used from sim inside ",
            ^
```
[R/code-checking.R:517:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L517) *style:* **Commas should always have a space after.**
```r
        " ",verb," used from sim inside ",
                 ^
```
[R/code-checking.R:519:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L519) *style:* **Commas should always have a space after.**
```r
        ", but ",verb," not declared in inputObjects"
                 ^
```
[R/code-checking.R:519:23:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L519) *style:* **Commas should always have a space after.**
```r
        ", but ",verb," not declared in inputObjects"
                      ^
```
[R/code-checking.R:662:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L662) *style:* **Commented code should be removed.**
```r
    #unique(pd[outerWh, "line1"])
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/code-checking.R:666:34:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L666) *style:* **Commas should always have a space after.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                 ^
```
[R/code-checking.R:666:64:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L666) *style:* **Commas should always have a space after.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                                               ^
```
[R/code-checking.R:666:79:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/code-checking.R#L666) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
      whInner <- any((pd[outerWh,"line1"] < lwf) & (pd[outerWh,"line2"] > lwf) )
                                                                              ^
```
[R/copy.R:37:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/copy.R#L37) *style:* **Variable or function name should be lowerCamelCase.**
```r
            sim_ <- object
            ^~~~
```
[R/downloadData.R:54:96:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L54) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
    io <- .parseModulePartial(sim, modules = list(module), defineModuleElement = "inputObjects" )
                                                                                ^
```
[R/downloadData.R:244:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L244) *style:* **Commented code should be removed.**
```r
    # parsedModule <- parse(file = file.path(path, module, paste0(module, '.R')))
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:245:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L245) *style:* **Commented code should be removed.**
```r
    # urls <- .getSourceURL(pattern = fileToDownload, x = parsedModule)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:252:8:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L252) *style:* **Commented code should be removed.**
```r
      #urls <- moduleMetadata(module, path)$inputObjects$sourceURL
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:277:27:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L277) *style:* **Commas should always have a space after.**
```r
    chksums2 <- chksums[0,]
                          ^
```
[R/downloadData.R:278:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L278) *style:* **Commented code should be removed.**
```r
    #children <- moduleMetadata(module, path)$childModules
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/downloadData.R:281:13:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L281) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
            ^
```
[R/downloadData.R:281:18:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L281) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                 ^
```
[R/downloadData.R:281:54:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L281) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                                                     ^
```
[R/downloadData.R:281:56:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/downloadData.R#L281) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
        if ( all( nzchar(children) & !is.na(children) ) ) {
                                                       ^
```
[R/experiment.R:214:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L214) *style:* **Commented code should be removed.**
```r
    # cl <- tryCatch(getCluster(), error = function(x) NULL)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:215:7:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L215) *style:* **Commented code should be removed.**
```r
    # on.exit(if (!is.null(cl)) returnCluster(), add = TRUE)
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:217:6:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L217) *style:* **Commented code should be removed.**
```r
    #if (length(modules) == 0) modules <- list(modules(sim)[-(1:4)])
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:221:11:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L221) *style:* **Commented code should be removed.**
```r
        # unlist(params[paramsTmp], recursive = FALSE)
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:313:58:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L313) *style:* **Commented code should be removed.**
```r
            params(sim_)[[mod[x]]][[param[[x]]]] <- val #factorialExp[ind,x]
                                                         ^~~~~~~~~~~~~~~~~~~
```
[R/experiment.R:356:11:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/experiment.R#L356) *style:* **Variable or function name should be lowerCamelCase.**
```r
          sim_ <- sim
          ^~~~
```
[R/helpers.R:232:74:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L232) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                             paste0("package:", .pkgEnv$corePackagesVec[-(1:2)]))
                                                                         ^
```
[R/helpers.R:264:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L264) *style:* **Variable or function name should be lowerCamelCase.**
```r
all.equal.simList <- function(target, current, ...) {
^~~~~~~~~~~~~~~~~
```
[R/helpers.R:280:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L280) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm("._startClockTime", envir = envir(target)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:281:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L281) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm("._startClockTime", envir = envir(current)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:282:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L282) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm("._firstEventClockTime", envir = envir(target)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:283:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L283) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm("._firstEventClockTime", envir = envir(current)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:284:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L284) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm(".timestamp", envir = envir(target)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/helpers.R:285:5:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/helpers.R#L285) *style:* **Commented code should be removed.**
```r
  # suppressWarnings(rm(".timestamp", envir = envir(current)))
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/load.R:198:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/load.R#L198) *style:* **Lines should not be more than 100 characters.**
```r
                names(argument) <- c(nam, names(formals(getFromNamespace(loadFun[y], loadPackage[y])))[1])
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/misc-methods.R:466:55:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L466) *style:* **Do not place spaces around code in parentheses or square brackets.**
```r
          (inherits(get(w, envir = envir), "integer")) ) {
                                                      ^
```
[R/misc-methods.R:645:1:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L645) *style:* **Variable or function name should be lowerCamelCase.**
```r
Paths <- .paths()
^~~~~
```
[R/misc-methods.R:661:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L661) *style:* **Variable or function name should be lowerCamelCase.**
```r
    defaults$CP <- TRUE
             ^~
```
[R/misc-methods.R:665:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L665) *style:* **Variable or function name should be lowerCamelCase.**
```r
    defaults$IP <- TRUE
             ^~
```
[R/misc-methods.R:669:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L669) *style:* **Variable or function name should be lowerCamelCase.**
```r
    defaults$MP <- TRUE
             ^~
```
[R/misc-methods.R:673:14:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L673) *style:* **Variable or function name should be lowerCamelCase.**
```r
    defaults$OP <- TRUE
             ^~
```
[R/misc-methods.R:694:73:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L694) *style:* **Commas should always have a space after.**
```r
              if (!defaults$CP) paste0("    reproducible.cachePath = '",normPath(cachePath),"'\n"),
                                                                        ^
```
[R/misc-methods.R:694:93:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L694) *style:* **Commas should always have a space after.**
```r
              if (!defaults$CP) paste0("    reproducible.cachePath = '",normPath(cachePath),"'\n"),
                                                                                ^
```
[R/misc-methods.R:695:67:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L695) *style:* **Commas should always have a space after.**
```r
              if (!defaults$IP) paste0("    spades.inputPath = '",normPath(inputPath),"'\n"),
                                                                  ^
```
[R/misc-methods.R:695:87:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L695) *style:* **Commas should always have a space after.**
```r
              if (!defaults$IP) paste0("    spades.inputPath = '",normPath(inputPath),"'\n"),
                                                                                ^
```
[R/misc-methods.R:696:68:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L696) *style:* **Commas should always have a space after.**
```r
              if (!defaults$OP) paste0("    spades.outputPath = '",normPath(outputPath),"'\n"),
                                                                   ^
```
[R/misc-methods.R:696:89:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L696) *style:* **Commas should always have a space after.**
```r
              if (!defaults$OP) paste0("    spades.outputPath = '",normPath(outputPath),"'\n"),
                                                                                ^
```
[R/misc-methods.R:697:68:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L697) *style:* **Commas should always have a space after.**
```r
              if (!defaults$MP) paste0("    spades.modulePath = '",modPaths,
                                                                   ^
```
[R/misc-methods.R:704:48:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L704) *style:* **Commas should always have a space after.**
```r
              "    reproducible.cachePath = '",normPath(cachePath),"'\n",
                                               ^
```
[R/misc-methods.R:704:68:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L704) *style:* **Commas should always have a space after.**
```r
              "    reproducible.cachePath = '",normPath(cachePath),"'\n",
                                                                   ^
```
[R/misc-methods.R:705:42:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L705) *style:* **Commas should always have a space after.**
```r
              "    spades.inputPath = '",normPath(inputPath),"'\n",
                                         ^
```
[R/misc-methods.R:705:62:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L705) *style:* **Commas should always have a space after.**
```r
              "    spades.inputPath = '",normPath(inputPath),"'\n",
                                                             ^
```
[R/misc-methods.R:706:43:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L706) *style:* **Commas should always have a space after.**
```r
              "    spades.outputPath = '",normPath(outputPath),"'\n",
                                          ^
```
[R/misc-methods.R:706:64:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L706) *style:* **Commas should always have a space after.**
```r
              "    spades.outputPath = '",normPath(outputPath),"'\n",
                                                               ^
```
[R/misc-methods.R:707:43:](https://github.com/PredictiveEcology/SpaDES.core/blob/936f710a3910204d80aa84cadfb171efac7f4111/R/misc-methods.R#L707) *style:* **Commas should always have a space after.**
```r
              "    spades.modulePath = '",modPaths,"'\n",
                                          ^
```


This solves #5


@cubbi do you mind if I ask how you found out these URLs were now available?


Hj fui dar uma olhada na minha aplicação e econtrei um error. Ontem tava tudo certo, mas depois do ultimo commit uma das nossas funcionalidades parou de funcionar. O erro é que ao inves de estar digitando no terminal tava digitando no codigo ai a palavra git entrou no meio do codigo e acabou dando erro na funcionalidade


wtf


Only one issue I see with this commit: function/ts/insertpublication.sql  has extra "character varying" preceding definition.


Yea you could... I'm not sure which is "better" or more efficient


hello , i have page not found when i enter the url : https://github.com/submit50/citizen0x


New changes made here to create global variables to maintain state of search result clicked whilst waiting on challenge.


This new function will be where the code you put for the modal/lightbox overlay to appear.


Please update for Laravel 5.7


![](https://github.trello.services/images/mini-trello-icon.png) [Add HTTP error pages (e.g. 404, 400, 5xx)](https://trello.com/c/fMx0vh7a/36-add-http-error-pages-eg-404-400-5xx)


Wrapped the callback into a single function that continues once the user has solved the challenge.


Shame the Django migrations framework doesn't let me add them back in without breaking everything...


Timer and Ajax trigger starts upon challenge completion


These values are now global so they can maintain state while challenge is being solved.


Yeah, node 11.2.0 has been out for a couple of weeks and nunjucks is therefore broke.


FWIW, PGXN supports only [semantic versions](https://semver.org). So it should be `"1.0.0"`.


looks good to me


Challenge is only made the first time, afterwards it is re-used by using refresh session logic.


The message is a bit confusing. I think that the error is reported when targets with different platforms depend on each other. The error message could reflect that and also print corresponding platforms along with modules' names.


Let's not create a new subdirectory.


I assume there's a reason why you asked to get the notary in a better way; so I thought I should get it from the state, but honestly I don't know why.


Removed the output state since we are "destroying" it


Removed the output state since we are destroying it


Yes, only these two.


3 connections need to be made to the garage door opener from the pi.  Constant power, negative and gpio 21.  GPIO 21 goes to a leg on the button which activates the opener.  You can test your garage door opener by putting a battery in it an run a wire from the positive battery side to leg on opener button which opened the door.  If your opener is same as mine try the way mine is wired in the picture (I have a few pics in 'issues' section).


> When a visual test is executed, a directory within the baseline folder is created. The directory's name is the name of the respective test case.

OK, this makes a lot of sense. Give the single, simple test case I created, this was not clear to me.
So the user doesn't have to modify the `Library    RobotEyes    testing` line to generate the baseline folder?

> It basically does not matter what order of test cases are executed.

I guess I need to create a larger, closer-to-reality test run to better understand how different runs are handled.
Let me do that before bugging you any further 😛

The optional argument would be a great feature, regardless 👍


Still some multiply defined labels that need hunting out, other than that the log is clean. (except a few over/underfull hboxes but you can't have it all


Hum....
Interessante
Seu burro da merda


You mentioned that I should have a better check so that the borrower doesn't initiate the destroy flow; I have in my contract (under destroy command) a check that both borrower and lender should sign the transaction, so even if the borrower initiated; the transaction will be committed only when both sign. Do you have an another idea?


How about we name this file "binder_internal.h" to make it easier to distinguish from the "real" binder.h.


Created multidev environment [rescue-x#oct-18](https://dashboard.pantheon.io/sites/c8430b75-4272-4aca-bfcd-206b12279315#oct-18).

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


Then, it should be rejected properly by front-end. Consequently, appearence of `ZERO` here, in deserialization, should be treated as an inconsistent analysis, and at this point the best we can do is to throw an exception with as much information in the message as possible.


These above few lines still have references to CounterApp


You are good


Technically if you were using the old method of defining a constructur, you'd want to have the same capitalization as the contract name, but the new way of defining a constructor is dropping `function` and simply defining `constructor(...) {`


I'm not entirely certain, but a convention that I've seen Aragon use - and may have some automagical effect - is naming ACL roles as all caps suffixed with `_ROLE`. There may be something to that...


As per your recommendation; the otherParty should come from the state not from the constructor; did I understand your concern right?


no need for this line

general review: leave a space between ) and {

no need for the {  on the new line, be consistent
     if (env.NEW_VMS_EMV == 'NO_NEW_VMS')
         {


Now could not deploy this pull request since the `now.json` file explicitly sets `dotenv` to true.
However, there's no `.env` file in the repo.

Commit a `.env` file or unset the `dotenv` flag.
For secret environment variables, check out [Now Secrets](https://zeit.co/docs/getting-started/secrets).


Created multidev environment [dgreat#pr-396](https://dashboard.pantheon.io/sites/41cad7fe-472d-43a9-a470-d391b9622376#pr-396).

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


Here's a link to the previous version so you can read your earlier comments if you need to:
https://github.com/adelRestom/cordapp-example/commit/ec39532042443db55407cf032dba0ef643048ab4


Successfully aliased the URL <https://reasonml-native-lisbon-13d8x7pg2.now.sh> to the following alias.

* https://reasonml-native-lisbon.now.sh


We mainly want `first` to be non-nil, and `sit-for` conveniently returns `t`. I didn't think about the edge case when the user interrupts `sit-for`.


I'm a bit confused by the wording here.
A modules does not have a compiler, it has a target platform.
Also it's a bit confusing, because currently only JVM modules can form cycles.
Also I think that JVM to JS dependencies (and vice versa) should be prohibited even when modules don't form cycles.


@DiegoSanchezE Btw Feel free to raise issues when you're done using the library extensively. One learning for me from this discussion would be to create a "How it works" section in the ReadMe. That should provide some clarity to others on how things work :)


ok


NetflixSubs0.13.18 error not download from Netflix, please help fix it


Я добавил exception and entity в прошлом коммите ( так случайно получилось )


## [Preview Installation](install-preview.md)


When i try Loader class in MainActivity than it show an error but when i place it in separate class than it run gracefully.


xD


Ah yes, that's right. Now I recall that at some point I've specifically introduced `cfgWithStdLib` tests, because I couldn't test contracts otherwise, which means that usual `cfg` tests couldn't use stdlib -> couldn't use `let` and other similar functions too.


Da hast du recht, der ist überflüssig und wird entfernt.


Change to: "Before reading this chapter, you should have a sound understanding of the basic concepts explained there."


Add a comma after "Right now"


> so we have a whitelist default to json

@SamSaffron did you have an idea of to do this? I came up with one solution - enclosing the entire `routes.rb`
file with

```
scope path: nil, constraints: { format: /(json|html)/ } do
    # Then you can override the constraint for a specific route
    get "somepath" => "controller#action", constraints: {format: /(xml)/}
end
```

That won't apply to plugins (e.g. polls) which append routes at runtime, but it is still better than what we have now.

What do you think, is there a tidier solution you were thinking of?


Add "you can" before "retrieve" - consistency again.


Consider replacing a property with an explicit parameter.
In this case you need to share `buildLogFile` between `doTest` and `performModificationsAndMake`.
The latter is called only by `doTest`.
In this case there is no benefit in using a mutable property.
But the property is not cleared after a test case finishes, so
it's possible to accidentally read a wrong value from it (not now, but in future).


This looks like a good, clean approach. As I mentioned in the comments, I'd like a CONFIG option to avoid compilation of the binderfs code -- some devices are *extremely* memory constrained and partners will insist on trimming dead code as much as possible, so let's just do it up front.


Log files for Real_Estate_NA &  Real_Estate_EAP.  Also, updated YAML files for Real_Estate_NA &  Real_Estate_EAP.


@killeriukas did you try disabling auto-resolution in the settings to work around this?


Meinst du einen Countdown im Sinne von Zeit? Wenn der Nutzer weg ist, würde ich direkt mit Senden aufhören.


# 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)|


Why statically configure this rather than get the IP on eth1 ?


Hi! This is the friendly automated conda-forge-webservice.

I updated the Github team because of this commit.

- @boegel was added to conda-forge. Welcome to conda-forge!
  Go to https://github.com/orgs/conda-forge/invitation see your invitation.

You should get push access to this feedstock and CI services.

Feel free to join the community [chat room](https://gitter.im/conda-forge/conda-forge.github.io).

NOTE: Please make sure to not push to the repository directly.
      Use branches in your fork for any changes and send a PR.
      More details [here](https://conda-forge.org/docs/conda-forge_gotchas.html#using-a-fork-vs-a-branch-when-updating-a-recipe)


Successfully aliased the URL <https://sso-zeit-os6bc75c2.now.sh> to the following alias.

* https://sso.philalsford.com


Klar, das Senden Richtung User sollte dann sofort aufhören.
Aber der Producer (das Ding das tweets von Twitter in den Kafka schaufelt) sollte erst dann beendet werden wenn dessen letzter User seit X Minuten offline ist.
Kann ja sein das der User nur kurz WLAN-Probleme o.ä. hatte. Da wäre es doof den Producer neu starten zu müssen .


Thanks Régis!


I think you need mac.com, too?


Why statically peg these IPs... we are already getting unique IPs on the private network [here](https://github.com/ligato/networkservicemesh/blob/dd2e826188fd7b4661970e0af70c2cc4365e0854/scripts/vagrant/Vagrantfile#L21)


Successfully aliased the URL <https://sso-zeit-2cfny8d0e.now.sh> to the following alias.

* https://sso.philalsford.com


So would something like the following be closer to intentions?

```diff
diff --git a/counsel.el b/counsel.el
index 7b91ff1..2cc70ee 100644
--- a/counsel.el
+++ b/counsel.el
@@ -4523,21 +4523,22 @@ counsel--sl

 (defun counsel-rhythmbox-play-song (song)
   "Let Rhythmbox play SONG."
-  (let ((first (when (equal (shell-command-to-string "pidof rhythmbox") "")
-                 (counsel--run "nohup" "rhythmbox")
-                 (sit-for 1.5)))
+  (let ((first (string= (shell-command-to-string "pidof rhythmbox") ""))
         (service "org.gnome.Rhythmbox3")
         (path "/org/mpris/MediaPlayer2")
         (interface "org.mpris.MediaPlayer2.Player"))
+    (when first
+      (counsel--run "nohup" "rhythmbox")
+      (sit-for 1.5))
     (dbus-call-method :session service path interface
                       "OpenUri" (cdr song))
-    (when first
-      (let ((id (cdr (counsel--wmctrl-parse
-                      (shell-command-to-string
-                       "wmctrl -l -p | grep $(pidof rhythmbox)")))))
-        (when id
-          (sit-for 0.2)
-          (counsel--run "wmctrl" "-ic" id))))))
+    (let ((id (and first
+                   (cdr (counsel--wmctrl-parse
+                         (shell-command-to-string
+                          "wmctrl -l -p | grep $(pidof rhythmbox)"))))))
+      (when id
+        (sit-for 0.2)
+        (counsel--run "wmctrl" "-ic" id)))))

 (defun counsel-rhythmbox-enqueue-song (song)
   "Let Rhythmbox enqueue SONG."
```


@aembler I think that Scrutinizer is right:
The method ``getMiddleware`` cannot be called on ``$middleware`` (of type ``string``).


HAHA!


A transaction can have only 1 notary. If 2 input states and the tx notary are different, it will be `verify`.
So you might as well choose one that increases your chances to `verify`.


Copy of this build can be found at
https://www.mewbuilds.com/build/f9be328248678a7bd9f03a7d28b6244d410ef70e/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmRGScd2jKuY9AP3Mye2gEFKLPSCQ4vTXaCRJaGu56S1e2


Where is the input state added?


Overall, look good.  A few issues:
ti.getdatasetpublicationstatusv2  should return Boolean (not small int); can v2 suffix be dropped?
ti.getelementtypebydatasettypeid hard coded to use datasetsetytpeid of 5 and ignore input parameter; same as current version on SQLServer.  Should check with Eric why that is.
ts.insertdepagenttypes This one is using the T-SQL syntax.  Should Return integer and instead of Select scope_identity() use  "INSERT INTO ndb.depagenttypes(depagent)
VALUES (_depagent) RETURNING depagentid
The last comment is on the option to use an alias.  Instead of something like "SELECT ndb.elementtypes.elementtypeid" in the FROM define an alias like "FROM ndb.elementtypes et" and in the SELECT use et.elementtypeid or something similar.  Fully-qualified is through but alias can reduce typing and make the text slightly more readable.


It should come from the state indeed.
So this flow can only be started from the lender's node, right? Or how can you make it such that it can be started from either side?


That worked!  Thank you!


okay will try to do same on our replica


Created multidev environment [d8-nature-up-north-v2#pr-1](https://dashboard.pantheon.io/sites/f696edcc-8347-415d-baeb-c4c23d243ebe#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-d8-nature-up-north-v2.pantheonsite.io/)


We will see this later. But in this situation you understand that the `Acceptor` accepts all.
We will add something in the vault to identify that "when this IOU comes for destroy, we will accept".
The Corda team will also improve this part, AFAIK, for instance by making it possible to "pop up" to the human in charge.


Hi! This is the friendly automated conda-forge-webservice.

I updated the Github team because of this commit.

- @cj-wright was added to this feedstock maintenance team.

You should get push access to this feedstock and CI services.

Feel free to join the community [chat room](https://gitter.im/conda-forge/conda-forge.github.io).

NOTE: Please make sure to not push to the repository directly.
      Use branches in your fork for any changes and send a PR.
      More details [here](https://conda-forge.org/docs/conda-forge_gotchas.html#using-a-fork-vs-a-branch-when-updating-a-recipe)


Alles klar, verstehe wie du das gemeint hast.


HI


It might be better to have a more precise advice: "Enable 'Delegate IDE build/run actions to Gradle' in Settings "


@larsjaas Forstår jeg det rett hvis dette er for å håndtere query params etter forum/index.php? Vi kan vel kanskje fjerne "url == 'forum/index.php' fra if'en lenger opp, da den også vil fanges av denne? Then again, direkte string-sammenligning er kanskje raskere enn regexp, så denne vil kun bruke tid hvis det er query params.


@michaelcahill, I didn't catch this in my initial review, but those `%d` formats won't match the types of `offset` and `size`, will they?


@halonazhao fortunately there is a simple solution to break the cycle.   Unitals.hs should not mention these assumptions.  The definition of load resistance is independent of our assumptions.  The definition of load resistance is actually for any type of glass.  Our calculation of load resistance needs to invoke the assumptions, but we do that through the data definition.  I will update the case study so that these assumption is not used in the terminology and definitions section.


@michaelcahill, I don't see any reason to remove this, but I don't understand why this has an explicit transaction.


🔥 [Jenkins job](https://jenkins.internalx.com/job/macios/job/master/234/display/redirect) (on internal Jenkins) failed in stage(s) 'Test run' 🔥

✅ [Build succeeded](https://jenkins.internalx.com/job/macios/job/master/234//console)
✅ [API Diff (from stable)](http://xamarin-storage/jenkins/xamarin-macios/master/acf8357ece1e43ed0d354e3c5e352a067929e96b/234/jenkins-results/api-diff/index.html)
⚠ [API Diff (from PR only)](http://xamarin-storage/jenkins/xamarin-macios/master/acf8357ece1e43ed0d354e3c5e352a067929e96b/234/jenkins-results/apicomparison/api-diff.html) (🔥 breaking changes 🔥)
✅ [Generator Diff](http://xamarin-storage/jenkins/xamarin-macios/master/acf8357ece1e43ed0d354e3c5e352a067929e96b/234/jenkins-results/generator-diff/index.html) (no change)
🔥 [Test run failed](http://xamarin-storage/jenkins/xamarin-macios/master/acf8357ece1e43ed0d354e3c5e352a067929e96b/234/jenkins-results/tests/index.html) 🔥

# Test results

<details>
<summary>1 tests failed, 0 tests skipped, 283 tests passed.</summary>

## Failed tests

 * [xUnit] Mono SystemJsonXunit/watchOS - simulator/Debug: Crashed
</details>


Is it a valid branch? Just imagine errors messages if this execution path is actually taken.


Pitäisi saada tämä tdsto poistettua.


based on the ?. 'local' could be null.  that seems bad to run the later code.  For example, i wouldn't expect FindReferenceAsync to be ok in that situation.


@andre-merzky, Travis fails because you import `json` and it is not used.


Try to have meaningful commit messages. Assume that hiring managers are looking at each commit messages.


Thanks Ralf!


can we push this check outside and run two for loops?


I wonder if this is really necessary, since a chunk's size cannot be > 1 for any MPP or Gradle project.


cool


Why not to use `MDL_BACKUP_TRANS_DML` with transactional system tables?


Are we sure that FeatureType is always float?


Damn.

I should really keep my hands off any Windows system.


The correction to the manual version of the case study was made in the glass_revisions branch of the Case Studies repo:

https://github.com/smiths/caseStudies/commit/05877c8b92cbcb8768abf5ac371426a752c1e4a8


Alright, I will make the change.


Ah, let me remove the `?.` and replace the prior `as` with a direct cast. We are sure that declStatement is of the form `Type x;`, i.e. declaration with a single variable and no initializer, and GetOperation should always return an `IVariableDeclarationGroupOperation` here.


its reset html pages


We need to guard ourselves here for really large files. Perhaps set a maximum limit to do the whole file reading into memory?


again shocking.


That does sound right. Because elements of `(use-compiled-file-paths)` always extend "compiled" in practice, probably "compiled" works well enough, but adding the paths in `(use-compiled-file-paths)` sounds more correct.


Successfully aliased the URL <https://sso-zeit-fq3flg8vr.now.sh> to the following alias.

* https://sso.philalsford.com


yup.  not good.  can you file issue? (or just fix if easy).


Aliasing failed with the following error:

~~~
Error! The alias muv is a deployment URL or it's in use by a different team.
~~~


it's because import glob loader adds its own semicolon after each import. so if we already have a semicolon there, the resulting code is two semicolons, which pisses off eslint.


@davkean The clickonce support in 16.0 is for .NET Core or .NET Framework projects? or both?


i personally think it should just be `out _`.  It's effectively doucmenting the parameter at not being relevant at all.  adding more info to it seems superfluous.


blank line should not have been removed here.


Yes, it does work with auto-resolution disabled, and I also had to disable the warning of auto-resolution being disabled. I, however, do not know if that is a good thing to leave it off. Ideas?

As a workaround - it worked, though. Thanks.


We should probably document that convention, since currently it's mostly enforced via the convenient `.gitignore` pattern.


are we not giving the discard (or the statement) the leading trivia of the original statement?  we should fix that if that's the case.


That was the initial implementation for expression statements (which we removed later as that diagnostic is now hidden). This diagnostic is for unused value assignment, where we don't do any type based bail outs.


this means datascript code is distributed as classfiles compiled using clojure 1.7. if and when subsequent clojure releases break the abi (add. remove methods in the runtime, etc) this will break, and users will not be able to use the datascript library. distributing libraries with aot'ed code is a terrible idea, please reconsider this


Successfully aliased the URL <https://sso-zeit-38bjpu718.now.sh> to the following alias.

* https://sso.philalsford.com


Created multidev environment [d8-nature-up-north-v2#ci-15](https://dashboard.pantheon.io/sites/f696edcc-8347-415d-baeb-c4c23d243ebe#ci-15).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-15-d8-nature-up-north-v2.pantheonsite.io/)


it's really nice that it does this.


Now zodiac is adding 6 bytes and not 4 bytes in each MPLS header, and adding the ethernet type of service after each MPLS header without sense. And in the POP  the same.


@auzunov-hhog

I believe Kamsar, Unicorn and Rainbow (and MicroChap) are meant to stay (not remove) since it isn't part of Sitecore's base install.

Is it also possible to have the "exclude" list have the wildcard Microsoft.* but then the Include version would override the exclude?

Similar to what gitignore does:
/Microsoft
!/Microsoft/Microsoft.example.dll

Looks like it already supports it based on the fact that you exclude Microsoft.Owin.* but then include Microsoft.Owin.Security.Facebook.


I am going to fix this to be `out _` and we can come back to it later if named argument is desirable.


wow :)


Successfully aliased the URL <https://sso-zeit-h31g7znw6.now.sh> to the following alias.

* https://sso.philalsford.com


На сколько я помню, если реализацию темплейтов разделять на h и cpp файлы, то в cpp файле надо потом дописывать нечто template class List<int>, и так надо добавлять для всех типов, которые используешь с этим шаблонным классом, и получается неудобно. Лучше все в h-ник кидать


this is only if you know htat "new TextSpan" is side-effect free.

i def think the main benefit of htis analyzer is to identify areas of interest.  The fixer itself is helpful in a few places.  But it's much more about shining a light on code that really shoudl be examined.


Copy of this build can be found at
https://www.mewbuilds.com/build/0856e4a99a1bd946a6c5bda963705095dd3f6001/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmZFqeThRHfHTrwCtxkPZ1AtHqxsrWatv7nHwx35KKmKgT


newline should not be removed.  i would consider this a largish problem with teh fixer.


Copy of this build can be found at
https://www.mewbuilds.com/build/d106454bf2df8150a978e20bd90447bf3a66e950/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmRJr7TVGvenpCeY14uEjA3fui9C3moMKE4jHr4cpJF3Xg


import


Successfully aliased the URL <https://sso-zeit-l68q5s8t8.now.sh> to the following alias.

* https://sso.philalsford.com


@nfeske, @chelmuth: please add to staging


Copy of this build can be found at
https://www.mewbuilds.com/build/a21f5a9c0eb78137de36c07f7451f2a2ee0b47fe/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmSHnrZaquUowJ3itPhQzhURiS2yM748boj9pLsgb3MDyJ


Created multidev environment [dgreat#pr-396](https://dashboard.pantheon.io/sites/41cad7fe-472d-43a9-a470-d391b9622376#pr-396).

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


Copy of this build can be found at
https://www.mewbuilds.com/build/952b64aedc50a30638664881078e11bff264e72f/index.html
                       https://www.cloudflare-ipfs.com/ipfs/QmWBHkyDCnBTWivEszLQ2ccwwvYoULSDf2pypKvzWZM9fW


I wonder if there's a sane way for us to share the configure options to various libs.  Maybe there isn't though, for now.


Successfully aliased the URL <https://sso-zeit-p9ywdswmh.now.sh> to the following alias.

* https://sso.philalsford.com


Thanks for the explanation!


Successfully aliased the URL <https://gh-release-branch-status-quhkciesgs.now.sh> to the following alias.

* https://gh-release-branch-status.now.sh


Bør vel bare skille path og query-string med en gang, så kan man bruke == i mye større grad nedover den ifelse-lenken...


Right, it just tells me we probably shouldn't _merge_ the fixes until they are reviewed in detail to find cases like this though. Sometimes we want to just Fix All and merge, but this is not one of those cases.


The diagnostic/fix here is aimed at pointing out _potential_ bugs. It cannot figure out if the value being assigned is *real* computation with its value being discarded OR just a redundant computation/allocation that can be removed. Former is a safe fix that does not change semantics, so is offered for this diagnostic, and would be a stepping stone towards the users deleting the entire statement if it is indeed the latter (redundant computation). I was planning to file an issue to offer a refactoring that offers to remove statements of the form `_ = ...`, which can be invoked after the code fix to delete such redundant code.