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

restrict increment-decrement fixer while fixing the postfix unary exp… #4415

Merged
merged 1 commit into from
Dec 29, 2018
Merged

restrict increment-decrement fixer while fixing the postfix unary exp… #4415

merged 1 commit into from
Dec 29, 2018

Conversation

enessoylu
Copy link
Contributor

@enessoylu enessoylu commented Dec 28, 2018

PR checklist

Overview of change:

Fixer for the increment-decrement rule changes some values in an unexpected way.
It is changing all occurrences with += or -=. But most of the cases for the postfix expressions, it is returning the wrong value.

const x = 1;
const y = 1;

x++ !== y+=1

CHANGELOG.md entry:

[bugfix] Correcting fixer for the increment-decrement

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.

Good find, thanks @enessoylu!

@JoshuaKGoldberg JoshuaKGoldberg merged commit 47a7218 into palantir:master Dec 29, 2018
@adidahiya adidahiya added this to the TSLint 5.12.1 milestone Jan 10, 2019
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.

Fixer for the increment-decrement rule is changing code semantics
3 participants