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

docs: fix typo #11

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

seigot
Copy link

@seigot seigot commented Jun 3, 2024

チームメンバーから、"run_autoware.sh" が失敗し、代わりに "run_autoware.bash" を実行すると成功したとの情報があり、
一旦 pull request させてもらいました。
ただ何が正しいかよくわかっておらず、もし認識ズレていましたらpull request破棄して頂いて構いません。

fix from "run_autoware.sh" to "run_autoware.bash"
@taikitanaka3
Copy link
Contributor

taikitanaka3 commented Jun 3, 2024

@seigot
ありがとうございます!

https://github.com/AutomotiveAIChallenge/aichallenge-2024/tree/main/aichallenge

こちら確かにご指摘のとおり.bashファイルのみです。
lgtmです!
もしよければfork元のaichallenge documentationに同様のPRを出していただけないでしょうか?

Copy link
Contributor

@masahiro-kubota masahiro-kubota left a comment

Choose a reason for hiding this comment

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

LGTMです!

@masahiro-kubota masahiro-kubota merged commit a5fedc6 into AutomotiveAIChallenge:main Jun 4, 2024
1 check passed
@seigot
Copy link
Author

seigot commented Jun 4, 2024

ありがとうございます!

以下を試みようとしたのですが、よくみたら"aichallenge documentation 2024"は"aichallenge documentation"からforkされている構造であることに気づきました。
私がfork元の"aichallenge documentation"に直接pull requestすると将来conflictする可能性を感じ、同様のPRを避けた方が良いかも、と思って現状特に何もしないままにしています。
(aichallenge documentation 2024、aichallenge documentation にmergeする、もしくはその逆の際に同じコミット同士でconflictする可能性)

もしよければfork元のaichallenge documentationに同様のPRを出していただけないでしょうか?

スクリーンショット 2024-06-04 8 57 19

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.

3 participants