Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

Use rubygems style guide#2

Merged
doodzik merged 3 commits into
mainfrom
use_rubygems_style_guide
Oct 29, 2021
Merged

Use rubygems style guide#2
doodzik merged 3 commits into
mainfrom
use_rubygems_style_guide

Conversation

@doodzik
Copy link
Copy Markdown

@doodzik doodzik commented Oct 29, 2021

Our goal is to eventually move the work on the plugin into RubyGems. To make this as easy as possible I'm adding the RubyGems rubocop rules to the repository and updating the repo accordingly.

@doodzik doodzik requested a review from a team October 29, 2021 04:46
Copy link
Copy Markdown

@jenshenny jenshenny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, would it be helpful to enable the GH actions in this repo to make sure that this change passes CI?

Copy link
Copy Markdown

@rochlefebvre rochlefebvre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call on picking rubygems' styling. Also, I disabled whitespace change highlighting while reviewing this PR to reduce the clutter:

image

@doodzik
Copy link
Copy Markdown
Author

doodzik commented Oct 29, 2021

would it be helpful to enable the GH actions in this repo to make sure that this change passes CI?

💯 I was thinking of doing that in a separate change.

@doodzik doodzik merged commit cf7fba9 into main Oct 29, 2021
@doodzik doodzik mentioned this pull request Oct 29, 2021
@doodzik doodzik mentioned this pull request Nov 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants