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

Added Documentation Building to Jenkins #80

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Valenzione
Copy link
Contributor

Hi @KineticCookie, I've added missing dependencies required for building documentation and sketched a groovy function that builds and pushes built documentation to the repo. I'm not sure when I should call it or how to integrate it with a current pipelineCommon. Would you mind giving me a helping hand here?

@KineticCookie
Copy link
Member

Hey there!
If your script works fine, you can plug it into different functions depending on when do you want the docs to update:

  1. on every push to master - then insert your function call into the https://github.com/Hydrospheredata/hydro-serving-sdk/blob/master/jenkinsfile.groovy#L17
  2. on release - insert your function call into the https://github.com/Hydrospheredata/hydro-serving-sdk/blob/master/jenkinsfile.groovy#L3

@Valenzione Valenzione marked this pull request as ready for review September 18, 2020 07:59
@Valenzione
Copy link
Contributor Author

@KineticCookie Thanks, I've added it to the release step. Can you check it? It'd be great to merge this before 2.4.0 stable release

Copy link
Member

@KineticCookie KineticCookie left a comment

Choose a reason for hiding this comment

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

👍

@KineticCookie KineticCookie marked this pull request as draft June 18, 2021 09:24
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