-
Notifications
You must be signed in to change notification settings - Fork 155
Add that now deploy
also accepts the --network
and --network-file
options
#162
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
Add that now deploy
also accepts the --network
and --network-file
options
#162
Conversation
|
||
In some cases, you will want to deploy the same subgraph to multiple Ethereum networks without duplicating all of its code. The main challenge that comes with this is that the contract addresses on these networks are different. | ||
|
||
### graph-cli >=0.29.0 |
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.
Should we change this heading since the options were only added in v0.32.0
for graph deploy
?
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 wasn't sure about that either, because the options are available in graph-cli for the build
command since v0.29.0, we just made them available for deploy
too, and now you can use whichever.
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.
Or I can change the heading to Using graph-cli
and Using templates/templating
or something of that sort
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.
Or I can change the heading to Using graph-cli and Using templates/templating or something of that sort
I think that would be better! 👍
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.
This is great, thank you!
Hey, I fixed a merge conflict caused by the new structure PR, fixed a typo and added a Note about the |
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.
🙌
No description provided.