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

docs: add info about TradingSDK to main readme #228

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

shoom3301
Copy link
Contributor

No description provided.

@shoom3301 shoom3301 requested review from a team December 24, 2024 12:01
@shoom3301 shoom3301 self-assigned this Dec 24, 2024
@shoom3301 shoom3301 merged commit 7658e39 into main Dec 24, 2024
7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 24, 2024
@coveralls
Copy link
Collaborator

Coverage Status

coverage: 78.378%. remained the same
when pulling b294384 on feat/add-trading-sdk-to-main
into 6f3f5c0 on main.

Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is much better as it highlights the Trading SDK, but should we either move some of this to the docs, https://docs.cow.fi/category/sdks or adapt it to reflect our new SDK?

@alfetopito alfetopito deleted the feat/add-trading-sdk-to-main branch December 30, 2024 11:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants