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

Remove the check, and just always register with CircleCI. #147

Merged
merged 1 commit into from
Apr 28, 2016

Conversation

pelson
Copy link
Member

@pelson pelson commented Apr 28, 2016

I got sick of the unpredictable response, and wouldn't be surprised if it was because we have too many repos for the API to handle. This PR removes the check before asking to follow a repo, and just always follows the repo instead (same effect, less data transferred).

@pelson pelson merged commit 42750c9 into conda-forge:master Apr 28, 2016
@pelson pelson deleted the circle_check branch April 28, 2016 11:41
@jakirkham
Copy link
Member

Weird that this would matter. Though the change seems totally reasonable and shouldn't have any real difference in the net effect.

@jakirkham
Copy link
Member

Thanks for looking into this, @pelson, and coming up with a good fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants