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

chore(deps): use thegraph dependency types and subgraph client #439

Merged

Conversation

LNSD
Copy link
Contributor

@LNSD LNSD commented Nov 28, 2023

This PR moves the gateway from using the thegraph types instead of the toolshed::thegraph.

@LNSD LNSD requested a review from Theodus November 28, 2023 16:37
@LNSD LNSD self-assigned this Nov 28, 2023
@LNSD LNSD changed the base branch from release to main November 28, 2023 16:37
@LNSD
Copy link
Contributor Author

LNSD commented Nov 28, 2023

This can be merged once the v15.1.0 release is made. No need to rush it

@LNSD LNSD added the DO NOT MERGE (delayed release) Delaying merge until next release cycle label Nov 28, 2023
@Theodus
Copy link
Member

Theodus commented Nov 28, 2023

@LNSD Actually, can you remove the changes from release? I'd like to merge this first since it cuts out a lot of stuff.

@LNSD LNSD force-pushed the chore-deps-use-thegraph-crate-types-and-subgraph-client branch from 6889913 to 7cfed3c Compare November 28, 2023 16:44
@LNSD LNSD removed the DO NOT MERGE (delayed release) Delaying merge until next release cycle label Nov 28, 2023
Copy link
Member

@Theodus Theodus left a comment

Choose a reason for hiding this comment

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

Great. Thanks!

@Theodus Theodus merged commit 3b5e832 into main Nov 28, 2023
1 check passed
@Theodus Theodus deleted the chore-deps-use-thegraph-crate-types-and-subgraph-client branch November 28, 2023 16:58
Theodus pushed a commit that referenced this pull request Dec 18, 2023
This PR moves the gateway from using the `thegraph` types instead of the
`toolshed::thegraph`.
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.

2 participants