-
Notifications
You must be signed in to change notification settings - Fork 142
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
docs(how-to-guides): add example procedure, case of localization #410
docs(how-to-guides): add example procedure, case of localization #410
Conversation
Signed-off-by: Shintaro Sakoda <[email protected]>
Signed-off-by: Shintaro Sakoda <[email protected]>
ce14249
to
2abb247
Compare
Signed-off-by: Shintaro Sakoda <[email protected]>
@mitsudome-r @kenji-miyake |
Documentation URL: https://autowarefoundation.github.io/autoware-documentation/pr-410/ |
I will review today! I aplogize for the delay. 🙇 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SakodaShintaro Sorry for keeping you waiting. I have made some comments and suggestions, please check them out!
docs/how-to-guides/others/an-example-procedure-for-adding-a-new-node.md
Outdated
Show resolved
Hide resolved
docs/how-to-guides/others/an-example-procedure-for-adding-a-new-node.md
Outdated
Show resolved
Hide resolved
docs/how-to-guides/others/an-example-procedure-for-adding-a-new-node.md
Outdated
Show resolved
Hide resolved
docs/how-to-guides/others/an-example-procedure-for-adding-a-new-node.md
Outdated
Show resolved
Hide resolved
docs/how-to-guides/others/an-example-procedure-for-adding-a-new-node.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Shintaro SAKODA <[email protected]>
…in index.md Signed-off-by: Shintaro SAKODA <[email protected]>
@KYabuuchi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks mostly good. There is one old title that needs to be fixed.
I will approve it when it is fixed.
…w node" Signed-off-by: Shintaro SAKODA <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
…owarefoundation#410) * add example procedure, case of localization Signed-off-by: Shintaro Sakoda <[email protected]> * style(pre-commit): autofix Signed-off-by: Shintaro Sakoda <[email protected]> * add language specifier Signed-off-by: Shintaro Sakoda <[email protected]> * Updated an-example-procedure-for-adding-and-evaluating-a-new-node.md Signed-off-by: Shintaro SAKODA <[email protected]> * Updated an-example-procedure-for-adding-and-evaluating-a-new-node.md in index.md Signed-off-by: Shintaro SAKODA <[email protected]> * Updated index of "An example procedure for adding and evaluating a new node" Signed-off-by: Shintaro SAKODA <[email protected]> --------- Signed-off-by: Shintaro Sakoda <[email protected]> Signed-off-by: Shintaro SAKODA <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Signed-off-by: guiping meng <[email protected]>
Description
PR for #400, which add document about procedure of introducing a new node of localization.
The added pages leads to the necessary information rather than adding new information.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.