-
Notifications
You must be signed in to change notification settings - Fork 20
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
Output class #47
Output class #47
Conversation
4d0020a
to
7bbff03
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ha36d , This looks good to me, Can you try and squash the commits as well? We try to keep our history atomic and clean!!
It's done @Kaushl2208. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes looks good.
Needs a test
@ha36d Please expose the new class Other functionality is working fine. |
@GMishx it's ok. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested, working as expected.
Adds a check to fix the second part of #38
Adds a separate class for output to fix the second part of #38