bbatsov/rubocop

A robust Ruby code analyzer, based on the community Ruby style guide.

Oppen

@Oppen

I mean, I'm working in my changeset, I then upload it to a branch on my fork and create a PR, but then I can't go back and change it to use the proper id afterwards.

(I could in a different commit, after creating the PR, but I've seen it done in a single one)

John Vandenberg

@jayvdb

@Oppen after you create the PR, you can add another commit to your branch and it will be included in the PR. You can also 'amend' (replace) the initial commit

Oppen

@Oppen

OK. I thought of amend, but that always carried weird consequences when I forced pushed.

Thanks.

Oppen

@Oppen

Hi all. I'm working in a performance related cop. Should I send it to rubocop, rubocop-performance or both?

Simon

@stoivo

I have a open PR rubocop-hq/rubocop#6649, Could someone take a look at it and give me some feedback

Lewis Jones

@LewisARJones_twitter

Hi, I use Rubocop to write custom cops within my companies code bases to enforce our domain and help aid big refactors. I've got blocked trying to write cops that check if methods are called on instances of our domain object. Basically ruby classes we have created.
For example I want to write a cop that creates an offence if anyone sends the method expiry_date to and instance of Quote object object.
Can anyone point in the direction of an example of how to write a cop that can check the class of the object getting the method sent to it?

Thomas Thomassen

@thomthom

@LewisARJones_twitter The dynamic nature of Ruby makes this hard. I've tried to do something similar for rubocop-sketchup - but I found no good way to do so. In some cases I've scanned the parent scope to find the initialisation of the instance. But the moment the instance is passed on to other methods it gets really difficult to track. In the end I fell back on doing simple naive checks like mentioned. Or make educated guesses on variable names.
You can see this being a challenge even for IDEs like RubyMine or VSCode with Ruby+Solargraph extensions. They tend to make use of YARD to pick up types from doc comments to aid the static analysic. But they also have to analyse the whole project to have chance. If you have good YARD comments you could try to leverage that in your custom cops.

Jônatas Davi Paganini

@jonatas

@LewisARJones_twitter any reason to not deprecate the usage of expiry_date in the Quote object? because it seems more reasonable than try to create a linter for it. Make the interface fail or warn developers about it. When they run tests they will know the api is no longer going to support it.

Lewis Jones

@LewisARJones_twitter

Thanks for your help @jonatas and @thomthom

Mailo Světel

@roolo

Hey guys, I just discovered AllCops are being rewritten by children configs, what is the right way to do it?

https://gitlab.com/lipoqil/rubocop-mailo/blob/459031ee5e101aa471dca90b202836f8d37025ca/rubocop.yaml#L1-7 If I use AllCops - Exclude in file which inherits from this one, the configs from gem are ignored 🥺

Rubocop 0.65.0

Mailo Světel

@roolo

https://github.com/rubocop-hq/rubocop/blob/master/manual/configuration.md#merging-arrays-using-inherit_mode FTW

Philipp Hansch

@phansch

Hi, is there some way to write a cop that detects usage of :[] on Hashes and suggest fetch instead? Specifically, how could I limit this to only work on Hash?

I suppose this is more difficult as I thought because of Ruby's dynamic nature..

i.e, I'm currently getting false positives on Dir[pat], some_array[x], etc

Jonas Arvidsson

@jonas054

@phansch The short answer is "No". You'd have to add functionality in RuboCop to guess the type of values, similar to what "intellisense"/auto-completion does in IDEs.

Colin Swenson-Healey

@chozandrias76

Has anyone else recently updated rubocop, added the rubocop-performance gem and found that rubocop no longer inspects any of the files you had it inspecting before?

It is inspecting only 2 files now when I run rubocop and I don't know which those are or why anything has changed

Lee-Jon

@lee-jon

Hey @chozandrias76 did you solve the issue? I have the same issue jumping from 47 -> 75

Lee-Jon

@lee-jon

I solved it by deleting the yml file and rewriting it line by line.

Vincent

@vincent-psarga

Hi all,
I'm having some difficulties finding a good solution to my current issue. In the project I work on (cucumber-ruby), I'm getting Rubocop errors on some string formatting. We have enforced annotated style for string formatting:

Style/FormatStringToken:
  EnforcedStyle: annotated

but we also use formatting to specify number of decimals to keep in stone functions, eg:

def format_duration(seconds)
  m, s = seconds.divmod(60)
  "#{m}m#{format('%.3f', s)}s"
end

And of course, rubocop complains that this is not annotated style:

Style/FormatStringToken: Prefer annotated tokens (like %<foo>s) over unannotated tokens (like %s).
        @io.print format_string(format('%.7f', stepdef_key.mean_duration), :skipped) + ' ' unless config.dry_run?

How to solve this kind of problem ? (hopefully I'm on the right channel :) )

Vincent

@vincent-psarga

ok, well after bit more research, that was pretty simple in fact: format('%<duration>.7f', duration: stepdef_key.mean_duration). I did not know the numbers formatting worked as the string ones :)

CJ Avilla

@w1zeman1p

Anyone happen to have a link handy for where I might learn about how to initialize and run rubocop directly in Ruby? I'm trying to build something to check snippets of ruby in some documentation so the source file is a view template not a ruby file and contains other languages. I want to pass a string in Ruby to some actor and have it return cops or some such.

For javascript, I'm doing something like

Eslintrb.lint(raw_code, :eslintrc)

I'll mostly be checking for Style cops

seems like SO suggestions are to write to file, then just use open3 https://stackoverflow.com/questions/58779791/programmatically-integrate-rubocop-formatter-with-stdin-stdout

halo

@halo

Hi! I have a question.

I have a rubocop.yml on Github, which I inherit_from in all my projects. Some projects are pure Ruby, some Rails, and some use RSpec.

It is fine to require: rubocop-rails in a Rails project, naturally.

But in a pure Ruby project I would not want to demand that the rubocop-rails gem is installed (neither should other developers have to install that gem, nor is it needed on CI).

So, how would I organize my shared rubocop config?

I cannot just specify any Rails/ cop in it, because that raises an error here

Similarly, the RSpec cops should only run in projects that use RSpec.

My idea is to have a rubocop.ruby.yml and a rubocop.rails.yml etc. on Github and inherit only the ones I need in my project.

Is that the recommended solution? I thought it was nice to have only one yml file with all cops, but I don't see how I could achieve that without the disadvantages I tried to describe.

Thank you very much.

Max Schwenk

@maschwenk

curious why ruby-parse considers a block that has multiple lines to be nested within a begin. I know this isn't a rubocop question, just trying to figure out how one would generically say:

this method call within a block of this type

is it easier to do that or

this block with this method call within

create_table :a_table do |z|
  z.goober :name
  z.string :col
end

create_table :a_table do |z|
  z.goober :name
end

parsed

(begin
  (block
    (send nil :create_table
      (sym :a_table))
    (args
      (procarg0
        (arg :z)))
    (begin
      (send
        (lvar :z) :goober
        (sym :name))
      (send
        (lvar :z) :string
        (sym :col))))
  (block
    (send nil :create_table
      (sym :a_table))
    (args
      (procarg0
        (arg :z)))
    (send
      (lvar :z) :goober
      (sym :name))))

Where communities thrive

People

Repo info

Activity