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

Missing whitespace around export statement with "check-module" argument #4014

Closed
hdurix opened this issue Jul 5, 2018 · 0 comments · Fixed by #4554
Closed

Missing whitespace around export statement with "check-module" argument #4014

hdurix opened this issue Jul 5, 2018 · 0 comments · Fixed by #4554
Labels
Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier Status: Accepting PRs Type: Bug
Milestone

Comments

@hdurix
Copy link

hdurix commented Jul 5, 2018

Bug Report

  • TSLint version: 5.10.0
  • TypeScript version: 2.9.2
  • Running TSLint via: CLI

TypeScript code being linted

import MockedForm from './MockedForm';

export {MockedForm};

with tslint.json configuration:

{
  "defaultSeverity": "warning",
  "extends": [
    "tslint:recommended"
  ],
  "linterOptions": {
    "exclude": [
      "node_modules/**"
    ]
  },
  "rules": {
    "quotemark": [true, "single"],
    "indent": [true, "spaces", 2],
    "interface-name": false,
    "object-literal-sort-keys": false,
    "whitespace": [true, "check-branch", "check-decl", "check-operator", "check-separator", "check-type", "check-module"]
  }
}

Actual behavior

No issue is detected by tslint.

Expected behavior

Add whitespace around export statements when missing. Because of rule "whitespace" with argument "check-module".

import MockedForm from './MockedForm';

export { MockedForm };
devmattrick added a commit to devmattrick/tslint that referenced this issue Feb 28, 2019
@adidahiya adidahiya added the Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier label Mar 5, 2019
@adidahiya adidahiya added this to the 5.14.0 milestone Mar 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Formatting rule Relates to a rule which enforces code formatting and likely overlaps with prettier Status: Accepting PRs Type: Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants