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

Fixes #2293.#2535

Closed
nivthefox wants to merge 1 commit into
angular-ui:masterfrom
nivthefox:master
Closed

Fixes #2293.#2535
nivthefox wants to merge 1 commit into
angular-ui:masterfrom
nivthefox:master

Conversation

@nivthefox

Copy link
Copy Markdown

Confirmed fixed with 1.3.0-beta.17.

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

23tux commented Aug 5, 2014

Copy link
Copy Markdown

What are the chances, that this PR is merged?

@miketierney

Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown

any change this can get merged in?

@Siyfion

Siyfion commented Aug 13, 2014

Copy link
Copy Markdown

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

@futurechan

Copy link
Copy Markdown

+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