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

Make version.ts use a constant and create the file during the build process. (#818) #824

Merged
merged 1 commit into from
Jun 8, 2022

Conversation

dpvc
Copy link
Member

@dpvc dpvc commented Jun 2, 2022

This PR resolves the build issue that was reported with v3.2.1 due to the dynamic lookup of the version number by creating the version.ts file via components/bin/version, which is run during the travis-ci publish process (and can be run by hand at any point).

Resolve issue #818 and mathjax/MathJax#2880

@dpvc dpvc requested a review from zorkow June 2, 2022 20:29
@dpvc dpvc added this to the 3.2.2 milestone Jun 2, 2022
Copy link
Member

@zorkow zorkow left a comment

Choose a reason for hiding this comment

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

lgtm.

Since I've just seen the line: Do we know if travis works with stable again?

@dpvc
Copy link
Member Author

dpvc commented Jun 8, 2022

Do we know if travis works with stable again?

Last time I found out by the fact that the publish run failed, and I tracked down the error message. Could always try that again. I will see if I can find something indicating the current state of affairs.

@dpvc
Copy link
Member Author

dpvc commented Jun 8, 2022

The default OS used by Travis-CI is ubuntu 16.04, unless you request otherwise. It looks like node.js v18 changed the library version they use, and that ubuntu 16.04 doesn't have it, nor does 18.04; you need 18.10 or 20.04 to allow it to work. Travis-CI does have a 20.04 option, so I will switch to that and see if that allows the use of the stable version of node.js again.

@dpvc dpvc merged commit 78e7754 into develop Jun 8, 2022
@dpvc dpvc deleted the issue818 branch June 8, 2022 16:01
@dpvc
Copy link
Member Author

dpvc commented Jun 8, 2022

It looks like the update to 20.04 did the trick. Back to stable version of node.

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.

None yet

2 participants