Skip to content

updated Japanese readme #395

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 2 commits into from
Apr 28, 2017
Merged

updated Japanese readme #395

merged 2 commits into from
Apr 28, 2017

Conversation

jbondpdx
Copy link
Contributor

@LarissaLane gave me an updated Japanese readme today, so let's get it in here.

@LarissaLane
Copy link

Thanks Jean!

@smortex
Copy link
Collaborator

smortex commented Apr 28, 2017

For your information, I just submitted #396 which fixes some spacing issues in the original README.markdown. It looks like these syntax problems also exist in your proposed change, so you will probably want to fix them 😉.

@pmcmaw
Copy link
Contributor

pmcmaw commented Apr 28, 2017

@smortex - Thank you very much for your input.
I have added review comments to ensure we do not miss this.


データタイプ: 文字列

構成ファイルのEPPテンプレートへの絶対パスまたは相対パスを指定します(値の例: 'ntp/ntp.conf.epp')。このパラメータと`config_template`パラメータの**両方**を指定すると、検証エラーがスローされます
構成ファイルのEPPテンプレートへの絶対パスまたは相対パスを指定します(値の例: 'ntp/ntp.conf.epp')。このパラメータと`config_template`パラメータの**両方**を指定すると、バリデーションエラーが発生します

####`config_file_mode`
Copy link
Contributor

Choose a reason for hiding this comment

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

Requires a space for rendering.

@@ -512,27 +522,27 @@ NTP統計の保存先(ntpモニタリングが有効になっている場合)

####`step_tickers_epp`
Copy link
Contributor

Choose a reason for hiding this comment

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

Requires a space for rendering.

@jbondpdx
Copy link
Contributor Author

@pmcmaw , I added those spaces in, so we should be good. Thanks again to @smortex for the catch on those.

@pmcmaw pmcmaw merged commit dbfc542 into puppetlabs:master Apr 28, 2017
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.

4 participants