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

Update CircleCI Integration #115

Closed
wants to merge 1 commit into from
Closed

Conversation

jbcurtin
Copy link
Contributor

@jbcurtin jbcurtin commented Sep 27, 2019

Updated the Circle CI Integration to compile each notebook individually before building all the pages. Failures are reported when the notebook cannot be built for some reason.

With this update, CircleCI was also update for spacetelescope/notebooks
Screen Shot 2019-09-26 at 11 44 43 AM
Screen Shot 2019-09-26 at 11 44 39 AM

Builds will only happen on master, every PR branch, and every forked PR branch

Here is a view of the CircleCI with reporting enabled
Screen Shot 2019-09-27 at 10 00 55 AM

@jbcurtin jbcurtin requested a review from eteq as a code owner September 27, 2019 13:56
@pllim
Copy link
Contributor

pllim commented Sep 27, 2019

Commit history is not quite right. It seems to contain unrelated commits.

…r each notebook. Also added TestCase handling that'll report when errors are found with the pip install -r requirements.txt.
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.

3 participants