-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
Added test cases for StylesContext #236
Conversation
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.
Hey @Suvrat1629, thank you for the PR. I've left a couple of comments to be addressed. Let me know if you need any clarifications 😊
I have made the changes as you told me please tell me if you need me to make anymore changes |
Hey @Suvrat1629, there're still outstanding comments unresolved 😅 |
Oh sorry about that |
Ok I have resolved the comments please take a look |
@tjtanjin updates? |
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.
LGTM!
Description
Please include a brief summary of the change and include the relevant issue(s).
Closes #232
What change does this PR introduce?
I have added a StylesContext.test.tsx file which contains test code for StylesContext.
Please select the relevant option(s).
What is the proposed approach?
Please give a short overview/explanation on the approach taken to resolve the issue(s).
Checklist: