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

Ci/add pre commit test (#8) #12

Closed

Conversation

masahiro-kubota
Copy link
Contributor

@masahiro-kubota masahiro-kubota commented Jan 7, 2025

Description

パッケージ単体リポジトリでのbuildとtestを実行するbuild-and-test workflowを追加しました(pre-commitに関しては後ほど別PR)。
workflowの追加にあたってリポジトリ内の構成変更を行っています。

pull requestが行われた際に、autowareで標準的に用いられている以下のアクションを利用したbuild-and-test workflowを実行をします。
https://github.com/autowarefoundation/autoware-github-actions/tree/main/colcon-build
https://github.com/autowarefoundation/autoware-github-actions/tree/main/colcon-test

Related Links

Test

pull requestを作成した際にbuild-and-test workflowが実行され、正常に完了していることを確認

masahiro-kubota#4
image

ビルドが失敗するときにbuildワークフローが失敗することを確認

masahiro-kubota#5
image

colcon testが失敗するときにtestワークフローが失敗することを確認

masahiro-kubota#6
image

リポジトリ内の構成変更の影響を調べるために、launchファイルからノードを起動できることを確認

image

* add pre commit setting

Signed-off-by: Masahiro Kubota <[email protected]>

* fix: files

Signed-off-by: Masahiro Kubota <[email protected]>

* add enter at the end of README

* add enter at the end of CMakeLists.txt

* modify pre-commit.yaml

* add enter at the end of pre-commit.yaml

Signed-off-by: Masahiro Kubota <[email protected]>

* modify pre-commit.yaml

* modify pre-commit.yaml

* style(pre-commit): autofix

* modify pre-commit.yaml

* style(pre-commit): autofix

* modify pre-commit.yaml

* style(pre-commit): autofix

---------

Signed-off-by: Masahiro Kubota <[email protected]>
Co-authored-by: masahiro-kubota <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@masahiro-kubota masahiro-kubota deleted the ci/add-pre-commit branch January 7, 2025 12:46
asa-naki pushed a commit that referenced this pull request Jan 10, 2025
* Ci/add commit test2 (#11)

* add-precommit

Signed-off-by: Masahiro Kubota <[email protected]>

* add precommit

Signed-off-by: Masahiro Kubota <[email protected]>

* modify

Signed-off-by: Masahiro Kubota <[email protected]>

* modify

Signed-off-by: Masahiro Kubota <[email protected]>

* modify

Signed-off-by: Masahiro Kubota <[email protected]>

* modify

Signed-off-by: Masahiro Kubota <[email protected]>

* delete redundant hook

Signed-off-by: Masahiro Kubota <[email protected]>

* add --fix arg

Signed-off-by: Masahiro Kubota <[email protected]>

* Revert "add --fix arg"

This reverts commit 4b06cca.

Signed-off-by: Masahiro Kubota <[email protected]>

* remove autoware-precommit

Signed-off-by: Masahiro Kubota <[email protected]>

* delete cache

Signed-off-by: Masahiro Kubota <[email protected]>

* modify precommit

Signed-off-by: Masahiro Kubota <[email protected]>

* modify precommit options

Signed-off-by: Masahiro Kubota <[email protected]>

* add config

Signed-off-by: Masahiro Kubota <[email protected]>

---------

Signed-off-by: Masahiro Kubota <[email protected]>

* add diff option (#12)

* add diff option

Signed-off-by: Masahiro Kubota <[email protected]>

* modify option

Signed-off-by: Masahiro Kubota <[email protected]>

* change fetch-depth

Signed-off-by: Masahiro Kubota <[email protected]>

* add eof break

Signed-off-by: Masahiro Kubota <[email protected]>

* pre-commit modification

Signed-off-by: Masahiro Kubota <[email protected]>

---------

Signed-off-by: Masahiro Kubota <[email protected]>

* pre-commit modification

Signed-off-by: Masahiro Kubota <[email protected]>

* add python hooks

Signed-off-by: Masahiro Kubota <[email protected]>

---------

Signed-off-by: Masahiro Kubota <[email protected]>
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.

1 participant