Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

Error bubbling and Better JSON output #10

Merged
merged 6 commits into from
Feb 13, 2020
Merged

Error bubbling and Better JSON output #10

merged 6 commits into from
Feb 13, 2020

Conversation

gbmor
Copy link
Contributor

@gbmor gbmor commented Feb 13, 2020

per issue #8

I also noticed in README.md that the "Installation From Script" url was incorrect, so I'm sneaking in a commit to fix that 😆

@athul
Copy link
Contributor

athul commented Feb 13, 2020

Screen Shot 2020-02-13 at 6 48 06 PM
Screen Shot 2020-02-13 at 6 47 43 PM
The first one is the current one v0.0.2. The second image is the updated one.

This formatter update just brings about 2 🐛 fixes 🎉 . I knew one of them existed but the other one was non-visible.

Now the o/p is fairly in good formatting and the it parses large json responses like the the request to https://api.github.com/repos/athul/pwcli/releases

Woo... today was super productive

@athul athul linked an issue Feb 13, 2020 that may be closed by this pull request
@athul athul changed the title Error bubbling Error bubbling and Better JSON output Feb 13, 2020
@athul
Copy link
Contributor

athul commented Feb 13, 2020

@gbmor what do you say? Squash when you find it good

@gbmor gbmor merged commit 4e31879 into master Feb 13, 2020
@gbmor gbmor deleted the error_bubbling branch February 13, 2020 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better Error Logs
2 participants