Skip to content

[CI] Add check-commit-message job#33

Merged
jinevening merged 1 commit intoSamsung:mainfrom
dayo09:0409-signoff
Apr 10, 2025
Merged

[CI] Add check-commit-message job#33
jinevening merged 1 commit intoSamsung:mainfrom
dayo09:0409-signoff

Conversation

@dayo09
Copy link
Contributor

@dayo09 dayo09 commented Apr 9, 2025

Let's check whether contributors are following TICO DCO.

TICO-DCO-1.0-Signed-off-by: Dayoung Lee dayoung.lee@samsung.com

#31

@dayo09 dayo09 marked this pull request as ready for review April 9, 2025 08:55
Let's check whether contributors are following TICO DCO.

TICO-DCO-1.0-Signed-off-by: Dayoung Lee <dayoung.lee@samsung.com>
Comment on lines +18 to +23
with:
# Checkout PR head commit
# Checkout Action use merge commit as default
ref: ${{ github.event.pull_request.head.sha }}
# Fetch all history and branch (default: 1)
fetch-depth: 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this really needed?

Suggested change
with:
# Checkout PR head commit
# Checkout Action use merge commit as default
ref: ${{ github.event.pull_request.head.sha }}
# Fetch all history and branch (default: 1)
fetch-depth: 0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mhs4670go Without that, github doesn't seem to pull the history log, thus it cannot get current commit message.

@dayo09 dayo09 requested a review from mhs4670go April 9, 2025 10:07
Copy link
Contributor

@mhs4670go mhs4670go left a comment

Choose a reason for hiding this comment

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

LGTM

@dayo09 dayo09 requested a review from a team April 10, 2025 01:09
@periannath
Copy link
Collaborator

Looks like Check PR workflow was failed for this PR. https://github.com/Samsung/TICO/actions/runs/14353138561?pr=33

Copy link
Contributor

@jinevening jinevening left a comment

Choose a reason for hiding this comment

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

LGTM

@jinevening jinevening merged commit 6c4076c into Samsung:main Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants