Skip to content

ci: run docurium to create documentation - #4961

Merged
ethomson merged 1 commit into
masterfrom
ethomson/ci_docurium
Jan 27, 2019
Merged

ci: run docurium to create documentation#4961
ethomson merged 1 commit into
masterfrom
ethomson/ci_docurium

Conversation

@ethomson

Copy link
Copy Markdown
Member

Run docurium as part of the build. The goal of this is to be able to evaluate the documentation in a given pull request; as such, this does not implement any sort of deployment pipeline.

This will allow us to download a snapshot of the documentation from the CI build and evaluate the docs for a particular pull request; before it's been merged.

Run docurium as part of the build.  The goal of this is to be able to
evaluate the documentation in a given pull request; as such, this does
not implement any sort of deployment pipeline.

This will allow us to download a snapshot of the documentation from the
CI build and evaluate the docs for a particular pull request; before
it's been merged.
@ethomson
ethomson force-pushed the ethomson/ci_docurium branch from 770fcf6 to ace20c6 Compare January 26, 2019 17:03
@ethomson

Copy link
Copy Markdown
Member Author

The goal here is so that I can get a snapshot of the docs to study the docs for a particular pull request. You can either grab these using the REST API or from the Azure Pipelines website:

screen shot 2019-01-26 at 17 06 19

I'd love to do something like what docs.microsoft.com does on pull requests, where it spins up a website with the changes for you to review and tears that back down when the PR is merged or abandoned. But that's not in scope here.

We'll reevaluate how to automatically generate the gh-pages branch that actually lands on libgit2.org, whether that's using GitHub Actions or a release out of the CI build.

@ethomson
ethomson merged commit e9a3486 into master Jan 27, 2019
@ethomson
ethomson deleted the ethomson/ci_docurium branch January 27, 2019 22:47
@pks-t

pks-t commented Feb 7, 2019

Copy link
Copy Markdown
Member

This is really cool. Thanks for working on it, @ethomson!

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