Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Fixes #2293. #2535

Closed
wants to merge 1 commit into from
Closed

Fixes #2293. #2535

wants to merge 1 commit into from

Conversation

nivthefox
Copy link

Confirmed fixed with 1.3.0-beta.17.

Confirmed fixed with 1.3.0-beta.17.
@23tux
Copy link

23tux commented Aug 5, 2014

What are the chances, that this PR is merged?

@miketierney
Copy link

@kkragenbrink - Any news on whether or not the Travis build can be fixed so this PR can be merged?

I've confirmed that the fix works (manually making the change myself in a local copy of the plugin), but admittedly I don't know enough about the test suite or the plugin to feel confident about getting it to run green.

@nivthefox
Copy link
Author

I've never used Travis CI before, nor this test harness, but I will try and figure it out this weekend and get it cleaned up.

@davisford
Copy link

any change this can get merged in?

@Siyfion
Copy link

Siyfion commented Aug 13, 2014

@davisford This won't get merged as it doesn't pass the build tests, but #2557 does.

@futurechan
Copy link

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants