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

Fix CI #15393

Merged
merged 2 commits into from
Apr 11, 2019
Merged

Fix CI #15393

merged 2 commits into from
Apr 11, 2019

Conversation

acdlite
Copy link
Collaborator

@acdlite acdlite commented Apr 11, 2019

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
Copy link

sizebot commented Apr 11, 2019

Warnings
⚠️

Base commit is broken: 687e4fb

Generated by 🚫 dangerJS

@acdlite
Copy link
Collaborator Author

acdlite commented Apr 11, 2019

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
Copy link
Collaborator Author

acdlite commented Apr 11, 2019

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
Copy link
Collaborator Author

acdlite commented Apr 11, 2019

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

@acdlite acdlite merged commit de75903 into facebook: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