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

[ci] Let failed types-next jobs pass #20007

Merged
merged 1 commit into from
Mar 6, 2020
Merged

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Mar 6, 2020

Follow-up on #19857

While it's ok for @typescript@next to fail ($ExpectError position changes) it's not ok for github to display the repo state as failing:

Screenshot from 2020-03-06 02-03-06

So we tell CircleCI to let failures in these jobs not affect the job status.

@mui-pr-bot
Copy link

mui-pr-bot commented Mar 6, 2020

No bundle size changes comparing dec3870...b9fc5c3

Generated by 🚫 dangerJS against b9fc5c3

@eps1lon eps1lon force-pushed the ci/green-ts-next branch from ac6c3bf to b9fc5c3 Compare March 6, 2020 01:16
@@ -22,6 +22,6 @@ or:

## The idea behind the example

The is an alternative example to [`/examples/gatsby`](https://github.com/mui-org/material-ui/tree/master/examples/gatsby) leveraging [gatsby-theme-material-ui](https://github.com/hupe1980/gatsby-theme-material-ui/tree/master/packages/gatsby-theme-material-ui).
This is an alternative example to [`/examples/gatsby`](https://github.com/mui-org/material-ui/tree/master/examples/gatsby) leveraging [gatsby-theme-material-ui](https://github.com/hupe1980/gatsby-theme-material-ui/tree/master/packages/gatsby-theme-material-ui).
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

github is weird these days

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

already on master daa0eff

@eps1lon eps1lon merged commit 91fa13b into mui:master Mar 6, 2020
@eps1lon eps1lon deleted the ci/green-ts-next branch March 6, 2020 01:30
EsoterikStare pushed a commit to EsoterikStare/material-ui that referenced this pull request Mar 30, 2020
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