Skip to content

Fix version number to burn correctly under Ubuntu#2525

Merged
corinagum merged 3 commits intomasterfrom
test-version
Oct 31, 2019
Merged

Fix version number to burn correctly under Ubuntu#2525
corinagum merged 3 commits intomasterfrom
test-version

Conversation

@compulim
Copy link
Copy Markdown
Contributor

Fixes #2524.

Changelog Entry

Description

Version number burnt into the source code was undefined when building under Ubuntu, but works on Windows.

Specific Changes

  • Change all instances of NPM_PACKAGE_VERSION to npm_package_version

  • Testing Added
    • Manual tests are sufficient in release checklist

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 63.345% when pulling d62e511 on test-version into 11332a8 on master.

@compulim compulim mentioned this pull request Oct 31, 2019
55 tasks
@compulim
Copy link
Copy Markdown
Contributor Author

Please ignore the build break for "[Experimental]" pipeline.

@corinagum corinagum merged commit dd80827 into master Oct 31, 2019
@compulim compulim deleted the test-version branch October 31, 2019 16:25
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.

Output build should have correct version number burnt

3 participants