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

support for error messages #25

Merged
merged 9 commits into from
May 22, 2020

Conversation

osdiab
Copy link
Contributor

@osdiab osdiab commented May 20, 2020

Closes #24 , also removes the trailing period for the provided value (i think it doesn't help/the period can hurt clarity).

built on top of #20 so there's noise in the diff, here are the changes without that: https://github.com/OliverJAsh/io-ts-reporters/pull/25/files/39716c36ea5205155c0ba7117a5218940d90d983..57f0b2ae54130cd428cfd634978add1bfe983e9d

I rebased to make it based on master.

Example output at https://github.com/OliverJAsh/io-ts-reporters/pull/25/files#diff-e1b1ef74bf2611df9cd1e2173b684deeR85

src/index.ts Outdated Show resolved Hide resolved
@osdiab
Copy link
Contributor Author

osdiab commented May 21, 2020

refactored, ready to go

@gillchristian
Copy link
Owner

@osdiab this needs merge conflicts resolving as well

@osdiab
Copy link
Contributor Author

osdiab commented May 22, 2020

it doesn't show any merge conflicts with master? @gillchristian

@gillchristian gillchristian merged commit a55b017 into gillchristian:master May 22, 2020
@gillchristian
Copy link
Owner

There were conflicts for rebasing and merging, but squash and merge worked :)

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.

include error messages in output
2 participants