Git Product home page Git Product logo

Comments (8)

pboling avatar pboling commented on May 27, 2024 1

@skryukov The number of files inspected is still changing wildly in unexpected ways. I'll update this with --debug info as well.

from rubocop-gradual.

pboling avatar pboling commented on May 27, 2024

I'm trying to understand why my Gemfile is linted when I run standard RuboCop, but not when I run RuboCop gradual.

In fact when I use NO_GRADUAL=1 I see:

Inspecting 436 files

But when I run rubocop-gradual I see:

Inspecting 81 file(s) for autocorrection...
.................................................................................
Fixed 0 file(s).
....................................................................................................................................................................................................................................................................................................................................................................................................................................................
Found 355 files with 13150 issue(s).

Adding the 81 files found for autocorrection to the 355 files with issues it adds up to the expected 436 that standard RuboCop processes...

Perhaps that makes sense... but if I create offenses in my Gemfile RuboCop can find and fix them, but Rubocop Gradual can't. I'm stumped. RuboCop reports 66 violations in my Gemfile right now that it can't auto-correct, and the rubocop_gradual.lock doesn't have any entries for my Gemfile at all, so I am sure it is not being processed.

from rubocop-gradual.

pboling avatar pboling commented on May 27, 2024

I'm guessing that this is relevant somehow (and also not entirely fixed):
rubocop/rubocop#5251

from rubocop-gradual.

skryukov avatar skryukov commented on May 27, 2024

Hey, sorry for the late response! I made rubocop-gradual -a to run autocorrection only for all new and changed files (according to the lock file, so it might contain old fully-linted files), this makes sense for my application – fixing files that I touched in a current PR. So, my guess is Gemfile wasn't updated when you ran the command 😅

Regarding --lint – it seems to be useful to implement it in rubocop-gradual 🤔

from rubocop-gradual.

pboling avatar pboling commented on May 27, 2024

It's not --lint, but --list. :)

Also, the issue isn't related to file changes. The huge difference between running vanilla rubocop and rubocop_gradual is with zero file changes in the app at all, and a prior clean run (force_update if needed) of rubocop_gradual.

Simply from changing the command I run from rubocop to rubocop_gradual, the files processed, and the rules used during that processing, are totally different.

Also, a separate issue recently started happening. The shim no longer works at all, and I've had to switch to using the rubocop_gradual rake task exclusively.

from rubocop-gradual.

skryukov avatar skryukov commented on May 27, 2024

Simply from changing the command I run from rubocop to rubocop_gradual, the files processed, and the rules used during that processing, are totally different.

Yup, rubocop-gradual changes -a and -A behavior in require mode, that's kinda by design 😅

The change in rules is strange, but I didn't find anything related to that yet 😫

from rubocop-gradual.

pboling avatar pboling commented on May 27, 2024

Oh I forgot to mention the other aspect of this, which is that the total number of files scanned is a huge hint as to the problem here.

Where vanilla rubocop says

Inspecting 436 files

rubocop gradual has this at the end of the run:

Found 355 files with 13150 issue(s).

This shouldn't have anything to do with the state of the file's changes.

Additionally, specifically regarding the Gemfile, it is not listed in my rubocop_gradual.lock at all, while vanilla rubocop finds plenty of violations.

from rubocop-gradual.

pboling avatar pboling commented on May 27, 2024

Yup, rubocop-gradual changes -a and -A behavior in require mode, that's kinda by design 😅

But it should only change which files are modified by RuboCop gradual, not which files are "found", right?

It has to be able to, at minimum, find every file that RuboCop vanilla can find, so that it can generate a hash of the file to track in the .lock. I guess that's my core issue. The .lock has nearly 100 fewer files in it than vanilla rubocop is evaluating.

from rubocop-gradual.

Related Issues (10)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.