-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix: Re-export json in typescript deploy-configs #5607
Conversation
|
✅ Deploy Preview for opstack-docs canceled.
|
This can still result in a diff between the two |
934a61f
to
e17ce0e
Compare
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.
Comment nit suggestions
34d6a51
to
4358d97
Compare
to de-duplicate hard-coded values, allow for backwards typescript compatibility, and maintain the use of comments.
cf50476
to
c848c41
Compare
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
Updates the
mainnet.ts
deploy-config to re-export the object defined inmainnet.json
to de-duplicate valuesand re-define values inline.Metadata