-
Notifications
You must be signed in to change notification settings - Fork 17
Enabled strict mode and added required changes to the current code #2126
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
Conversation
…o Mil4n0r/strict-mode-true
GomezIvann
left a comment
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.
This is a partial review! Before continue reviewing this PR Enrique I would like you to avoid changes that are not related to Typescript errors please It is not because I differ with your programming preferences, is a question of how difficult it makes it for me to review these types of scenarios where I don't know if you have made the change out of typescript/lint or personal preference.
I would like to keep this PR as clean and focused as possible on the flag strict: true please!
…halstack-react into Mil4n0r/strict-mode-true
…hnology/halstack-react into Mil4n0r/strict-mode-true
…halstack-react into Mil4n0r/strict-mode-true
…halstack-react into Mil4n0r/strict-mode-true
…halstack-react into Mil4n0r/strict-mode-true
…halstack-react into Mil4n0r/strict-mode-true
…hnology/halstack-react into Mil4n0r/strict-mode-true
Checklist
/libdirectory./websiteas needed.Description
Enabled strict mode