Skip to content
This repository was archived by the owner on Jun 21, 2022. It is now read-only.

Conversation

@escattone
Copy link
Contributor

This PR fixes the Jenkins build/push pipeline for the Kumascript master branch (see https://ci.us-west.moz.works/blue/organizations/jenkins/kumascript_multibranch_pipeline/detail/master/187/pipeline). When running the "build-kumascript" and "push-kumascript" commands within the Kuma-based Makefile, we were using VERSION=latest when we should have been using KS_VERSION=latest. This resulted in running the Kumascript tests (make test VERSION=latest) against an image different from the one just built using make build-kumascript VERSION=latest.

* use KS_VERSION instead of VERSION when running
  "make build-kumascript" and "make push-kumascript"
@escattone escattone requested a review from jwhitlock September 11, 2017 21:56
Copy link
Contributor

@jwhitlock jwhitlock left a comment

Choose a reason for hiding this comment

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

👍 good debugging!

@jwhitlock jwhitlock merged commit a0f0e82 into mdn:master Sep 11, 2017
@escattone escattone deleted the fix-jenkins-1398947 branch September 11, 2017 22:15
jwhitlock added a commit to mdn/kuma that referenced this pull request Sep 18, 2017
* mdn/kumascript#312 - Fix Jenkins image building
* mdn/kumascript#313 - SeeCompatTable: Def. of experimental
* mdn/kumascript#314 - SpecName, spec2: Intersection Observer
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants