Skip to content

Conversation

benoitvidis
Copy link
Contributor

What does this PR do?

Adds automatic builds to documentation.
Builds will be deployed to https://docs.kuzzle.io when merged to master and to https://nex-docs.kuzzle.io on 3-dev.

Thanks to node 10 and possibly the upgrade of vuepress, the build can be generated in a relatively acceptable time (~35mn, i.e. https://travis-ci.org/kuzzleio/documentation/jobs/541275090).

Other changes

  • upgrade vuepress to 1.0.0-beta.1

@Aschen
Copy link
Contributor

Aschen commented Jun 5, 2019

@benoitvidis We can even try node 11 or 12 no?

@benoitvidis benoitvidis added the wip label Jun 5, 2019
Copy link
Contributor

@xbill82 xbill82 left a comment

Choose a reason for hiding this comment

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

Looks OK to me but, why did you take away all the test reports?

"yaml": "^1.5.1"
},
"devDependencies": {
"colors": "^1.3.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a dependency for the current snippet test stack.
I don't know how this happened but it was previously registered in the package-lock.json but not in the package.json...

@benoitvidis
Copy link
Contributor Author

@xbill82 about the tests reports: They were not working anyway (cf for instance: #299).
I could have tried to fix them but this feature adds quite some complexity while its benefits is only to avoid one click to jump to the travis report page.

@benoitvidis
Copy link
Contributor Author

@Aschen , node 12 cannot be used as node-sass is not compatible with it yet. I tried node 11 and it does not change anything to the process time so I chose to stick to a LTS version.

@benoitvidis benoitvidis removed the wip label Jun 6, 2019
@scottinet scottinet merged commit 9f81292 into 3-dev Jun 6, 2019
@scottinet scottinet deleted the kzl-1154-ci-build branch June 6, 2019 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants