Skip to content

KO: Inspiration for optimization#75

Closed
jtimon wants to merge 1 commit into
cdecker:masterfrom
jtimon:travis-50m
Closed

KO: Inspiration for optimization#75
jtimon wants to merge 1 commit into
cdecker:masterfrom
jtimon:travis-50m

Conversation

@jtimon
Copy link
Copy Markdown
Contributor

@jtimon jtimon commented Jul 31, 2019

KO stands for never merge but please Keep Opened even though it is expected to fail forever until closed. This PR passing travis is a bug.
Can be treated as an opened issue with context code attached.
Based on #74 . This should fail.

The prophecy says one day this will be replaced one day by something that attempts the same but actually does it.

The job exceeded the maximum time limit for jobs, and has been terminated.

Note: The expected travis failure can be seen in:

https://github.com/jtimon/lightning-integration/commits/travis-50m

https://github.com/jtimon/lightning-integration/runs/181455697
https://travis-ci.com/jtimon/lightning-integration/builds/121347924

@jtimon
Copy link
Copy Markdown
Contributor Author

jtimon commented Aug 7, 2019

Rebased, but it doesn't seem to be triggering travis. I think you need to activate it for this repo.

@jtimon jtimon mentioned this pull request Aug 9, 2019
@jtimon jtimon mentioned this pull request Aug 29, 2019
1 task
@jtimon
Copy link
Copy Markdown
Contributor Author

jtimon commented Aug 29, 2019

Closed in favor of #77

@jtimon jtimon deleted the travis-50m branch August 29, 2019 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant