Skip to content

feat: replace the workflows withe github apps#291

Merged
san-ci[bot] merged 2 commits into
genai-io:mainfrom
hchenxa:hchenxa-test
Jul 9, 2026
Merged

feat: replace the workflows withe github apps#291
san-ci[bot] merged 2 commits into
genai-io:mainfrom
hchenxa:hchenxa-test

Conversation

@hchenxa

@hchenxa hchenxa commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

What & why

#253

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Docs
  • Other:

Testing

GOCACHE=/private/tmp/san-go-build-cache go test ./...

Checklist

  • Follows Conventional Commits (feat:, fix:, docs:, …)
  • All commits signed off (git commit -s) — certifies the DCO
  • Tests pass locally
  • Docs updated (if behavior or config changed)
  • Read and follow the Code of Conduct

@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

Hi @hchenxa! Based on the OWNERS file, I suggest requesting a review from @yanmxa.

@san-ci san-ci Bot added the need-rebase label Jul 9, 2026
@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

Hi @hchenxa! This PR needs to be rebased on the latest base branch. Please rebase your branch so it can be reviewed again. The lgtm label has been removed.

@hchenxa hchenxa changed the title this is used to test the github apps <WIP>this is used to test the github apps Jul 9, 2026
@san-ci san-ci Bot added the need-rebase label Jul 9, 2026
@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[REBASE NOTIFIER] Hi @hchenxa! This PR is behind the base branch and needs to be rebased.

The lgtm label has been removed and the reviewer will need to re-review after you fix this.

Details **What to do:** Run `git rebase main` locally, then force-push the updated branch.

After fixing:

  • The need-rebase label will be automatically removed
  • Request a new review from the OWNERS
  • The reviewer can add lgtm once satisfied

@san-ci san-ci Bot added the dco:true label Jul 9, 2026
@hchenxa hchenxa changed the title <WIP>this is used to test the github apps this is used to test the github apps Jul 9, 2026
@san-ci san-ci Bot removed the work-in-progress label Jul 9, 2026
@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[PR TITLE LINT] PR title does not follow Conventional Commits format.

Please update your PR title to follow Conventional Commits format:

type(scope): description

Allowed types:
feat | fix | docs | style | refactor | perf | test | build | ci | chore | revert

Examples:

feat: add user login
fix(api): handle null response
docs: update readme
refactor: extract auth middleware

You can edit the title at the top of this PR.

@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[DCO NOTIFIER] Hi @hchenxa! Some commits in this PR are missing a Signed-off-by.

What to do:

git commit --amend -s
git push --force

Or to sign off the most recent commit without changing the message:

git commit --amend --no-edit -s
git push --force

Once all commits are signed, the dco:no label will be automatically removed and replaced with dco:true.

hchenxa added 2 commits July 9, 2026 15:28
Signed-off-by: hchenxa <hchenxa1986@qq.com>
Signed-off-by: hchenxa <hchenxa1986@qq.com>
@san-ci san-ci Bot added dco:true and removed dco:no labels Jul 9, 2026
@hchenxa hchenxa changed the title this is used to test the github apps feat: this is used to test the github apps Jul 9, 2026
@hchenxa

hchenxa commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

/assign @yanmxa

@san-ci san-ci Bot assigned yanmxa Jul 9, 2026
@hchenxa hchenxa changed the title feat: this is used to test the github apps feat: replace the workflows withe github apps Jul 9, 2026
@yanmxa

yanmxa commented Jul 9, 2026

Copy link
Copy Markdown
Member

/lgtm

@san-ci san-ci Bot added the lgtm label Jul 9, 2026
@san-ci

san-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR has the lgtm label but is NOT APPROVED yet.

Please request an approving review from @yanmxa (see OWNERS).

Once approved, the Review Gate will automatically squash-merge this PR.

@yanmxa

yanmxa commented Jul 9, 2026

Copy link
Copy Markdown
Member

/approve

@san-ci san-ci Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved

@san-ci san-ci Bot merged commit efe8468 into genai-io:main Jul 9, 2026
8 of 9 checks passed
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.

2 participants