Skip to content

コーディングスタイルの統一、コード解析の自動化、全コードのフォーマット#33

Draft
Meatwo310 wants to merge 11 commits into
Valine3gDev:mainfrom
Meatwo310:main
Draft

コーディングスタイルの統一、コード解析の自動化、全コードのフォーマット#33
Meatwo310 wants to merge 11 commits into
Valine3gDev:mainfrom
Meatwo310:main

Conversation

@Meatwo310

@Meatwo310 Meatwo310 commented Apr 11, 2025

Copy link
Copy Markdown
Contributor

訳: ほとんどのコードがキレイキレイになります (fixes #22)

次の要素を導入します

.editorconfig

  • インデントサイズや末尾のスペースなど、重要なコーディングスタイルを統一します。

ESLint

  • コードを解析し、.editorconfigでカバーしきれない詳細なスタイルを統一します。
  • バグの発生しやすい、あるいは単に無駄な書き方を禁止し、コードの品質を向上させます。
  • このリポジトリにpushされたコードはすべて解析が行われます。
  • workflowを手動で実行すると、コードが自動で修正され、Actions Botによりコミットされます。

reviewdog

  • Pull RequestされたコードをESLintで解析し、修正リクエストをレビューとして自動で提出します。
    • レビュー書くだけで修正リクエスト出してくれなかったので消し飛ばしました

次の処理を行いました

  • *.jsファイルをESLintで自動修正 (npx eslint --fix)
  • 自動修正できなかったファイルを手動で修正しエラーや警告を解消

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot reviewed 58 out of 60 changed files in this pull request and generated 1 comment.

Files not reviewed (2)
  • .editorconfig: Language not supported
  • .vscode/extensions.json: Language not supported

Comment thread kubejs/server_scripts/1000mek_cables.js
@Meatwo310 Meatwo310 marked this pull request as ready for review April 11, 2025 18:05
@Meatwo310 Meatwo310 self-assigned this Apr 11, 2025
@Meatwo310

Copy link
Copy Markdown
Contributor Author

TODO: ExChillyから変更をバックポートする

@Meatwo310 Meatwo310 marked this pull request as draft April 16, 2025 06:36
@HardSmoothy HardSmoothy removed their request for review September 30, 2025 16:40
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.

pushされるたびに文法チェック/軽めのフォーマットを行う

3 participants