Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Fix strict-type-predicate for unknown #4444

Merged

Conversation

ThomasdenH
Copy link
Contributor

@ThomasdenH ThomasdenH commented Jan 7, 2019

PR checklist

Overview of change:

Fixes the issue where a comparison with unknown would falsly give an error. Instead, comparison with unknown should always be allowed.

CHANGELOG.md entry:

[bugfix] fix strict-type-predicate with unknown

@ThomasdenH
Copy link
Contributor Author

Does a maintainer have time to review this?

@adidahiya adidahiya added this to the TSLint 5.12.1 milestone Jan 10, 2019
Copy link
Contributor

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@JoshuaKGoldberg JoshuaKGoldberg merged commit c054cbb into palantir:master Jan 10, 2019
adidahiya pushed a commit that referenced this pull request Jan 10, 2019
* Add tests for strict-type-predicates and unknown

* Fix unknown with strict-type-predicate

* Fix unknown literal
ColCh added a commit to ColCh/tslint that referenced this pull request Feb 3, 2019
* master: (60 commits)
  Added tslint-brunch to the list of 3rd party tools (palantir#4251)
  Switch to tslint-plugin-prettier, clean up rule options config syntax (palantir#4488)
  Enable grouped-imports for ordered-imports rule in tslint:all config (palantir#4420)
  Ordered imports grouping (palantir#4134)
  trailing-comma: check for a closing parenthesis (palantir#4457)
  Update index.md (palantir#4473)
  [bugfix] `no-unsafe-any`: allow implicitly downcasting `any` to `unknown` (palantir#4442)
  Add v5.12.1 changelog
  Bump version to 5.12.1
  Fix quotemark avoid-template issues (palantir#4408)
  Skip linting JSON files entirely (palantir#4001)
  Fix strict-type-predicate for unknown (palantir#4444)
  restrict increment-decrement fixer while fixing the postfix unary expressions (palantir#4415)
  Mention file names in script parse failures (palantir#4397)
  Revert breaking change to tslint:recommended, update tslint:latest (palantir#4404)
  Fix quotemark avoid-template issues (palantir#4408)
  Bump tslint dev dependency to 5.12.0 (palantir#4452)
  Skip linting JSON files entirely (palantir#4001)
  Fix strict-type-predicate for unknown (palantir#4444)
  [README] Update link for Webstorm (palantir#4450)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unknown and strict-type-predicate
3 participants