Skip to content

Conversation

@Mil4n0r
Copy link
Collaborator

@Mil4n0r Mil4n0r commented Dec 3, 2024

Checklist

  • Build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Description
Enabled strict mode

@Mil4n0r Mil4n0r marked this pull request as ready for review December 4, 2024 10:16
@Mil4n0r Mil4n0r requested a review from GomezIvann December 4, 2024 10:16
@Mil4n0r Mil4n0r marked this pull request as draft December 5, 2024 16:26
@Mil4n0r Mil4n0r marked this pull request as ready for review December 5, 2024 16:47
Copy link
Collaborator

@GomezIvann GomezIvann left a 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!

@Mil4n0r Mil4n0r requested a review from GomezIvann December 16, 2024 07:19
@GomezIvann GomezIvann merged commit d004266 into master Jan 7, 2025
4 checks passed
@GomezIvann GomezIvann deleted the Mil4n0r/strict-mode-true branch January 7, 2025 12:21
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.

3 participants