-
Notifications
You must be signed in to change notification settings - Fork 191
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: upgrade ses to 0.12.3 to avoid console noise #2552
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.
LGTM!
lint:types is not happy. interesting!
|
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.
LGTM!
@FUDCo any clues on the swingset-runner test/test-demo.js:25 failure? https://github.com/Agoric/agoric-sdk/pull/2552/checks?check_run_id=2009422750 |
cc5ba13
to
6cf75c6
Compare
6cf75c6
to
84f9ccf
Compare
// FIXME: Without importing vega ahead of lockdown, | ||
// node -r esm bin/runner --init --memdb run demo/encouragementBot | ||
// exits with error code 1, just printing `{}`. | ||
// | ||
// Since this started with ses@0.12.3 we suspect vega is somehow | ||
// hitting the override mistake with `constructor`. | ||
import 'vega'; |
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.
We need to investigate this.
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.
FYI, the comment is too specific. The problem will occur with any invocation of swingset-runner regardless of the command line options.
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.
See #2575
fixes #2324