You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by wu...@apache.org on 2026/09/23 15:46:06 UTC
(skywalking-ai-sessionizer) branch require-ci created (now 6d9dc28)
This is an automated email from the ASF dual-hosted git repository.
wu-sheng pushed a change to branch require-ci
in repository https://gitbox.apache.org/repos/asf/skywalking-ai-sessionizer.git
at 6d9dc28 Require the CI check and an up-to-date branch to merge into main
This branch includes the following new commits:
new 6d9dc28 Require the CI check and an up-to-date branch to merge into main
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
(skywalking-ai-sessionizer) 01/01: Require the CI check and an up-to-date branch to merge into main
Posted by wu...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
wu-sheng pushed a commit to branch require-ci
in repository https://gitbox.apache.org/repos/asf/skywalking-ai-sessionizer.git
commit 6d9dc2822007e86e0405534a884e7e37d0d8bf10
Author: Wu Sheng <wu...@foxmail.com>
AuthorDate: Wed Sep 23 23:45:56 2026 +0800
Require the CI check and an up-to-date branch to merge into main
main had no protection. The CI workflow already fans every job into one
check, Required, but nothing enforced it. .asf.yaml now requires that check
to pass on a branch that is up to date with main, and shows the Update
branch button on a pull request that is behind. No review is required: the
project has one committer.
---
.asf.yaml | 17 +++++++++++++----
docs/en/guides/contributing.md | 4 +++-
2 files changed, 16 insertions(+), 5 deletions(-)
diff --git a/.asf.yaml b/.asf.yaml
index 57b9c3c..21a321b 100644
--- a/.asf.yaml
+++ b/.asf.yaml
@@ -33,7 +33,16 @@ github:
merge: false
rebase: false
dependabot_updates: true
- # No branch protection and no required review. The project is new and has one
- # committer, so there is nobody to review a pull request. Null is the form the
- # ASF documentation gives for removing every protection rule.
- protected_branches: ~
+ # A pull request merges only when the Required check passes on a branch that
+ # is up to date with main. Required is the one CI job that every other job
+ # fans in to. No review is required: the project has one committer, so there
+ # is nobody to review a pull request.
+ protected_branches:
+ main:
+ required_status_checks:
+ strict: true
+ contexts:
+ - Required
+ # Shows the Update branch button on a pull request that is behind main.
+ pull_requests:
+ allow_update_branch: true
diff --git a/docs/en/guides/contributing.md b/docs/en/guides/contributing.md
index 2218030..2f40f67 100644
--- a/docs/en/guides/contributing.md
+++ b/docs/en/guides/contributing.md
@@ -29,7 +29,9 @@ side.
## Before opening a pull request
-- `make check` passes. CI runs the same targets and one required check fans them in.
+- `make check` passes. CI runs the same targets and one required check, `Required`, fans them in.
+ A pull request merges only when that check passes on a branch that is up to date with `main`.
+ Use the Update branch button when it is behind. No review is required.
- Record each change users can see in `docs/en/changes/changes.md`, the changelog of the version
in development, under the heading of its area.
- Every source file carries the Apache license header. `make license-fix` inserts missing ones.