Skip to content
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

Setup linting #643

Closed
tegandbiscuits opened this issue Oct 8, 2020 · 3 comments · Fixed by #644
Closed

Setup linting #643

tegandbiscuits opened this issue Oct 8, 2020 · 3 comments · Fixed by #644

Comments

@tegandbiscuits
Copy link
Contributor

I noticed that it doesn't look like there's any linting inside the repo (correct me if I'm wrong), so I was wondering if I'd be able to setup rubocop (and also rubocop-rspec and rubocop-rails) then resolve any issues it finds?

I pulled this down and ran rubocop locally, and it looks like it's mostly just style changes and a few things to help maintenance.

@DeeDeeG
Copy link
Contributor

DeeDeeG commented Oct 8, 2020

Hi @tegandbiscuits, this was mentioned by another contributor just recently this week. So, it's starting to sound like something we should do.

Thanks for opening the issue.

Note to other maintainers: @tkwidmer @mi-wood if you have any thoughts or insights about this, objections, whatever, feel free to let me know. Otherwise I am going to look into just adding this myself.

@tegandbiscuits
Copy link
Contributor Author

If it'd be helpful, I could make a pull request for it. I've already got it mostly done in a fork and could make a pull request with it tonight (or a draft PR now).

@DeeDeeG
Copy link
Contributor

DeeDeeG commented Oct 8, 2020

That would be fine.

I took a look at the fork you mentioned, and fair warning: I can say this is a bit beyond my ability to meaningfully review. I would be hoping for the tests to pass, but beyond that, the changes are a bit above my level of Ruby expertise (which as you might guess isn't very much!). I'll have to ask some of the other maintainers to take a look at this in order to review it properly. And they may not be able to stop and look at it right away.

If you're okay with that, then a PR would be quite welcome. Thanks a bunch for working on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants