-
Notifications
You must be signed in to change notification settings - Fork 908
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
Update circleci ci-setup.md #3989
Conversation
The syntax of the circleci config.yml was out of date.
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
Hey @HennaAbbas thanks for this. Would you mind fixing the failing linting check? I can also help with that. |
How can a change to an |
I guess prettier isn't happy with it. Will fix it. |
Locally it's not failing, yay?!... Let's re-trigger |
Still failing, I don't understand why |
The prettier step needs better output, maybe print in CI log what |
Looks like it was some sort of caching issue locally, weird. Thanks @HennaAbbas ! |
The syntax of the circleci config.yml was out of date.
Description
Motivation and Context
Usage examples
How Has This Been Tested?
Types of changes
Checklist: