It's not needed to put the require option in the Gemfile, since your gem already requires everything you have in lib folder.

So in your Gemfile you can simply put: gem "sinatra-param"
Instead of: gem "sinatra-param", require: "sinatra/param"

Effective Go says:

Every package should have a package comment, a block comment preceding the package clause.

The command mvdan.cc/unparam does not have a package comment.

This can be seen by visiting https://pkg.go.dev/mvdan.cc/unparam or running go doc mvdan.cc/unparam inside the module.

Currently we use a docstring trick to generate signatures for Parameterized classes. This approach is no longer supported by IPython in recent versions and is also pretty hacky (and ugly to look at in a docstring since all the keywords appear there). Instead we should use the __signature__ object (see PEP362).

![Screen Shot 2020-01-18 at 9 51 33

For regressionBF, I don't know how to interpret the output from model_parameters and the documentation is not giving me any hints. Is this something we should add to the docs?

BayesFactor output

# setup
set.seed(123)
library(parameters)
library(BayesFactor)
#> Loading required package: coda
#> Loading required package: Matrix
#> ************
#> Welcome to BayesFactor 0.9

Currently if you run the command, and you haven't set up AWS credentials properly, it won't execute your command, but have a zero exit status, suggesting that everything went okay. Wouldn't it make more sense if it exited with a non-zero status?

Example:

bash-4.4# env-aws-params --prefix /foobar /bin/ls && echo $?
ERROR: NoCredentialProviders: no valid providers in chain. Deprecated.

Okay, so if you pass a filterWith rule, that function should return true|false.

However, you can throw a custom error in that function instead. It's just that the customer error needs to implement toJSON and return { field, message }.

Currently ToString / Save methods in SharedParameterFile ignore the original format (e.g. the absence of Description and/or UserModifiable columns). File shall be persisted exactly as it was read

• Fix Travis stalls build when it comes down to testing lkm_mev (local run works)
• Fix Build Warnings
  warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
• Fix following warnings/issues in .travis.yml
  • root: duplicate key: install
  • root: deprecated key sudo (The key sudo has no effect anymore.)