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

Improve UI #2230

Merged
merged 3 commits into from Mar 27, 2020
Merged

Improve UI #2230

merged 3 commits into from Mar 27, 2020

Conversation

sdispater
Copy link
Member

@sdispater sdispater commented Mar 26, 2020

Pull Request Check List

  • Added tests for changed code.
  • Updated documentation for changed code.

This PR changes the user interface a bit to make it more intuitive. It changes colors in some places to be more readable and provide a consistent interface on dark and light terminal themes.

It also upgrades Cleo and CliKit to provide beautiful and more intuitive errors with various levels of detail depending on the current verbosity. The rendered stack traces will also collapse duplicated frames (useful for recursion errors).

Here are some examples:

Verbose mode

Screenshot 2020-03-25 at 19 49 43

Debug mode

Screenshot 2020-03-25 at 19 51 08

Duplicated frames

Screenshot 2020-03-25 at 19 58 08

@sdispater sdispater force-pushed the improve-ui branch 3 times, most recently from 2445393 to b0bfab9 Compare Mar 27, 2020
@JBKahn
Copy link
Contributor

JBKahn commented May 28, 2020

Where can I find the woops source code repository?

@SanketDG
Copy link
Contributor

SanketDG commented May 28, 2020

https://pypi.org/project/woops/

Looks like it's not open source'd yet.

@JBKahn
Copy link
Contributor

JBKahn commented May 28, 2020

We can download and read it but I was hoping it was in a public repo for convenience.

@abitrolly
Copy link
Contributor

abitrolly commented Dec 18, 2020

I must admit this part of poetry is my favourite.

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