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

Updates to work with Sinatra v2 #12

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

Updates to work with Sinatra v2 #12

wants to merge 9 commits into from

Conversation

yb66
Copy link

@yb66 yb66 commented Jan 13, 2018

Hi Konstantin,

This may or may not update the lib to work with v2 as I've not really acquainted myself with its changes, but the specs pass :) Let me know if there's a problem.

Regards,
iain

@yb66 yb66 changed the title Develop Updates to work with Sinatra v2 Jan 14, 2018
@pixeline
Copy link

@rkh could you kindly have a look at this PR and see if it solve the v2 compatibility issue?

@rkh
Copy link
Owner

rkh commented Nov 30, 2022

Changes look good, but I haven't touched this project in years and don't have anything running that I could try it out with.

@yb66
Copy link
Author

yb66 commented Dec 1, 2022

I forgot I'd done this! Maybe @kamaradclimber from issue #11 might have something to test against?

Regards,
iain

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.

None yet

3 participants