Skip to content

Textlint エラーの修正 #280

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, 2021
Merged

Textlint エラーの修正 #280

merged 1 commit into from
Apr 24, 2021

Conversation

naokie
Copy link
Member

@naokie naokie commented Apr 24, 2021

Description of Problem

Textlint のエラーが発生していた箇所を修正しました。

Proposed Solution

「HTML」の前後にスペースを追加しました。

$ node -e "var shell=require('shelljs');var files=shell.find(['./src/**/*.md']).filter(function(file){return !file.endsWith('/guide/team.md')}).join(' ');if(shell.exec('textlint --rulesdir ./node_modules/textlint-checker-for-vuejs-jp-docs/rules/textlint-rule-vue-jp-docs -f pretty-error '+files).code!==0){shell.exit(1)};"
textlint-rule-vue-jp-docs: 半角と全角の間は一文字あける :  [Lが] ... ...以下のHTMLが生成されます:
/home/runner/work/ja.vuejs.org/ja.vuejs.org/src/guide/migration/attrs-includes-class-style.md:41:1
        v
    40. 
    41. ...以下のHTMLが生成されます:
    42. 
        ^

textlint-rule-vue-jp-docs: 半角と全角の間は一文字あける :  [Lが] ...  には、すべての属性が含まれているので、すべての属性を別の要素に適用することが簡単にできます。先ほどの例は、次のHTMLが生成されます:
/home/runner/work/ja.vuejs.org/ja.vuejs.org/src/guide/migration/attrs-includes-class-style.md:51:9
                v
    50. 
    51. `$attrs` には、すべての属性が含まれているので、すべての属性を別の要素に適用することが簡単にできます。先ほどの例は、次のHTMLが生成されます:
    52. 
                ^

✖ 2 problems (2 errors, 0 warnings)

Additional Information

@netlify
Copy link

netlify bot commented Apr 24, 2021

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit 6fc7216

https://deploy-preview-280--vuejs-v3-ja-doc-preview.netlify.app

Copy link
Member

@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

LGTM!

@kazupon kazupon merged commit e8c12be into vuejs-jp:lang-ja Apr 24, 2021
@naokie naokie deleted the hotfix/guide/migration/attrs-includes-class-style branch April 25, 2021 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants