Skip to content

Fix CI#15393

Merged
acdlite merged 2 commits into
react:masterfrom
acdlite:fix-ci
Apr 11, 2019
Merged

Fix CI#15393
acdlite merged 2 commits into
react:masterfrom
acdlite:fix-ci

Conversation

@acdlite

@acdlite acdlite commented Apr 11, 2019

Copy link
Copy Markdown
Collaborator

CI is currently failing. I think it's because I pushed a commit directly to master without first opening a PR. Generally we shouldn't do this, but sometimes it makes sense, like if you want to merge a branch without squashing the commits.

I'll use this branch to try out some strategies for fixing it.

@acdlite acdlite marked this pull request as ready for review April 11, 2019 23:28
@sizebot

sizebot commented Apr 11, 2019

Copy link
Copy Markdown
Warnings
⚠️

Base commit is broken: 687e4fb

Generated by 🚫 dangerJS

@acdlite

acdlite commented Apr 11, 2019

Copy link
Copy Markdown
Collaborator Author

Ok this is what I would expect to see:

Screen Shot 2019-04-11 at 4 31 37 PM

It can't find the build artifact for results.json for the base commit because until this PR they weren't being stored. It should work for subsequent PRs.

@acdlite

acdlite commented Apr 11, 2019

Copy link
Copy Markdown
Collaborator Author

Screen Shot 2019-04-11 at 4 40 41 PM

It fails because the base commit is broken... I suppose I should make that a warn instead :D

@acdlite

acdlite commented Apr 11, 2019

Copy link
Copy Markdown
Collaborator Author

Ok I think this will fix master. I'll merge and see.

@acdlite acdlite merged commit de75903 into react:master Apr 11, 2019
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