Bumps [loofah](https://github.com/flavorjones/loofah) from 2.3.0 to 2.3.1.
- [Release notes](https://github.com/flavorjones/loofah/releases)
- [Changelog](https://github.com/flavorjones/loofah/blob/master/CHANGELOG.md)
- [Commits](https://github.com/flavorjones/loofah/compare/v2.3.0...v2.3.1)
Signed-off-by: dependabot[bot] <support@github.com>

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.6.6.2 to 1.10.4.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sparklemotion/nokogiri/compare/v1.6.6.2...v1.10.4)
Signed-off-by: dependabot[bot] <support@github.com>

Changed references to `--additional_file_patterns` to be `--additional-file-patterns` to make it consistent with other flags.

Adds tests for `.include_routes?`, `.include_models?`, `.skip_on_migration?`. Also moves the `TRUE_RE` under the `Annotate::Constants` namespace.

Prior to this change, `Annotate.include_models?` returned the inverse of `Annotate.include_routes?`. This made it so annotating models and routes was not possible to do together.

This PR adds an explicit `--models` flag and also adds it the option to `lib/generators/annotate/templates/auto_annotate_models.rake` with the default being set to `false`. 

Fixes #563 and undoes the bug introduced in #485.

In looking into #563, I realized there are gaps in test coverage. For example, `bin/annotate` doesn't have any tests. 

Tidying the spec directory to make it easier to add tests in future work.

Adds option for additional file patterns (implemented in #633) in the CLI.

Refactored Parser to isolate changes being made to ENV. This way we have an intermediate step where we know the environment variables being set.

This PR adds an option named `additional_file_patterns`.  You can specify custom path patterns (including globs) that the gem will use to annotate.

For example, I used it like this on a project:

```ruby
Annotate.set_defaults(
      'additional_file_patterns' => [
        File.join(Rails.application.root, 'app/lib/forms/%PLURALIZED_MODEL_NAME%/**/*.rb'),
        File.join(Rails.application.root, 'spec/lib/forms/%PLURALIZED_MODEL_NAME%/**/*.rb')
      ],
      ...
)
```

This makes it possible to have files nested under a directory which corresponds to the model.

I believe this fixes #594.

Currently there's no test coverage on CLI. This adds tests for a new class Parser which will replace the command line options currently in bin/annotate.

The technical direction I'm planning to go is to remove ENV variables completely and to have things passed into as arguments. Will be adding deprecation warnings in 3.0 and then deprecate ENV variables completely in 3.1. Deprecation warnings will be non-blocking in 3.0 when an ENV variable is set and an argument isn't passed in, and will becoming blocking in 3.1.

`BigDecimal.new` is deprecated in future version of Ruby in favor of using `BigDecimal(...)`.

Handle potential negative padding in `#mb_chars_ljust` when having a `col_length` > 16, the default hardcoded value of `bare_type_allowance`.

* Update rubocop requirement from ~> 0.46.0 to ~> 0.67.2

Updates the requirements on [rubocop](https://github.com/rubocop-hq/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop-hq/rubocop/releases)
- [Changelog](https://github.com/rubocop-hq/rubocop/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rubocop-hq/rubocop/compare/v0.46.0...v0.67.2)
Signed-off-by: dependabot[bot] <support@dependabot.com>

* Make changes to let Rubocop pass in CI

Add --frozen option to abort instead of write annotation.
This is mainly for CI environment to detect missing annotation.
Similar motivation with bundle install --frozen.

works like:
```
$ rake db:migrate frozen=yes

annotate error. spec/models/user_spec.rb needs to be updated, but annotate was run with `--frozen
```

- Use Rake::Task#enhance insteaad of defining same tasks again
- Remove hooking db:migrate:change task which doesn't exist
- Fix hooking db:migrate:reset task so that the annotation runs after
  all migration tasks (#548)

This fixes alignment of Japanese, Korean and Chinese fullwidth character comments.

The displayed widths of multi-byte fullwidth characters are generally
twice as large as the ASCII characters, but String#size returns only the number of characters.
So if the column comment contains fullwidth multibyte characters, the alignment is broken.

In rails 4, columns do not respond to `bigint?`.  However, in both rails
4 and rails 5, columns do respond to `sql_type`. This way, annotations
should work in both versions.

* Hook up with-comment option to rake tasks and CL.
* with_comment option handles commentless columns.
* Style fixes.

* Refactor AnnotateModels#get_model_files
* Add the tests for `AnnotateModels#get_model_files`
* Remove unnecessary `reject` call

Fix #436 where extra line break is added when using windows CRLF (#490)

* add show_complete_foreign_keys as setting

* test for show_complete_foreign_keys setting

Support Database Comment for Rails 5
Added with_comment options

Do not check if an index includes another column if the name is a String, instead of an Array. (#442)

* Do not check if an index includes another column if the name is a String, instead of an Array.

* Fix Rubocop test conditions.