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

Fix/#63 degrade clippy #64

Closed
wants to merge 2 commits into from
Closed

Fix/#63 degrade clippy #64

wants to merge 2 commits into from

Conversation

comavius
Copy link
Collaborator

@comavius comavius commented Oct 7, 2024

関連Issue

概要

lintより他の事柄に関心を向けたいため、当面はclippyは努力義務として運用したい。

変更内容

  • workflowからclippy lintを削除
  • prのテンプレートにclippy lintの結果を確認するよう促すチェックボックスを追加

チェックリスト

  • テストが通っている
  • 下記のいずれかを満たしている
      • 変更点についてテストを追加/修正した
      • テストを追加するissueを立てた
      • テストが不要な変更である
  • レビュワーを指定した
  • タグをつけた

補足

lintはしたほうが良いのでclippyをうまいこと使いこなせるようになったら再導入したい。

@comavius comavius added the ci/test Relating to CI or test label Oct 7, 2024
@comavius comavius self-assigned this Oct 7, 2024
@comavius
Copy link
Collaborator Author

comavius commented Oct 8, 2024

ごめん必要なさそう

@comavius comavius closed this Oct 8, 2024
@kenken714
Copy link
Collaborator

@comavius これ、releaseじゃない方だけ切るのは良いかもって思った(warningがない時点でCIの意味が薄れているため)
逆に [ ] cargo clippyの結果を確認し、適宜修正を入れた は入れて欲しいかも

@kenken714
Copy link
Collaborator

(closeした後でごめん)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/test Relating to CI or test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants