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

feat(cargo-shuttle): add suggestions in case of cmd failures #1245

Conversation

iulianbarbu
Copy link
Contributor

Description of change

Added suggestions in case of requests' errors when running cargo-shuttle commands. These suggestions do not consider the requests' status codes. This is a short-term solution for handling errors across CLI user flows (like ProjectNotReady 503s) by suggesting commands that verify if previous commands worked, repeating the command again and if the fialures persist, checking the status page or reaching out on Discord.

How has this been tested? (if applicable)

N/A

Copy link
Member

@chesedo chesedo left a comment

Choose a reason for hiding this comment

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

Can we demo these somehow?

@iulianbarbu
Copy link
Contributor Author

Can we demo these somehow?

Good question. Not very sure how to do it easier than simulating the error by interfering with the code, when running cargo-shuttle against a local Shuttle deployment.

cargo-shuttle/src/suggestions.rs Outdated Show resolved Hide resolved
@iulianbarbu iulianbarbu force-pushed the feature/eng-1283-assess-whether-we-can-suggest-project-restart-anywhere branch from a9122bf to 3025bcc Compare September 18, 2023 10:44
@iulianbarbu iulianbarbu merged commit 27092b8 into shuttle-hq:main Sep 18, 2023
33 checks passed
iulianbarbu added a commit to GugaGongadze/shuttle that referenced this pull request Sep 27, 2023
…-hq#1245)

* feat(cargo-shuttle): add suggestions in case of cmd failures

* refactor(cargo-shuttle): break suggestions module
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

3 participants