You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
code change to support the upcoming node alpha.10. The changes are related to bumping devnet to v2.0.0-alpha.11, @rollups/contracts v2.2.0 and the SDK package.
The artefact used for this development: build artefact can be found here from branch next/2.0.
The dependency waits on the release of the node alpha.10. Once it is done, I can update the sdk and release it, then update this PR accordingly, and once it is merged, a new CLI release can be done.
This commit here 827b23e will be removed once the PR #449 is merged, as I pushed that change there too. So, if you use cartesi run, the change avoids issues.
As I understand it, a user can, at ease, re-enable an application that went into a failed state. Let me know if you don't think the change for the upcoming rollups-node does not worth the maintenance, and I will undo it.
As I understand it, a user can, at ease, re-enable an application that went into a failed state. Let me know if you don't think the change for the upcoming rollups-node does not worth the maintenance, and I will undo it.
From an node operator perspective it makes sense, but from an application developer perspective, why does he needs that command?
As I understand it, a user can, at ease, re-enable an application that went into a failed state. Let me know if you don't think the change for the upcoming rollups-node does not worth the maintenance, and I will undo it.
From an node operator perspective it makes sense, but from an application developer perspective, why does he needs that command?
What are the profile or profiles the cartesi-cli was meant to support? Is the initial assumption that it is only for application developers? Also, who develop the application never operates it?
So, by what @tuler is saying, I figure we don't need that in the cartesi-cli for the users we discussed earlier @vfusco , do you agree with that? They would do it directly using the rollups-node. Also, it is my understanding that the release is just to package the next node, since the current set of features is perfectly fine without any extras, right @tuler? Just making sure my understanding of this thread is sound.
The reason will be displayed to describe this comment to others. Learn more.
I mentioned here that the commit will be removed as I already pushed one in #449 , here is just in case one of you wanted to have a run and test locally to avoid figuring out why rollups-node was not starting.
This file contains hidden or 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
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.
Summary
code change to support the upcoming node
alpha.10. The changes are related to bumping devnet tov2.0.0-alpha.11, @rollups/contractsv2.2.0and the SDK package.The artefact used for this development: build artefact can be found here from branch
next/2.0.Dependencies
The dependency waits on the release of the node
alpha.10. Once it is done, I can update thesdkand release it, then update this PR accordingly, and once it is merged, a new CLI release can be done.