这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@fengtality
Copy link
Contributor

Before submitting this PR, please make sure:

  • Your code builds clean without any errors or warnings
  • You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

  • Fixes yarn dependencies
  • Updates axios

@fengtality
Copy link
Contributor Author

@rapcmia I will merge this in to fix these dependencies before the PancakeSwapV3 and Osmosis connectors are reviewed.

@fengtality fengtality merged commit 4afa163 into development Dec 1, 2023
@fengtality fengtality deleted the fix/fix-yarn branch December 1, 2023 22:09
@chesoft
Copy link
Contributor

chesoft commented Dec 7, 2023

@rapcmia I will merge this in to fix these dependencies before the PancakeSwapV3 and Osmosis connectors are reviewed.

I'd merged this into my Osmosis feature branch a few days ago and just now noticed my tests started failing.

I ended up doing a clean pull of https://github.com/hummingbot/gateway/tree/development

And here's yarn test from that - every module that uses axios is failing now
image

@chesoft
Copy link
Contributor

chesoft commented Dec 7, 2023

I got jest working again

jest.config.js

  moduleNameMapper: {
    eccrypto: '<rootDir>/test/mock/eccrypto-mock.js',
    axios: '<rootDir>/node_modules/axios/dist/node/axios.cjs'
  },

pecuniafinance@6b6a0ec

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