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

Stringsの修正,DarkColorのオプションの追加 #226

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

areteruhiro
Copy link
Collaborator

@areteruhiro areteruhiro commented Dec 15, 2024

確認項目

  • 動作確認済み
  • 誤字脱字無し

説明

@areteruhiro areteruhiro changed the title Stringsの修正 Stringsの修正,DarkColorのオプションの追加 Dec 15, 2024
This was referenced Dec 15, 2024
Copy link
Collaborator

@s1204IT s1204IT left a comment

Choose a reason for hiding this comment

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

前も言いましたが、ブランチ名に日本語を用いるのはGit上でバグを引き起こす原因になりかね無いので辞めてください。

また、変数名は一貫性を意識してください。

@areteruhiro
Copy link
Collaborator Author

前も言いましたが、ブランチ名に日本語を用いるのはGit上でバグを引き起こす原因になりかね無いので辞めてください。

また、変数名は一貫性を意識してください。

一個一個の機能を別々でPRしていきたいんですが、方法はないのでしょうか?

@areteruhiro
Copy link
Collaborator Author

変数名とかっていうのは、どのあたりになりますか?

@areteruhiro
Copy link
Collaborator Author

ブランチ確かに日本語文字化けしちゃいますもんね、、すみません

@s1204IT
Copy link
Collaborator

s1204IT commented Dec 15, 2024

変数名とかっていうのは、どのあたりになりますか?

Screenshot_20241215_222818_Chrome
これであれば、小文字でかつ単語ごとにアンダーバーをつけるべきです。

全て単語ごとに先頭を大文字にするか、どちらかに揃えた方が良いです。
ただ、Android標準の書き方としては、前者の単語ごとにアンダーバーを付けるスタイルです。

@s1204IT
Copy link
Collaborator

s1204IT commented Dec 15, 2024

前も言いましたが、ブランチ名に日本語を用いるのはGit上でバグを引き起こす原因になりかね無いので辞めてください。
また、変数名は一貫性を意識してください。

一個一個の機能を別々でPRしていきたいんですが、方法はないのでしょうか?

master ブランチを追加したい機能の分だけ複製し、プルリク。
競合は別に解決できるので気にしなくて大丈夫です。

一番確実なのは、一つ一つの機能をマージしてから新たにプルリクを出すことです。

@areteruhiro
Copy link
Collaborator Author

修正して、PRし直します

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