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

improvement(repl): add in feedback when resetting REPL state #14521

Merged
merged 1 commit into from Feb 28, 2022

Conversation

ckipp01
Copy link
Member

@ckipp01 ckipp01 commented Feb 20, 2022

As a user I'm a big fan of getting feedback from the tools I'm working
with. Without out, you have to go on assumption that what you just tried
worked unless it explodes. In Scala 2 when you do a :reset in the REPL
you get a nice message about it being reset and even some details about
what exactly was reset.

Following that train of thought this just adds in small note when you
reset your session. Here are a couple examples which you can also see in
the changed tests:

scala>:reset -deprecation
Resetting REPL state with the following settings:
  -deprecation
scala>:reset
Resetting REPL state.

As a user I'm a big fan of getting feedback from the tools I'm working
with. Without out, you have to go on assumption that what you just tried
worked unless it explodes. In Scala 2 when you do a `:reset` in the REPL
you get a nice message about it being reset and even some details about
what exactly was reset.

Following that train of thought this just adds in small note when you
reset your session. Here are a couple examples which you can also see in
the changed tests:

```none
scala>:reset -deprecation
Resetting REPL state with the following settings:
  -deprecation
```

```none
scala>:reset
  Resetting REPL state.
```
@TheElectronWill TheElectronWill merged commit b60d90f into scala:main Feb 28, 2022
@ckipp01 ckipp01 deleted the reset branch March 1, 2022 13:58
@Kordyjan Kordyjan added this to the 3.1.3 milestone Aug 2, 2023
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

4 participants