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

[ops] prettier のセットアップ #48

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

taniiicom
Copy link
Collaborator

@taniiicom taniiicom commented Aug 15, 2024

  • devcontainer レベルで有効になります
  • いったん以下の設定にしています
    {
      "printWidth": 120,
      "trailingComma": "all",
      "tabWidth": 2,
      "semi": true,
      "singleQuote": true,
      "jsxSingleQuote": true,
      "endOfLine": "lf"
    }
    
  • 好みの部分もあると思うので, 異論は大歓迎です 🙌

Copy link
Collaborator

@hikuohiku hikuohiku left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@woodnx woodnx left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@Atotti Atotti left a comment

Choose a reason for hiding this comment

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

LooksGoodTtMe(あんまりちゃんと見れてないけど、2人LGTMって言ってるし、LGTMでいっか~)

@woodnx
Copy link
Member

woodnx commented Aug 18, 2024

@taniiicom
ちなみになんですけど、opsってなんの略ですか?

@taniiicom
Copy link
Collaborator Author

operations です
「オペレーション改善(に関すること)」の意味ですmm

@taniiicom
Copy link
Collaborator Author

マージするときに同時にスタイルによる差分吸収するので, 作業してる人が少ないタイミングでマージします

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