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
little bits of todo cleanup #2611
Conversation
2b161c2
to
c8e765c
Compare
@@ -492,10 +492,10 @@ export function deleteMetadata(field: string): DeleteMetadataFieldAction { | |||
}; | |||
} | |||
|
|||
// TODO: we should probably remove the default here. | |||
export function killKernel( | |||
payload: { restarting: boolean, ref?: KernelRef } = { restarting: false } |
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.
🙇 thanks!
@@ -847,10 +847,10 @@ export function shutdownReplyTimedOut(payload: { | |||
}; | |||
} | |||
|
|||
// TODO: probably should remove default here. | |||
export function restartKernel( | |||
payload: { clearOutputs: boolean, ref?: KernelRef } = { clearOutputs: false } |
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.
woo! yah, ditching defaults is definitely helpful here!
Howdy! I'm 🔓🤖! In order to keep information timely (based on the most recent release), we want all activity to be added to either new issues or open issues and PRs. In service to that goal, I, the lock bot close inactive closed issues when they haven't had activity in 120 days. Feel free to open a new issue for related bugs and link to relevant comments from this thread. |
Since I know this affects things with our flow setup, I'm removing these defaults for kill and restart.