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

translate: guide/file-structure #404

Merged
merged 2 commits into from
Jun 16, 2019

Conversation

k-ta-yamada
Copy link
Contributor

close #381
guide/file-structure.mdの追加翻訳です。

@netlify
Copy link

netlify bot commented Jun 12, 2019

Deploy preview for angular-ja ready!

Built with commit 9dd5846

https://deploy-preview-404--angular-ja.netlify.com

@k-ta-yamada k-ta-yamada force-pushed the translate/guide/file-structure branch from 310a5b3 to 76eb22f Compare June 15, 2019 05:35
@k-ta-yamada k-ta-yamada marked this pull request as ready for review June 15, 2019 05:37
@k-ta-yamada
Copy link
Contributor Author

@lacolaco
翻訳完了しましたが、Netlifyで下記のようにDangling Linksのエラーとなってしまいました。
下記のどちらかの対応になるかな、と思うのですが、どちらがいいでしょうか。

  1. ヘッダの部分を翻訳しない
  2. 該当箇所に{@a xxx}という感じでanchorを付け足す
2:41:32 PM: info:    running processor: checkAnchorLinksProcessor
2:41:33 PM: warn:    Dangling Links Found in "guide/creating-libraries.json":
2:41:33 PM:  - guide/file-structure#library-project-files
2:41:33 PM: warn:    Dangling Links Found in "guide/deployment.json":
2:41:33 PM:  - guide/file-structure#application-configuration-files
2:41:35 PM: error:   Error: 2 unmatched links

@k-ta-yamada
Copy link
Contributor Author

circleciはrsyncのダウンロードに失敗しているようです。

E: Failed to fetch http://deb.debian.org/debian/pool/main/r/rsync/rsync_3.1.2-1+deb9u1_amd64.deb  404  Not Found

rsync_3.1.2-1+deb9u1_amd64.deb
ではなく
rsync_3.1.2-1+deb9u2_amd64.deb
になったようです。

@lacolaco
Copy link
Collaborator

@k-ta-yamada リンクのエラーはこれを参考にしてください
https://github.com/angular/angular-ja/blob/master/CONTRIBUTING.md#dangling-links-found-%E3%81%AB%E3%81%A4%E3%81%84%E3%81%A6-%E3%82%A2%E3%83%B3%E3%82%AB%E3%83%BC%E3%81%AE%E8%BF%BD%E5%8A%A0

@k-ta-yamada k-ta-yamada force-pushed the translate/guide/file-structure branch from 76eb22f to 6891baa Compare June 15, 2019 08:15
@k-ta-yamada
Copy link
Contributor Author

@lacolaco
ご確認、ご対応ありがとうございます。
CONTRIBUTING.mdの記載を見落としていました 🙇

CircleCI側の対応分も取り込むため修正後にrebase実施しております。

Copy link
Collaborator

@lacolaco lacolaco left a comment

Choose a reason for hiding this comment

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

翻訳ありがとうございます!1点コメントしたのと、ライブラリーライブラリ の表記ゆれがありますので、 ライブラリ への統一をお願いします

When you create projects this way, the file structure of the workspace is entirely consistent with the structure of the [workspace configuration file](guide/workspace-config), `angular.json`.
最初に明示的に生成されたアプリケーションは、ワークスペース内の他のすべてのプロジェクトと共に `projects/` フォルダーに入ります。
新しく生成されたライブラリも `projects/` の下に追加されています。
この方法でプロジェクトを作成すると、ワークスペースのファイル構造は、 [ワークスペース構成ファイル](guide/workspace-config) 、 `angle.json` の構造と完全に一致します。
Copy link
Collaborator

Choose a reason for hiding this comment

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

angular.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

おっと、なんでこんなミスを。。。ご指摘ありがとうございます 😃

@k-ta-yamada
Copy link
Contributor Author

コメント忘れてました。
修正完了しました 🙌

@lacolaco
Copy link
Collaborator

修正ありがとうございました!

@lacolaco lacolaco merged commit c71952b into angular:master Jun 16, 2019
k-neda added a commit to k-neda/angular-ja that referenced this pull request Jun 29, 2019
* master: (62 commits)
  chore: Sync origin 7fe95b188 (angular#408)
  translate: guide/workspace-config.md (angular#406)
  chore: update navigation.json
  translate: guide/app-shell (angular#403)
  translate: guide/file-structure (angular#404)
  ci: fix apt-get (angular#405)
  fix: typo start/index.md (angular#402)
  chore: navigation.jsonの更新
  chore: lint構成を刷新 (angular#393)
  translate: cli/usage-analytics-gathering.md (angular#391)
  translate: creating-libraries.md (angular#362)
  chore: update navigation.json
  translate: start/data.md (angular#397)
  chore: update navigation.json
  translate: guide/setup-localの翻訳 (angular#401)
  chore: update navigation.json
  fix: guide/structural directive (angular#399)
  translate: guide/structural-directives (angular#396)
  build(deps): bump js-yaml from 3.12.0 to 3.13.1 (angular#395)
  chore: update navigation.json
  ...

# Conflicts:
#	aio-ja/content/guide/glossary.md

glossary.mdのコンフリクト解消に加えて追加箇所と変更箇所の翻訳
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.

[PR募集] guide/file-structure の追加翻訳
3 participants