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

Test that the docs build #1453

Merged
merged 1 commit into from
Aug 16, 2018
Merged

Conversation

peterbe
Copy link
Contributor

@peterbe peterbe commented Aug 16, 2018

Fixes #1441

These steps work locally but I'm going to see if messing with one of the .rst files actually breaks the build.

  • Test that it works
  • Test that .rst failures "break the build"
  • Squash commits to clean up

@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

Nice!
screen shot 2018-08-16 at 10 29 48 am

@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

And the caching worked for the docs part.

screen shot 2018-08-16 at 10 31 31 am

@peterbe peterbe requested a review from mythmon August 16, 2018 14:33
@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

Weird that I didn't get this locally!

Warning, treated as error:
/home/circleci/repo/normandy/recipes/filters.py:docstring of normandy.recipes.filters:1:Inline literal start-string without end-string.
Makefile:55: recipe for target 'html' failed
make: *** [html] Error 2
Exited with code 2

UPDATE Nevermind. I did get this error locally.

Copy link
Contributor

@mythmon mythmon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Can you add ci/circleci: docs as a required check to the bors config?

@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

Done

mv ~/docker-compose /usr/local/bin/docker-compose
curl -L https://github.com/docker/compose/releases/download/1.19.0/docker-compose-`uname -s`-`uname -m` > ~/docker-compose
chmod +x ~/docker-compose
mv ~/docker-compose /usr/local/bin/docker-compose
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By the way, some Yaml plugin in vscode did this. I'm a bit surprised it was enabled to "formatOnSave" by default. Either way, can't hurt.

@mythmon
Copy link
Contributor

mythmon commented Aug 16, 2018

Looks good. I'll let you handle bors yourself, since it looks like wanted to squash all this.

bors delegate= @peterbe

@bors
Copy link
Contributor

bors bot commented Aug 16, 2018

✌️ peterbe can now approve this pull request

@peterbe peterbe force-pushed the 1441-test-that-the-docs-build branch from e004f02 to 080898f Compare August 16, 2018 19:01
@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

uh? What do I do? Just bors r+?

@peterbe
Copy link
Contributor Author

peterbe commented Aug 16, 2018

bors r+

bors bot added a commit that referenced this pull request Aug 16, 2018
1453: Test that the docs build r=peterbe a=peterbe

Fixes #1441

These steps work locally but I'm going to see if messing with one of the `.rst` files actually breaks the build. 

- [x] Test that it works
- [x] Test that `.rst` failures "break the build"
- [x] Squash commits to clean up

Co-authored-by: Peter Bengtsson <mail@peterbe.com>
@bors
Copy link
Contributor

bors bot commented Aug 16, 2018

Build succeeded

@bors bors bot merged commit 080898f into mozilla:master Aug 16, 2018
@peterbe peterbe deleted the 1441-test-that-the-docs-build branch August 16, 2018 20:00
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