iwamatsu0430 / textlint-rule-doubled-spaces

textlint rule for check doubled spaces in sentence.
MIT License
1 stars 1 forks source link

Doesn't work #21

Closed AlekSi closed 1 year ago

AlekSi commented 1 year ago

I tried to use that rule and it doesn't work for me.

It seems to be loaded correctly:

  textlint:cli textlint --version: 12.2.3 +0ms
  textlint:cli Running on files, stdin-filename: undefined +1ms
  textlint:module-loader config Config {
  textlint:module-loader   configFile: '/workdir/build/.textlintrc',
  textlint:module-loader   rulesBaseDirectory: undefined,
  textlint:module-loader   disabledRules: [],
  textlint:module-loader   rules: [
  textlint:module-loader     '@0x6b/normalize-whitespaces',
  textlint:module-loader     'doubled-spaces',
  textlint:module-loader     'no-zero-width-spaces',
  textlint:module-loader     'one-sentence-per-line'
  textlint:module-loader   ],
  textlint:module-loader   disabledFilterRules: [],
  textlint:module-loader   filterRules: [],
  textlint:module-loader   presets: [],
  textlint:module-loader   plugins: [],
  textlint:module-loader   pluginsConfig: {},
  textlint:module-loader   rulesConfig: {
  textlint:module-loader     '@0x6b/normalize-whitespaces': true,
  textlint:module-loader     'doubled-spaces': true,
  textlint:module-loader     'no-zero-width-spaces': true,
  textlint:module-loader     'one-sentence-per-line': true
  textlint:module-loader   },
  textlint:module-loader   filterRulesConfig: {},
  textlint:module-loader   rulePaths: [],
  textlint:module-loader   formatterName: undefined,
  textlint:module-loader   quiet: false,
  textlint:module-loader   color: true,
  textlint:module-loader   cache: false,
  textlint:module-loader   cacheLocation: '/workdir/.textlintcache',
  textlint:module-loader   ignoreFile: '/workdir/.textlintignore'
  textlint:module-loader } +0ms
  textlint:module-loader Loading rules from /deps/node_modules/@0x6b/textlint-rule-normalize-whitespaces/lib/index.js +3ms
  textlint:module-loader Loading rules from /deps/node_modules/textlint-rule-doubled-spaces/lib/index.js +4ms

...

But then nothing is being fixed:

{
    "filePath": "/workdir/website/docs/test.md",
    "output": "# Header\n\nPen  Pineapple   Apple  Pen.\n",
    "messages": [],
    "applyingMessages": [],
    "remainingMessages": []
}

Other rules seem to work. Any ideas what could be broken?

iwamatsu0430 commented 1 year ago

@AlekSi I'm sorry I didn't notice immediately.

Hmm.. I don't know why not working. 🤔 I think

I added example. please confirm your config. https://github.com/iwamatsu0430/textlint-rule-doubled-spaces/pull/35/commits/e449dcf8b6e9c402a87735cee8a1fb73afb653b0

AlekSi commented 1 year ago

Our configuration is there: https://github.com/FerretDB/FerretDB/blob/main/build/.textlintrc See also https://github.com/FerretDB/deps/tree/main/ferretdb-textlint.

iwamatsu0430 commented 1 year ago

@AlekSi Thank you for your information! I have understand the problem and fixed for Fixable Rule.

Please try to update to 1.0.2