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

[Tests](fix): Fix flaky test timeout #111

Closed
wants to merge 1 commit into from

Conversation

alexandre-merle
Copy link
Contributor

  • Increase timeout to remove flacky test from
    public Circle CI

Fixes #93

@rahulreddy
Copy link
Collaborator

👍

@rahulreddy
Copy link
Collaborator

Can you fix the commit message typo, from flacky to flaky 😄

@rahulreddy
Copy link
Collaborator

I just realized, this should probably go into rel/1.1 since it's not related to just Smart Bulb release

- Increase timeout to remove flaky test from
  public Circle CI
@alexandre-merle
Copy link
Contributor Author

Done :)

@alexandre-merle alexandre-merle changed the title [Tests](fix): Fix flacky test timeout [Tests](fix): Fix flaky test timeout Jul 14, 2016
@alexandre-merle
Copy link
Contributor Author

you right :) i will do that too

@ghost
Copy link

ghost commented Jul 14, 2016

👍 :) Failed too many times for nothing :D

@ghost
Copy link

ghost commented Jul 14, 2016

Note: This one should be closed in favor of #113

@alexandre-merle
Copy link
Contributor Author

@DavidPineauScality this one is for master :)

@ghost
Copy link

ghost commented Jul 15, 2016

Yes, and that's what forward ports are for. To port fixes pushed into release branches into the associated master branches.

@alexandre-merle
Copy link
Contributor Author

Closing for #114 :)

@ghost ghost deleted the fix/flacky-test branch July 18, 2016 09:29
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.

2 participants