-
Notifications
You must be signed in to change notification settings - Fork 138
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
developing-page #746
Open
idalithb
wants to merge
6
commits into
main
Choose a base branch
from
developing-update
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
developing-page #746
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This comment was marked as off-topic.
This comment was marked as off-topic.
saihaj
reviewed
Aug 19, 2024
saihaj
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 19, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
|
||
As with all subgraph development, it starts with local development and testing. Developers can use the same local setup whether they are building for The Graph Network or a local Graph Node, leveraging `graph-cli` and `graph-ts` to build their subgraph. Developers are encouraged to use tools such as [Matchstick](https://github.com/LimeChain/matchstick) for unit testing to improve the robustness of their subgraphs. | ||
1. Create, deploy and publish subgraphs to The Graph using **Graph CLI** and [Subgraph Studio](https://thegraph.com/studio/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the bold on "Graph CLI" is necessary. Would be nice if we could link to a page with more information on it (like a reference of all the commands), but that doesn't seem to exist. 🥲 cc @MichaelMacaulay
So for now, I just suggest:
Suggested change
1. Create, deploy and publish subgraphs to The Graph using **Graph CLI** and [Subgraph Studio](https://thegraph.com/studio/). | |
1. Create, deploy and publish subgraphs to The Graph using Graph CLI and [Subgraph Studio](https://thegraph.com/studio/). |
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
benface
reviewed
Aug 20, 2024
idalithb
requested review from
brandonleekramer
and removed request for
alex-pakalniskis,
saihaj and
brandonleekramer
September 11, 2024 15:57
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
-Bullet points over paragraphs
-Added additional context
@saihaj @alex-pakalniskis please let me know any info can be removed