Skip to content

Conversation

@ltrouton
Copy link
Contributor

Related Ticket: TINY-12772

Description of Changes:

  • Add a custom eslint rule that suggests to use the Type module from katamari instead of manualy 'typeof' and individual undefined and null checks

Pre-checks:

  • Changelog entry added
  • package.json version bumped (if first change of new major/minor)
  • Tests have been added (if applicable)

@ltrouton ltrouton requested review from a team, MitchC1999, TheSpyder, jscasca and tiny-ben-tran and removed request for a team August 27, 2025 04:10
@ltrouton ltrouton requested review from a team as code owners August 27, 2025 04:10
@ltrouton
Copy link
Contributor Author

Use of rule in tinymce repo: tinymce/tinymce#10558

Copy link
Member

@spocke spocke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we also detect truthy checks I guess that needs type information? I assume this will create a lot of errors on the old tinymce code base. But we can just turn of the rule or make a monster PR to fix it.

@ltrouton
Copy link
Contributor Author

ltrouton commented Dec 9, 2025

I am thinking of handling falsy and truthy checks using native typescript eslint rules:

I need to look into this more again

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.

4 participants