Skip to content

Conversation

@Tiryoh
Copy link
Contributor

@Tiryoh Tiryoh commented Jul 7, 2021

What does this implement/fix?

  • clang-formatが適用されているかを確認します
  • rtmouse.cのコメント内のバージョンとMODULE_VERSIONが一致しているかを確認します

Does this close any currently open issues?

しません

How has this been tested?

本PR内で確認しています。

clang-format未適用のままpushする
https://github.com/rt-net/RaspberryPiMouse/runs/3006491277
Screenshot from 2021-07-15 16-08-28

rtmouse.cのコメント内のバージョンとMODULE_VERSIONが一致していない状態でpushする
https://github.com/rt-net/RaspberryPiMouse/runs/3006508043

Screenshot from 2021-07-15 16-09-37

Any other comments?

Checklists

  • I have read the CONTRIBUTING guidelines.
  • I have checked to ensure there aren't other open Pull Requests for the same change.

@Tiryoh Tiryoh added the Type: CI Changes to CI configuration files and scripts label Jul 9, 2021
@Tiryoh Tiryoh marked this pull request as ready for review July 15, 2021 07:10
@Tiryoh Tiryoh requested a review from ShotaAk July 15, 2021 07:10
@ShotaAk ShotaAk self-assigned this Jul 16, 2021
- '.github/workflows/driver-cross-build.yml'

jobs:
build:
Copy link
Contributor

Choose a reason for hiding this comment

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

Lintをワークフローではなくジョブにして、Lintが失敗したらクロスビルドをキャンセルするように変更するのはどうでしょうか?

Copy link
Contributor Author

@Tiryoh Tiryoh Jul 19, 2021

Choose a reason for hiding this comment

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

1つのワークフローにまとめてLint失敗時はビルドテストしないようにしました。

https://github.com/rt-net/RaspberryPiMouse/runs/3100729669

Screenshot from 2021-07-19 13-58-22
Screenshot from 2021-07-19 14-00-31

@Tiryoh Tiryoh requested a review from ShotaAk July 19, 2021 05:02
Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

lintのジョブ化ありがとうございます。動作良いと思います。

@ShotaAk ShotaAk merged commit 8c36a6c into master Jul 19, 2021
@ShotaAk ShotaAk deleted the feature/add-lint branch July 19, 2021 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: CI Changes to CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants