Jonathan del Strother

If we're ok dropping support for ansible-lint < 5, I'll rebase & submit [my branch](https://github.com/flycheck/flycheck/compare/master...jdelStrother:flycheck:ansible-lint) as a new PR? (ansible-lint 5 has been out for over 2 years now)

I could be wrong, but I believe this is caused by Rails running its built-in EXPLAIN query, and QueryReviewer running an EXPLAIN on that. I'm not sure it's anything to...

Is everyone here using the mysql2 gem, and only seeing it on inserts?

So... I can reproduce just by loading up Rails console, enabling query reviewer with : ``` ruby Thread.current["queries"] = QueryReviewer::SqlQueryCollection.new; Thread.current["query_reviewer_enabled"] = true ``` and then performing an insert (eg...

@cjheath I'm a bit confused by the update - the readme tells people they can download a Geolite2 database from https://dev.maxmind.com/geoip/geoip2/geolite2, but those aren't supported with GeoIP, right?

I’ve not tried it yet, but https://github.com/yhirose/maxminddb is pure ruby and handles geoip2. If it’s any good, we could point users to that rather than maintaining a competing gem

Yes please - any hope of a new release with this in?

I'm still seeing this with stylelint 15.3.0 and stylelint-scss 4.6.0. What am I missing? ```shell $ yarn stylelint 'app/**/*.scss' app/assets/stylesheets/_definitions.scss 47:16 ✖ Unexpected unknown annotation "!default" annotation-no-unknown 77:24 ✖ Unexpected...

I think this was broken by 1dea755592e0afa8e49289ce98dcc8bcbea3536f, which exits early from `add_query_output_to_view` unless `cookies["query_review_enabled"]` is set.

I think my pull request would involve just deleting the `return unless Thread.current["query_reviewer_enabled"]` line... but then, I can't quite follow exactly why that's there in the first place. @mikegee ?