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

Kzl 1154 ci build #320

Merged
merged 7 commits into from
Jun 6, 2019
Merged

Kzl 1154 ci build #320

merged 7 commits into from
Jun 6, 2019

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?

"vuepress-plugin-container": "^2.0.1",
"vuepress-validate-frontmatter": "^2.0.10",
"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