-
Notifications
You must be signed in to change notification settings - Fork 205
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 placeholder flag for contract upgrading #17126
Conversation
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.
Awesome work Simon. LGTM :)
@@ -20,8 +20,6 @@ import scala.language.implicitConversions | |||
|
|||
class ConfigSpec extends AsyncWordSpec with Matchers with CantonFixture { | |||
|
|||
final override protected lazy val devMode: Boolean = true |
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.
👍
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.
Can someone explain this for me please.
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.
Setting devMode
to be true means Canton logging will be kept.
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.
Ah, not quite correct (I think it used to be?). devMode
now means our patched version of Canton is used.
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.
devMode now means our patched version of Canton is used.
@carlpulley-da you are confusing with cantonFixtureDebugMode
devMode = true
always meant our patched version of Canton is used (+ canton is started in dev
mode).
Such tests that use the patched version of Canton need be mark with the tag dev-canton-test
in the bazel Build file which was not the case of this ConfigTest.
Taking into account ConfigTest test does not seems to used any dev
feature, it seems more natural to remove the devMode = true
that to mark it with dev-canton-test
Woo Hoo. Finally! |
Once canton has been compiled against this version we can re-enable the tests. |
Related canton PR: https://github.com/DACH-NY/canton/pull/13971