Recached

lfit / releng-gerrit_to_platform
93%

DEFAULT BRANCH: main
Repo Added
Token ZJoTDr8Mj6eECbROcMN6YSvmUHVsRstjO regen
Build 241 Last
Files 8
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH main
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 93.214%. Remained the same
28169858206

push

github

askb 
Docs: document gerrit workflow filename matching and naming

The "GitHub Workflow Configuration" section stated only that a triggered
workflow filename "must contain 'gerrit'" and "must contain the search
filter". That is accurate but under-specified, and operators have been
surprised by three behaviours that follow directly from the matching
implementation in ``github.filter_workflows`` / ``filter_path``:

  * The test is ``path.lower().find(filter) >= 0`` -- a case-insensitive
    *substring* match anywhere in the filename, not a prefix or exact
    match.
  * gerrit-to-platform dispatches *every* workflow that matches an
    event, not just one. Any file whose name contains ``gerrit`` and the
    event filter fires; e.g. both ``gerrit-maven-merge.yaml`` and
    ``gerrit-release-merge.yaml`` run on every change-merged event.
  * For comment-added, the search filter is the value mapped to the
    keyword in ``[mapping "comment-added"]``, so the workflow filename
    must contain that mapped filter (not the keyword itself).

Document the recommended ``gerrit-<description>-<event>.yaml`` naming
convention, spell out the substring / all-matching-workflows-fire
semantics, add the resulting cautions (avoid unintended name collisions;
workflows that must not act on every event have to self-gate), and give a
worked comment-added example (``stage-release = stage`` ->
``gerrit-maven-stage.yaml``). Documentation-only change; no code or
behaviour is affected.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Change-Id: Iaa6e05c67
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

522 of 560 relevant lines covered (93.21%)

0.93 hits per line

Relevant lines Covered
560 RELEVANT LINES 522 COVERED LINES
0.93 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
28169858206 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
28169838509 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
28169853305 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
28169844367 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
28169833359 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
28169828325 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
27759180658 main Docs: document gerrit workflow filename matching and naming The "GitHub Workflow Configuration" section stated only that a triggered workflow filename "must contain 'gerrit'" and "must contain the search filter". That is accurate but under-specif... push askb github
93.21
27664983768 main Chore: Address easy aislop scan findings Remove redundant print() calls in exception handlers where log.exception() already captures the error. Replace print() with warnings.warn() in docs/conf.py for build failures. Remove trivial comment that r... push tykeal github
93.21
27664981911 main Chore: Address easy aislop scan findings Remove redundant print() calls in exception handlers where log.exception() already captures the error. Replace print() with warnings.warn() in docs/conf.py for build failures. Remove trivial comment that r... push tykeal github
93.21
27664589971 main Chore: Address easy aislop scan findings Remove redundant print() calls in exception handlers where log.exception() already captures the error. Replace print() with warnings.warn() in docs/conf.py for build failures. Remove trivial comment that r... push tykeal github
92.88
See All Builds (230)

Badge your Repo: releng-gerrit_to_platform

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh