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.

Sign up for GitHub

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

Jump to bottom

wl/start (and restart etc.) and package loads should return a value on success #111

Closed
light-matters opened this issue Sep 7, 2024 · 2 comments
Closed

wl/start (and restart etc.) and package loads should return a value on success #111

light-matters opened this issue Sep 7, 2024 · 2 comments
Labels
UX User/developer eXperience

Comments

@light-matters
Copy link
Contributor

It might be better for wl/start and wl/restart to return a value, e.g. :ok, :started, :success, :restarted or a welcome string to communicate positively with the user.

This would be less ambiguous than 'nil'.
@light-matters light-matters added the UX User/developer eXperience label Sep 7, 2024
@light-matters
Copy link
Contributor Author

same for package loading.

@light-matters light-matters changed the title wl/start (and restart etc.) should return a value on success wl/start (and restart etc.) and package loads should return a value on success Sep 7, 2024
@light-matters
Copy link
Contributor Author

currently package loads return nil, even if the file doesn't exist.

holyjak added a commit that referenced this issue Sep 20, 2024
@holyjak
Fix #111: wl/start etc return value
39498fd 
Issue: An inexperienced user may not know what the return value of nil means.

Fix: Be explicit and return a map with `:status :ok` and perhaps additional info. E.g. wolfram version seems useful.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
UX User/developer eXperience
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@light-matters