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 dev gems to Gemfile, drop Gemfile.lock from VCS #1045

Merged
merged 12 commits into from Feb 27, 2019
Merged

Conversation

kke
Copy link
Contributor

@kke kke commented Feb 4, 2019

On Ruby 2.6 bundle exec irb fails unless you have irb in the bundle.

The gemfile.lock in the repo seems to cause more problems than benefits.

Added byebug while at it.

@kke kke added the chore label Feb 4, 2019
@kke
Copy link
Contributor Author

kke commented Feb 4, 2019

Rubocop gets updated because of this and causes a lot of failures. Fixing.

@jakolehm jakolehm added this to the 2.3.0 milestone Feb 4, 2019
@jakolehm
Copy link
Contributor

jakolehm commented Feb 7, 2019

Conflicts...

@kke
Copy link
Contributor Author

kke commented Feb 7, 2019

Confict be gone

@jakolehm
Copy link
Contributor

Conflict again..

@kke
Copy link
Contributor Author

kke commented Feb 20, 2019

Conflict away again

@@ -49,4 +49,6 @@ Gem::Specification.new do |spec|
spec.add_development_dependency "rspec", "~> 3.7"
spec.add_development_dependency "fakefs", "~> 0.13"
spec.add_development_dependency "rubocop", "~> 0.57"
spec.add_development_dependency "irb", "~> 1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we switch to pry?

Copy link
Contributor Author

@kke kke Feb 20, 2019

Choose a reason for hiding this comment

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

We can try pry

@kke
Copy link
Contributor Author

kke commented Feb 25, 2019

Back in sync again.

@jakolehm jakolehm merged commit 60dda34 into master Feb 27, 2019
@jakolehm jakolehm deleted the chore/add_dev_gems branch February 27, 2019 09:38
@jakolehm jakolehm mentioned this pull request Mar 1, 2019
@jakolehm jakolehm mentioned this pull request Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants