Skip to content
This repository was archived by the owner on Jan 29, 2022. It is now read-only.

Allow github to send form-urlencoded payloads and bug fix in webrick_opts#45

Merged
dhollinger merged 3 commits intovoxpupuli:masterfrom
williamberman:master
Feb 12, 2018
Merged

Allow github to send form-urlencoded payloads and bug fix in webrick_opts#45
dhollinger merged 3 commits intovoxpupuli:masterfrom
williamberman:master

Conversation

@williamberman
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@dhollinger dhollinger left a comment

Choose a reason for hiding this comment

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

Looks like a great addition. There are some rubocop failures, could you take a look at those and correct them before we merge this?

Additionally, in the future, it would be easier to have bugfixes and enhancements separated into separate PRs.

Otherwise, looks good to me!

@williamberman
Copy link
Copy Markdown
Contributor Author

Great! Yeah sorry first time contributing to open source. Will get on those.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+2.6%) to 50.746% when pulling 7c1f21d on williamberman:master into 178cf03 on voxpupuli:master.

@voxpupuli voxpupuli deleted a comment Feb 12, 2018
@voxpupuli voxpupuli deleted a comment Feb 12, 2018
@voxpupuli voxpupuli deleted a comment Feb 12, 2018
@voxpupuli voxpupuli deleted a comment Feb 12, 2018
@dhollinger dhollinger merged commit 7741957 into voxpupuli:master Feb 12, 2018
@dhollinger
Copy link
Copy Markdown
Member

Thanks!

Merged!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants