Skip to content

Conversation

@LeeDr
Copy link

@LeeDr LeeDr commented Jan 26, 2016

No description provided.

@LeeDr
Copy link
Author

LeeDr commented Jan 27, 2016

jenkins, test it

Copy link
Author

Choose a reason for hiding this comment

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

I think I should change this to include (instead of exclude) protocol, hostname, port. But I'm not sure I'm doing this the best way.

@jbudz
Copy link
Contributor

jbudz commented Jan 27, 2016

LGTM

@jbudz jbudz assigned LeeDr and unassigned jbudz Jan 27, 2016
LeeDr pushed a commit that referenced this pull request Jan 27, 2016
Discover shared link tests
This brings test count to 77.
@LeeDr LeeDr merged commit e72cbc2 into elastic:master Jan 27, 2016
LeeDr pushed a commit that referenced this pull request Jan 27, 2016
Discover shared link tests 4x merge from #6010
@LeeDr LeeDr deleted the discoverSharedLinkTests branch January 27, 2016 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants