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

Add Rubocop configuration #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jordanmaguire
Copy link
Contributor

@jordanmaguire jordanmaguire commented Jul 27, 2020

Hi everybody

Now that we have a second Rails app on the horizon, I'm taking some time up front to share our configuration where possible, so both apps have access to all the cool stuff we've built.

In this case, I'm starting small with just sharing the Rubocop config.

Here's what App integration looks like: https://github.com/interexchange/app/pull/4486

The downside to sharing configs is that now we have to start maintaining our configuration repos as well as App. Since I've chosen to make Rubocop a dependency of this gem, we will need to upgrade it here.

@@ -0,0 +1,21 @@
MIT License

Copyright (c) 2020 InterExchange.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this fine for a copyright?

@jordanmaguire jordanmaguire force-pushed the initial_configuration branch from bfd41b1 to f548fab Compare July 27, 2020 07:31
@jordanmaguire jordanmaguire force-pushed the initial_configuration branch from d946bf0 to ac2e0ef Compare July 27, 2020 07:34
spec.name = "iex-style"
spec.version = "1.0.0"
spec.authors = ["InterExchange"]
spec.email = ["[email protected]"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We cool with this as an email? I'm sure we can simply omit one as well

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 this pull request may close these issues.

1 participant