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

Add error for no profiles found (new user) #303

Merged
merged 5 commits into from
Feb 23, 2021
Merged

Conversation

JakeDawkins
Copy link
Contributor

@JakeDawkins JakeDawkins commented Feb 23, 2021

I think we could potentially build off this for a custom printer to avoid printing the error: syntax. This is just to get something out there for the preview :)

@@ -72,7 +72,7 @@ impl Display for RoverError {
suggestion_descriptor_message.push(' ');
}
let suggestion_descriptor = Cyan.bold().paint(&suggestion_descriptor_message);
write!(formatter, "{} {}", suggestion_descriptor, suggestion)?;
writeln!(formatter, "{} {}", suggestion_descriptor, suggestion)?;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was mentioned by Jesse-

zsh prints a % after a command if it doesn't end with a newline

@JakeDawkins JakeDawkins added the feature 🎉 new commands, flags, functionality, and improved error messages label Feb 23, 2021
@JakeDawkins JakeDawkins linked an issue Feb 23, 2021 that may be closed by this pull request
Copy link
Member

@lrlna lrlna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@lrlna lrlna merged commit 3a13ed8 into main Feb 23, 2021
@lrlna lrlna deleted the jake/add-friendly-error branch February 23, 2021 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 🎉 new commands, flags, functionality, and improved error messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RFC: "Welcome to Rover" on first command run
2 participants