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

MRG: Add circle-CI #6192

Merged
merged 1 commit into from
May 29, 2016
Merged

MRG: Add circle-CI #6192

merged 1 commit into from
May 29, 2016

Conversation

larsoner
Copy link
Member

Add CircleCI doc building, with an artifact of the new docs. This should hopefully make it easier for e.g. @rgommers to critique my mediocre docstring plots.

Closes #6129.

Produces this:

https://circle-artifacts.com/gh/Eric89GXL/scipy/27/artifacts/0/home/ubuntu/scipy/doc/build/html-scipyorg/generated/scipy.signal.remez.html#scipy.signal.remez

@rgommers
Copy link
Member

That looks good, thanks Eric!

1500 build minutes a month may be borderline, but we'll see how far we get with a free account I guess.

@rgommers rgommers added the enhancement A new feature or improvement label May 27, 2016
@larsoner
Copy link
Member Author

I've been using it on another repo for a couple of months with fewer PRs/commits but probably longer build times and so far we've been okay, so I'm optimistic. I'll go ahead and merge and activate in a couple of days unless there are other comments.

@rgommers
Copy link
Member

For reference, here is the TravisCI artifacts support: https://docs.travis-ci.com/user/uploading-artifacts/. Summary: it doesn't store artifacts, only supports uploading to Amazon S3. Hence CircleCI is better in this regard.

We currently only build the docs after a PR is merged (to update the dev docs on docs.scipy.org).

@rgommers rgommers merged commit 9bc28e1 into scipy:master May 29, 2016
@rgommers
Copy link
Member

in it goes, thanks @Eric89GXL

@larsoner
Copy link
Member Author

@rgommers can you enable CircleCI on their website? I don't think I have sufficient permissions

@rgommers
Copy link
Member

done, building at https://circleci.com/gh/scipy/scipy/1 now

@rgommers
Copy link
Member

Eh, make that failed at https://circleci.com/gh/scipy/scipy/1

Reading state information... Done

Note, selecting 'libpng12-dev' instead of 'libpng-dev'
libatlas-base-dev is already the newest version.
libatlas-dev is already the newest version.
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 liblapack-dev : Depends: liblapack3 (= 3.5.0-2ubuntu1) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

sudo apt-get install libatlas-dev libatlas-base-dev liblapack-dev gfortran libgmp-dev libmpfr-dev libfreetype6-dev libpng-dev zlib1g-dev texlive-fonts-recommended returned exit code 100

Action failed: sudo apt-get install libatlas-dev libatlas-base-dev liblapack-dev gfortran libgmp-dev libmpfr-dev libfreetype6-dev libpng-dev zlib1g-dev texlive-fonts-recommended

@larsoner
Copy link
Member Author

larsoner commented May 29, 2016 via email

@larsoner
Copy link
Member Author

Looks like I had permission to change it, so I did. Next build will hopefully work.

@larsoner
Copy link
Member Author

I rebased and push forced #5989 so hopefully we see it work there

@larsoner larsoner deleted the circle branch May 29, 2016 13:38
@rgommers
Copy link
Member

Worked like a charm this time.

@pv
Copy link
Member

pv commented May 31, 2016 via email

@pv
Copy link
Member

pv commented May 31, 2016 via email

@larsoner
Copy link
Member Author

larsoner commented May 31, 2016 via email

@ev-br ev-br added this to the 0.18.0 milestone Jun 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants