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.