Skip to content
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

Feature: shutdown: honor the incoming restart argument #73

Merged
merged 1 commit into from
Mar 5, 2020

Conversation

tinmarino
Copy link
Contributor

Fix: Issue #64

Side changes: Kernel variable execution_count and sandbox changed
scope: from lexical to object member so that they can be changed without
being passed as function argument

Side feature: when the user types ‘exit’ or ‘quit’ (or their uppercase magic equivalents) the argument is boolified (default to zero) and it is do-restart

Fix: Issue bduggan#64

Side changes: Kernel variable `execution_count` and `sandbox` changed
scope: from lexical to object member so that they can be changed without
being passed as function argument

Side feature: when the user types ‘exit’ or ‘quit’ (or their uppercase magic equivalents) the argument is boolified (default to zero) and it is `do-restart`
@bduggan bduggan merged commit f4fb804 into bduggan:master Mar 5, 2020
@bduggan
Copy link
Owner

bduggan commented Mar 5, 2020

Great!

@tinmarino
Copy link
Contributor Author

Awesome integration work bduggan (as 3 days ago).
I'm back to work then. Doing the 3 Issues linked to context persistence.

And then I'll be slower as I'll go for a nice way to integrate help (#74). Looking at p6doc.
And then ... I have no more improvement idea it was so good in the first place.

@tinmarino tinmarino deleted the merge_restart branch March 5, 2020 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants