Skip to content

Conversation

@delvedor
Copy link
Member

@delvedor delvedor commented Aug 9, 2019

  • Move from matrix jobs to multibranch/pipeline
  • Remove old job definition
  • Brand new Jenkinsfile

Thanks to @v1v @kuisathaverat @mdelapenya for all the help!

@delvedor delvedor added the tests label Aug 9, 2019
.ci/Jenkinsfile Outdated
script {
buildDockerImage(image: "node:${env.NODE_JS_DEFAULT_VERSION}-alpine").inside(){
dir("${BASE_DIR}"){
sh(label: 'System info', script: '''node --version
Copy link
Member

Choose a reason for hiding this comment

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

Cosmetic change: you can use a single line with ; between both commands if you don't like the multine

ARG NODE_JS_VERSION=10
FROM node:${NODE_JS_VERSION}-alpine

RUN apk --no-cache add git
Copy link
Member

Choose a reason for hiding this comment

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

❤️

@delvedor delvedor merged commit 34e440b into master Aug 9, 2019
@delvedor delvedor deleted the jenkins-pipelines branch August 9, 2019 10:11
delvedor added a commit that referenced this pull request Aug 9, 2019
delvedor added a commit that referenced this pull request Aug 9, 2019
delvedor added a commit that referenced this pull request Aug 9, 2019
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.

3 participants