Skip to content

Create commit-check.yml#13

Closed
Yashasvi Nancherla (ynancher) wants to merge 1 commit into
qualcomm-linux:mainfrom
ynancher:ynancher-kgsl
Closed

Create commit-check.yml#13
Yashasvi Nancherla (ynancher) wants to merge 1 commit into
qualcomm-linux:mainfrom
ynancher:ynancher-kgsl

Conversation

@ynancher

Copy link
Copy Markdown

Adds the commit-msg-check GitHub Action
Checks commit messages added in pull requests
Valid for any PRs raised on this repository
Enforces these checks:

Commit Subject: Must exist and be ≤ 50 characters
Commit Body: Each line must wrap at 72 characters
Blank Line Check: Ensures a blank line between subject and body Also checks for blank line before Signed-off-by

Adds the commit-msg-check GitHub Action
Checks commit messages added in pull requests
Valid for any PRs raised on this repository
Enforces these checks:

Commit Subject: Must exist and be ≤ 50 characters
Commit Body: Each line must wrap at 72 characters
Blank Line Check: Ensures a blank line between subject and body
Also checks for blank line before Signed-off-by

Signed-off-by: Yashasvi Nancherla <ynancher@qti.qualcomm.com>
@shivrawa

Copy link
Copy Markdown
Contributor

Hi Yashasvi Nancherla (@ynancher)
These checks are ensured by the LBSA(Linux build service account) on the Gerrit.
we don't upload PR/code changes directly to the KGSL GitHub repo.

So i believe we don't need this change and we can close this PR. Yashasvi Nancherla (@ynancher) Let me know your view also on this.

@ynancher

Copy link
Copy Markdown
Author

Hi Yashasvi Nancherla (@ynancher) These checks are ensured by the LBSA(Linux build service account) on the Gerrit. we don't upload PR/code changes directly to the KGSL GitHub repo.

So i believe we don't need this change and we can close this PR. Yashasvi Nancherla (@ynancher) Let me know your view also on this.

Sure Shivam Rawat (@shivrawa) , makes sense

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.

2 participants