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

Gulp: migrate baseline target from make.js #8091

Merged
merged 1 commit into from
Feb 21, 2017

Conversation

timvandermeij
Copy link
Contributor

@timvandermeij timvandermeij commented Feb 21, 2017

Fixes a part of #7062 (which belongs to the ES6 modules project).

Testing steps:

  • Run gulp baseline and verify that an error is thrown since the BASELINE variable is missing.
  • Run BASELINE=bc3bceb gulp baseline and verify that the success message is shown and that the build/basline directory is a Git repository at commit bc3bceb.
  • Run BASELINE=bc3bceb gulp baseline again and verify that it finishes sooner because it only does a fetch (print initializeCommand when in doubt).
  • Run BASELINE=foobar gulp baseline and verify that an error is thrown since foobar is not a valid commit hash.
  • Run BASELINE=bc3bceb node make mozcentralbaseline mozcentraldiff and verify that a diff is created (issue Unable to create mozcentraldiff #7743).

@timvandermeij
Copy link
Contributor Author

Thank you for the review!

@timvandermeij timvandermeij merged commit cada411 into mozilla:master Feb 21, 2017
@timvandermeij timvandermeij deleted the gulp-baseline branch February 21, 2017 23:31
movsb pushed a commit to movsb/pdf.js that referenced this pull request Jul 14, 2018
Gulp: migrate `baseline` target from `make.js`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants