Skip to content

chore: fine tune code and settings - #184

Merged
tisonkun merged 3 commits into
mainfrom
finetune
Aug 6, 2026
Merged

chore: fine tune code and settings#184
tisonkun merged 3 commits into
mainfrom
finetune

Conversation

@tisonkun

@tisonkun tisonkun commented Aug 6, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: tison <wander4096@gmail.com>
@tisonkun
tisonkun requested review from PsiACE and ZENOTME August 6, 2026 03:12
Comment thread .github/workflows/ci.yml
Comment on lines -22 to -23
push:
branches: [main]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Now all commits go through PR - need not to rerun CI in push.

Comment thread .asf.yaml
- Required
required_pull_request_reviews:
required_approving_review_count: 1
required_approving_review_count: 0

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

A committer can already leverage a contributor's PR to merge everything he/she wants.

As @gstein suggested, since we have VCS to hold the full history, we may prefer CTR. Releases still go under a vote.

Signed-off-by: tison <wander4096@gmail.com>

@PsiACE PsiACE left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@tisonkun
tisonkun merged commit 5f2e809 into main Aug 6, 2026
10 checks passed
@tisonkun
tisonkun deleted the finetune branch August 6, 2026 04:00
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