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

Downgrade nodeJS to v12 #16

Merged
merged 1 commit into from
Sep 6, 2021
Merged

Downgrade nodeJS to v12 #16

merged 1 commit into from
Sep 6, 2021

Conversation

StriderDM
Copy link
Contributor

No description provided.

aviator-app bot pushed a commit to tari-project/tari that referenced this pull request Sep 2, 2021
Description
Made types more explicit for ffi interface and easier to maintain going forwards.
Initialized InterfaceFFI in world rather than WalletFFIClient.
Updated steps in WalletFFI.feature
Updated some const variables in wrapper classes to be let.
Better handling of callback function pointer variables in wallet.
Added additional steps to tests.
Downgrade code to be compatible with nodejs v12.

Motivation and Context


How Has This Been Tested?
nvm use v12.22.6 && ./node_modules/.bin/cucumber-js features/WalletFFI.feature

Run log:
[run.log](https://github.com/tari-project/tari/files/7097096/run.log)


NodeJS version downgrade:
tari-labs/docker#16
@stringhandler stringhandler merged commit fc04875 into master Sep 6, 2021
@stringhandler stringhandler deleted the nodejs_version branch September 6, 2021 13:56
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.

2 participants