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

Feature/clang tidy #28

Merged
merged 8 commits into from
Dec 20, 2023
Merged

Feature/clang tidy #28

merged 8 commits into from
Dec 20, 2023

Conversation

nyxrobotics
Copy link

@nyxrobotics nyxrobotics commented Nov 1, 2023

Summary

  • sbgisenのコーディングルールの設定を複製
  • clang-tidy, clang-formatをかけました
  • 一部手作業で修正
  • 動作に変更はありません

Detail

つくばチャレンジの出場に伴って行った変更が多く、あとから見やすいようフォーマット・ファイル分割を掛けてから変更作業に移ろうと思っています
本来は本家のコーディングルールを使いたいのですがclang-formatの設定しかなく、clang-formatだけ元の書式にしてもあまり意味はないかなと思い全てsbgisenの設定にしました

forkしたリポジトリなのでciの設定はどうすべきか微妙だったのでgithubのチェックは走っていません

Impact

Test

Attention

@github-actions github-actions bot added the enhancement New feature or request label Nov 2, 2023
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@nyxrobotics

This comment was marked as resolved.

@nyxrobotics nyxrobotics force-pushed the feature/clang-tidy branch 2 times, most recently from 34b2db3 to 5d14708 Compare December 18, 2023 10:32
@nyxrobotics
Copy link
Author

リポジトリでキーの設定せずにciの設定だけコピーしたのでsshでコケていました
ciの設定は一旦後回しにしてフォーマットだけ掛けます

@nyxrobotics nyxrobotics marked this pull request as ready for review December 18, 2023 10:34
@jsupratman13
Copy link

@nyxrobotics
まだ編集中ですか?もし編集し続けるならdraftにしてもいいと思います.

@nyxrobotics
Copy link
Author

とりあえず作業完了です

@jsupratman13 jsupratman13 merged commit 40bdcdc into master Dec 20, 2023
@jsupratman13 jsupratman13 deleted the feature/clang-tidy branch December 20, 2023 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants