-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat(2151): helm-release-v12.1.0 #106
feat(2151): helm-release-v12.1.0 #106
Conversation
mdebarros
commented
Jun 7, 2021
•
edited
Loading
edited
- Updated dependencies
- Fixes for updated dependencies
- SQLite requiring the most refactoring --> ref: https://github.com/kriasoft/node-sqlite/blob/master/CHANGELOG.md#401---sat-apr-04-2020-214639.
- Added SQLite3 as a dependency.
- Fixes import issues for UUID dependency --> ref: https://github.com/uuidjs/uuid#upgrading-from-uuid3.
- Fixes to validate lib due to json-rules-engine dependency upgrade --> https://github.com/CacheControl/json-rules-engine/blob/master/CHANGELOG.md#400--2019-08-22
- Bump patch version
- Fixed lint issues
- Added dep:check/update scripts for main project, libs, and added "all" variant using npm-run-all
- Updated dependencies - Fixes for updated depenencies - Bump patch version - Fixed lint issues - Added dep:check/update scripts for main project and libs, and added "all" variant using npm-run-all
I am not 100% this PR is LGTM yet, I suspect that the Going to create a temp docker image for this, and run some tests on OSS environments to see if I can pick up any issues. |
The issue is most likely here --> https://github.com/mojaloop/mojaloop-simulator/blob/master/src/lib/rules-engine/index.js#L80. Which is due to the breaking changes introduced by upgrading |
Also had to address the error below:
|
Tested using OSS dev3 environment. Fixed all errors/issues that I was able to find. It LGTM now! :D/ TTK GP Tests are all passing using
|