Bozhidar Batsov

icon indicating a website link https://metaredux.com icon indicating an email [email protected]

icon company Toptal (@toptal) icon location Bulgaria icon location Hacker. Emacs fanatic. Author of RuboCop, CIDER, Projectile and others. I'm not the best, but I'm pretty good. ;-)

Results 1757 comments of Bozhidar Batsov

Replace regex with `Bundler::LockfileParser`

I agree it's not prudent to add a runtime dependency on Bundler, so I see three paths ahead: - Using this parser only when available - Inlining something similar into...

Replace regex with `Bundler::LockfileParser`

Ah, excellent - so I guess you know how to proceed now. :-)

Replace regex with `Bundler::LockfileParser`

@amomchilov Any updates here?

Replace regex with `Bundler::LockfileParser`

> I'm not sure how to proceed. Do you think I should modify RuboCop::Lockfile to take in the path as a param (as determined by the existing RuboCopConfig#bundler_lock_file_path logic), or...

Replace regex with `Bundler::LockfileParser`

I guess you can rebase this one now that #12186 is finally merged.

[Fix #12242] Support `AllowModifiersOnAttrs` option for `Style/AccessModifierDeclarations`

Hmm, I'm a bit puzzled by the proposal as I imagine that almost everyone will prefer to use the `inline` style in this case. Is there something I'm missing here?

[Fix #12242] Support `AllowModifiersOnAttrs` option for `Style/AccessModifierDeclarations`

@krororo Sorry for dropping the ball on this! Too much focus on Prism support recently. > For example, in applications that have been maintained for a long time, much of...

[Fix #11878] Add new `Style/MapIntoArray` cop

I like the cop overall, but I'm a bit unsure about the name. @rubocop/rubocop-core Suggestions welcome!

[Fix #11878] Add new `Style/MapIntoArray` cop

I guess it's time to finally merge this. Thanks for the PR, @ymap!

[Fix #12547] User experience for infinite loop detected

I think that's a good small improvement. Sorry for taking so long to merge this!