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.

Sign up for GitHub

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

Jump to bottom

feat: add colcon-build and colcon-test actions #49

Merged
merged 54 commits into from
Feb 10, 2022
Merged

feat: add colcon-build and colcon-test actions #49

merged 54 commits into from
Feb 10, 2022

Conversation

KeisukeShima
Copy link
Contributor

Related autowarefoundation/autoware-documentation#2 (comment)

I added colcon-labeled-test workflow to run colcon test for each test label and generate code coverage results.
We can use build cache from colcon-build-and-test workflow, which means that we only need to run the build once.
Also, the output variable named coverage-reports contains the path to the coverage report, we can be used as input for codecov/codecov-action.

Signed-off-by: Keisuke Shima 19993104+KeisukeShima@users.noreply.github.com

@KeisukeShima
feat: add colcon-labeled-test workflow
3f04028 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima KeisukeShima self-assigned this Feb 4, 2022
@KeisukeShima KeisukeShima mentioned this pull request Feb 4, 2022
Merged
@KeisukeShima KeisukeShima changed the title feat(colcon-labeled-test): add colcon-labeled-test workflow feat(colcon-build, colcon-test): add colcon-build and colcon-test workflow Feb 4, 2022
@KeisukeShima
Copy link
Contributor Author

I will test the new workflow on my fork repo.

@kenji-miyake kenji-miyake changed the title feat(colcon-build, colcon-test): add colcon-build and colcon-test workflow feat: add colcon-build and colcon-test actions Feb 4, 2022
kenji-miyake
kenji-miyake reviewed Feb 4, 2022
View reviewed changes
colcon-build/README.md Outdated Show resolved Hide resolved
colcon-test/README.md Outdated Show resolved Hide resolved
colcon-test/README.md Outdated Show resolved Hide resolved
colcon-test/README.md Outdated Show resolved Hide resolved
colcon-build/action.yaml Outdated Show resolved Hide resolved
colcon-test/action.yaml Show resolved Hide resolved
colcon-test/action.yaml Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
kenji-miyake
kenji-miyake reviewed Feb 6, 2022
View reviewed changes
colcon-build/README.md Outdated Show resolved Hide resolved
KeisukeShima
KeisukeShima commented Feb 7, 2022
View reviewed changes
colcon-build/README.md Outdated Show resolved Hide resolved
@KeisukeShima KeisukeShima mentioned this pull request Feb 7, 2022
Closed
KeisukeShima
KeisukeShima commented Feb 7, 2022
View reviewed changes
colcon-build/README.md Outdated Show resolved Hide resolved
@KeisukeShima KeisukeShima enabled auto-merge (squash) February 7, 2022 09:29
kenji-miyake
kenji-miyake reviewed Feb 7, 2022
View reviewed changes
colcon-build/README.md Outdated Show resolved Hide resolved
colcon-test/README.md Outdated Show resolved Hide resolved
colcon-build/action.yaml Outdated Show resolved Hide resolved
colcon-build/action.yaml Outdated Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
colcon-test/README.md Outdated Show resolved Hide resolved
colcon-test/action.yaml Outdated Show resolved Hide resolved
kenji-miyake
kenji-miyake reviewed Feb 7, 2022
View reviewed changes
colcon-test/README.md Outdated Show resolved Hide resolved
kenji-miyake
kenji-miyake reviewed Feb 7, 2022
View reviewed changes
colcon-test/README.md Outdated Show resolved Hide resolved
KeisukeShima
KeisukeShima commented Feb 7, 2022
View reviewed changes
colcon-test/README.md Outdated Show resolved Hide resolved
@KeisukeShima KeisukeShima enabled auto-merge (squash) February 7, 2022 09:53
kenji-miyake
kenji-miyake reviewed Feb 7, 2022
View reviewed changes
colcon-test/README.md Outdated Show resolved Hide resolved
kenji-miyake
kenji-miyake reviewed Feb 7, 2022
View reviewed changes
colcon-test/README.md Outdated Show resolved Hide resolved
kenji-miyake
kenji-miyake previously approved these changes Feb 7, 2022
View reviewed changes
Copy link
Contributor

@kenji-miyake kenji-miyake left a comment

Choose a reason for hiding this comment

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

👍

kenji-miyake and others added 19 commits February 7, 2022 19:59
@kenji-miyake @KeisukeShima
feat(sync-branches): add changelog to body (autowarefoundation#55)
ead1927 
Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@kenji-miyake @KeisukeShima
fix(generate-changelog): remove CHANGELOG.md (autowarefoundation#56)
63a9626 
* style: fix yamllint

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* fix(generate-changelog): remove CHANGELOG.md

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@kenji-miyake @KeisukeShima
ci: add GitHub release workflow (autowarefoundation#38)
bc130b0 
* chore: update cliff.toml

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>

* ci: add GitHub release workflow

Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-build/README.md
a3529d2 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-test/README.md
143cb3d 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-build/action.yaml
b1de609 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-build/action.yaml
ec086a1 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-test/action.yaml
15f26a0 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-test/action.yaml
f719f48 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-test/README.md
0a6229c 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@pre-commit-ci @KeisukeShima
ci(pre-commit): autofix
91fd4b9 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima @kenji-miyake
Update colcon-test/README.md
9fe89e8 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Co-authored-by: Kenji Miyake <31987104+kenji-miyake@users.noreply.github.com>
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
fix directory name of the result
c5db608 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
Update colcon-test/README.md
c2a55bf 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
Update colcon-test/README.md
d62d628 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
modify argument name
ee28477 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
change if statement
000879c 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@KeisukeShima
change output name
ff50b8a 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
@pre-commit-ci @KeisukeShima
ci(pre-commit): autofix
fdb25cf 
Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>
This was referenced Feb 7, 2022
Open
Merged
@KeisukeShima KeisukeShima merged commit 660577f into autowarefoundation:tier4/proposal Feb 10, 2022
@awf-autoware-bot awf-autoware-bot bot mentioned this pull request Feb 11, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kenji-miyake kenji-miyake kenji-miyake approved these changes

Assignees

@KeisukeShima KeisukeShima

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@KeisukeShima @kenji-miyake