Skip to content

fix(ja-space-between-half-and-full-widt): Collect dependencies #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

hokaccha
Copy link
Contributor

It was causing an error because it is being used here but was not included in the dependencies.

import { matchPatterns } from "@textlint/regexp-string-matcher";

$ textlint --debug --preset preset-ja-spacing README.md
  textlint:cli cliOptions: {"debug":true,"preset":["preset-ja-spacing"],"ignorePath":"/.../.textlintignore","init":false,"stdin":false,"format":"stylish","color":true,"quiet":false,"textlintrc":true,"cache":false,"cacheLocation":"/.../.textlintcache","_":["README.md"]} +0ms
  textlint:cli textlint --version: 14.0.4 +0ms
  textlint:cli Running on files, stdin-filename: undefined +0ms
  textlint:cli-loader loadPackagesFromRawConfig failed: { ok: false, error: { message: 'Can not load rule', errors: [ [Error] ] } } +0ms
Error
Failed to load packages

Stack trace
Error: Failed to load packages
    at loadCliDescriptor (/.../node_modules/textlint/lib/src/loader/CliLoader.js:44:15)
    at async loadDescriptor (/.../node_modules/textlint/lib/src/cli.js:24:27)
    at async Object.executeWithOptions (/.../node_modules/textlint/lib/src/cli.js:111:28)

@azu azu added the Type: Bug Bug or Bug fixes label Apr 24, 2024
@azu azu changed the title Collect dependencies fix(ja-space-between-half-and-full-widt): Collect dependencies Apr 24, 2024
@azu azu merged commit c59ae80 into textlint-ja:master Apr 24, 2024
@azu
Copy link
Member

azu commented Apr 24, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug or Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants