Skip to content

[WIP] Various fixes#1323

Open
maclover7 wants to merge 3 commits into
ruby-grape:masterfrom
maclover7:various-fixes
Open

[WIP] Various fixes#1323
maclover7 wants to merge 3 commits into
ruby-grape:masterfrom
maclover7:various-fixes

Conversation

@maclover7
Copy link
Copy Markdown

cc @dblock

@maclover7
Copy link
Copy Markdown
Author

While this is open, is it worth it to remove support for Ruby 1.9.3? It has already EOL'd, and isn't included in the Travis build matrix...

source "https://rubygems.org"

gem 'rack', '1.5.2'
gem "rack", "1.5.2"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this back to 1.8.x syntax?

@dblock
Copy link
Copy Markdown
Member

dblock commented Mar 15, 2016

Anything not in the Travis matrix is not supported, so yes.

@dblock
Copy link
Copy Markdown
Member

dblock commented Mar 16, 2016

Add a comment here when the build passes, don't get notifications on additional commits.

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 1, 2016

Bump @maclover7, want to finish this? Part of this?

@maclover7
Copy link
Copy Markdown
Author

This main holding issue is my rack-contrib PR 😬

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.

2 participants