-
Notifications
You must be signed in to change notification settings - Fork 149
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
Change CI service #123
Comments
That is strange. We used circle CI at Shopify and I'm sure it runs builds on PRs. Is not any configuration missing? |
Ah, rack also uses circle CI and all PRs have builds for circle CI. rack/rack#1314 |
Sure I have missed some documentation. |
Just an FYI that I'm using this repo as a rails example for the Concourse CI docs. Would mind using it for a Pull Request example as well if the need is there. |
I fixed this issue on Circle CI some months ago and forgot to close it. We don't need to change CIs anymore. |
CircleCI runs the test suite when we merge to
master
, but apparently as of this writing CircleCI does not support checking PRs.Guess we have to switch to Travis? What do you think @yhirano55?
The text was updated successfully, but these errors were encountered: