envoyproxy / envoyproxy/envoy

RepoKitteh-based PR checklists

Open
#12,949 13 comments 0 reactions 1 assignee Claimed by @siddharthkhonde View on GitHub
area/repokitteh beginner enhancement help wanted
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

We can have RK automatically label any change to `source/` or `api/` without a change to `current.rst` with a `release-notes` blocker, requiring manual confirmation from maintainers that the release notes step has been satisfied. Similarly this could work for documentation checks, i.e. validate that at least some change has happened in `docs/`.

I'd propose that we take the existing `ownerscheck.star` and add a path expression that suppresses a given check. That should be all that is needed for release notes or docs.

@envoyproxy/maintainers are there other common review checklist items that RK could help with?
CC @itayd

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.