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

fix: ci publish issue #903

Conversation

mdebarros
Copy link
Member

No description provided.

mdebarros added 24 commits May 24, 2022 13:42
feat(mojaloop/#2092): upgrade nodeJS version for core services - mojaloop/project#2092
- standardised CI scripts
- fixed lint issues
- updated .nvmrc to latest LTS version
- added standard CI scripts/config to package.json: release, snapshot, standard-version, etc
- updated gitignore to include test/results and IGNORE patterns
- updated README with standard auto-release information
- Fixed imports
- Cleaned up Package.json
- Removed unnecessary dependencies
- Removed unecessary files

- 'get-port' dependency version is fixed to v5.1.1, this is because v9+ only supports ESM loaders and not CJS. This will need to be addressed in a future story.
- Helm chart mountPaths need to be updated from `/opt/central-ledger` to `/opt/app` as follows:
    ```YAML
        volumeMounts:
        - mountPath: /opt/app/config
          name: <deployment-name>
    ```

BREAKING CHANGE: Major version bump for node v16 LTS support, re-structuring of project directories to align to core Mojaloop repositories and docker image now uses `/opt/app` instead of `/opt/central-ledger` which will impact config mounts.
…r does not have any, also renamed re-factored test:int npm script to align to standards
…ed deprecated files for the integration-runner which is no longer necessary
…nto feat/#2092-Upgrade-NodeJS-Version-for-core-services
@mdebarros mdebarros marked this pull request as ready for review May 26, 2022 15:29
@mdebarros mdebarros requested a review from vijayg10 as a code owner May 26, 2022 15:29
@mdebarros mdebarros merged commit 49b3f06 into mojaloop:master May 26, 2022
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.

3 participants