-
Notifications
You must be signed in to change notification settings - Fork 45
Add support for standardrb #351
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
It should be possible to just include the standard gem in all regular channels that have compatible rubocop versions, then let people's rubocop config decide whether or not to use it. |
So actually @cquinones100 we do have a dedicated To enable standard for your codebase, simply add the following to the plugins section of your
I'm really curious how this will work out for you. Please give it a shot and let us know if that solves your issue, otherwise happy to look into that some more 😄 Also so sorry for not responding sooner, will try and keep an eye out more in the future. |
Hi @RubyBrewsday thanks for the response! We don't use standard directly, instead we use it via rubocop[https://github.com/testdouble/standard#usage-via-rubocop], so I don't think we need the standard plugin-- instead i think we need a rubocop plugin channel that includes standard as @BrianHawley mentioned. Additional context: we use rubocop version 1.39.0 |
Ah gotcha, thanks for the extra context @cquinones100, makes sense to me. Yeah I think we should be able to make that happen for you. |
@cquinones100 Hey there, I hope you had a good weekend. Just wanted to give a heads up that I've tested out a fix and we should be pushing that up later on today. I'll let you know when that is ready to go but you should be able to use standard if you set your rubocop channel to |
@cquinones100 Okie doke, those changes should have gone out yesterday afternoon. If you could try re-running analysis on your repo and let us know how that works out for you that would be awesome. In case you might not know how to manually refresh your repo, check out this support article and that should show you the easiest way too (besides pushing up new code 😄): https://docs.codeclimate.com/docs/branches-and-comparisons#section-what-triggers-a-branch-analysis |
@RubyBrewsday you should also make a PR for channel/beta. |
- General bundle update. - Update the docs too. - Add standard gem. Note this limits the rubocop-performance version. Fixes codeclimate#351.
- General bundle update. - Update the docs too. - Add standard gem. Note this limits the rubocop-performance version. Fixes codeclimate#351.
- General bundle update. - Update the docs too. - Add standard gem. Note this limits the rubocop-performance version. Fixes codeclimate#351.
- General bundle update. - Update the docs too. - Add standard gem. Note this limits the rubocop-performance version. Fixes #351.
It turns out that the standard gems are not released at the same pace as the corresponding rubocop gems that they reference. This means that referencing the standard gems at all in this engine prevents us from updating the other rubocop gems when new versions come out. This is currently preventing us from updating rubocop to 1.51.0, for instance. It might be a good idea to have a separate set of channels for rubocop with the standard gems included, so the regular rubocop channels can keep up to date. Alternatively, it might be a good idea to have the standard engine be able to handle the case of people running standard through rubocop instead of through its own tools, so the regular rubocop engine doesn't have to reference the standard gems. |
We use standard along with rubocop in our project. We use the
inherit_gem
config in.rubocop.yml
to add on to standard's config. We also use therequire
config to require standard along with other gems we use (rubocop-performance
,rubocop-rails
, etc).Can a channel be made that requires
standard
?The text was updated successfully, but these errors were encountered: