Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

Get CI Green #714

Closed
wants to merge 2 commits into from
Closed

Get CI Green #714

wants to merge 2 commits into from

Conversation

daviddias
Copy link
Contributor

@ghost ghost assigned daviddias Mar 14, 2018
@ghost ghost added the in progress label Mar 14, 2018
@daviddias
Copy link
Contributor Author

@vmx let's see what CI tells us now :)

@daviddias
Copy link
Contributor Author

Some new test fails, but browser tests do run! :)

image

@daviddias daviddias removed their assignment Mar 14, 2018
@daviddias
Copy link
Contributor Author

@vmx can you land the rest?

@vmx
Copy link
Contributor

vmx commented Mar 14, 2018

@diasdavid That one needs ipfs-inactive/interface-js-ipfs-core#227 (comment). Either reverting the commit or fixing the tests.

I currently work on a branch with that one reverted trying to fix the other sporadic failures.

@ghost ghost assigned vmx Mar 14, 2018
@vmx vmx closed this Mar 14, 2018
@vmx vmx deleted the get-ci-green branch March 14, 2018 14:53
@ghost ghost removed the in progress label Mar 14, 2018
@vmx vmx restored the get-ci-green branch March 14, 2018 14:53
@daviddias daviddias reopened this Mar 14, 2018
@ghost ghost assigned daviddias Mar 14, 2018
@ghost ghost added the in progress label Mar 14, 2018
@vmx
Copy link
Contributor

vmx commented Mar 14, 2018

One of the recent merges broke the MFS files tests. Reverting ipfs-inactive/interface-js-ipfs-core#227 isn't enough anymore.

Update: Reverting b08f21a as well is they key. I'll go from there.

@vmx vmx closed this Mar 14, 2018
@vmx vmx deleted the get-ci-green branch March 14, 2018 15:14
@ghost ghost removed the in progress label Mar 14, 2018
@vmx vmx restored the get-ci-green branch March 14, 2018 15:15
@daviddias daviddias deleted the get-ci-green branch March 14, 2018 17:01
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.

2 participants