fasterer

fasterer copied to clipboard

Array#count vs Array#size

1

I added the offense `Array#count` vs `Array#size`/`Array#length`. I'm aware that this will generate a lot of noise since `.count` is ActiveRecord method used very often. But keep in mind that...

sinecode

Update #check_fetch_offense method

3

Updated method `#check_fetch_offense` to verify method `#fetch` if it has a block and return a correct error message. Related issue: https://github.com/DamirSvrtan/fasterer/issues/37

connicov1

Feature Request: Line-level Inline Exclusions

1

@DamirSvrtan hi! Thanks for gem, we used it as part of our ci linting tools in company. We faces an issue that we want manually disable speedups by comment in...

fargelus

Fasterer speedup for hash_update_vs_hash_brackets

1

**Hash.update** is an alias for **Hash.merge!** Currently we have `hash_merge_bang_vs_hash_brackets` which suggests us to replace **Hash#merge!()** with **Hash#[]** but it doesn't suggest for **Hash#update()**. **Example** - hash.merge!({"key" => "value"}) ->...

manjunath724

Add support for json output

1

It'd be nice to have JSON output, maybe something like `--format` json or even `--json`. This would make the integration with editors like VSCode way easier.

fnando

Feature Request: Line-level Inline Exclusions

5

I think it would be a great enhancement if I was able to disable/exclude a speed suggestion for a specific line via a magic comment (much like how Rubocop does...

ChefAustin

Recommended fetch with block over args as faster

7

From https://github.com/rails/rails/pull/26599 i'm seeing that's not the case. Btw, this is a really cool project. cc/ @timrogers

schneems

Hash#update() vs Hash#[]=

2

Hash#update is an alias for Hash#merge! Ergo, the suggestions for Hash#update should be similar to Hash#merge! Ref https://github.com/DamirSvrtan/fasterer/issues/82 Link to Fast-Ruby PR : https://github.com/JuanitoFatas/fast-ruby/pull/195

manjunath724

Replace colorize with rainbow

6

This change replaces `colorize` gem with `rainbow` gem. That is because `fasterer` has MIT license, and `colorize` has GNU GPL license, hence `rainbow` with its MIT license seems a better...

robertino

Add links to report

2

#63

dukaev