Move scripts to adapters-cli #397
Conversation
bergarces
commented
Feb 18, 2025
•
bergarces
commented
- Moves scripts from adapters-library to adapters-cli.
- Remove script dependencies from adapters-library
- Prunes docker images so that only production dependencies are kept
- Template snapshots are now in adapters-cli, using vitest as it's friendlier with esm.
| - name: Run test | ||
| run: npm run test -w packages/adapters-cli | ||
| env: | ||
| DEFI_ADAPTERS_LOG_LEVEL: warn |
There was a problem hiding this comment.
Move this to its own job, as it's now in a different package and it was difficult to open the logs on the previous job due to the size of this test.
| @@ -38,6 +38,7 @@ function filterErrors( | |||
| 'NotImplementedError', | |||
| 'NotSupportedError', | |||
| 'ProtocolTokenFilterRequiredError', | |||
| 'NotSupportedUnlimitedGetLogsBlockRange', | |||
There was a problem hiding this comment.
This stops the detect-errors test from failing.
There was a problem hiding this comment.
looks like its failed still
There was a problem hiding this comment.
It failed for something else. All of it BSC-related...
|
| RUN npm run build -w packages/adapters-library | ||
| RUN npm run build -w packages/workers |
There was a problem hiding this comment.
might be worth creating a npm script?
| RUN npm run build -w packages/adapters-library | ||
| RUN npm run build -w packages/adapters-api |
There was a problem hiding this comment.
might be worth creating a npm script?
There was a problem hiding this comment.
There is one that builds library, workers and api.
I'm purposefully not using it, because I don't need to build the workers to run the api.
