Skip to content
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

refactor(textlint): 敬体(ですます調)と常体(である調)の使い分けを厳密に #94

Merged
merged 4 commits into from
May 6, 2016

Conversation

azu
Copy link
Owner

@azu azu commented May 6, 2016

https://github.com/azu/textlint-rule-no-mix-dearu-desumasu @ 2 でpreferオプションが導入されて、
本文は"ですます"、箇条書きは"である"で書くように設定したので、それに合わせてリファクタリング。

https://github.com/azu/textlint-rule-no-mix-dearu-desumasu @ 2 でpreferオプションが導入されて、
本文は"ですます"、箇条書きは"である"で書くように設定したので、それに合わせてリファクタリング。
@codecov-io
Copy link

codecov-io commented May 6, 2016

Current coverage is 96.21%

Merging #94 into master will decrease coverage by -0.09%

@@             master        #94   diff @@
==========================================
  Files             6          6          
  Lines          1033       1029     -4   
  Methods           0          0          
  Messages         38         39     +1   
  Branches          0          0          
==========================================
- Hits            995        990     -5   
- Misses           38         39     +1   
  Partials          0          0          

Powered by Codecov. Last updated by 85e7e6a...be5afc4

@azu azu merged commit b5f4779 into master May 6, 2016
@azu azu deleted the textlint-dearu branch May 6, 2016 02:27
@@ -105,7 +105,7 @@ calculator.fn = calculator.prototype;
<script src="greenify.js"></script>
```

まだNode.jsで使われているCommonJSやES6 Modulesといったものがなかった時代に作られた仕組みであるため
まだNode.jsで使われているCommonJSやES6 Modulesなどがなかった時代に作られた仕組みなので
Copy link
Owner Author

Choose a reason for hiding this comment

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

s/まだ/また/ ?

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

Successfully merging this pull request may close these issues.

2 participants