Skip to content

Commit 9910946

Browse files
docs: add data-analytics/general areas; update README, labels, labeler, CODEOWNERS, workflows, templates
1 parent d39918c commit 9910946

File tree

8 files changed

+50
-6
lines changed

8 files changed

+50
-6
lines changed

.github/CODEOWNERS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@ rfcs/platform/** @yatharthranjan @afolarin @baixiac @mpgxvii @pvannierop
1010
rfcs/backend/** @yatharthranjan @afolarin @mpgxvii @pvannierop @this-Aditya @ewelinagr
1111
rfcs/mobile/** @mpgxvii @this-Aditya @ec18130 @peyman-mohtashami
1212

13+
# New areas
14+
rfcs/data-analytics/** @afolarin @Hsankesara
15+
rfcs/general/** @yatharthranjan @afolarin @mpgxvii @pvannierop
16+
1317
# GitHub metadata and workflows
1418
.github/** @radar-base/admins
1519

.github/ISSUE_TEMPLATE/pre-rfc-discussion.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,19 @@ body:
3131
attributes:
3232
label: Stakeholders
3333
description: Teams/roles who should be involved
34+
- type: dropdown
35+
id: area
36+
attributes:
37+
label: Area
38+
description: Select the primary area for this discussion
39+
options:
40+
- platform
41+
- backend
42+
- mobile
43+
- data-analytics
44+
- general
45+
validations:
46+
required: true
3447
- type: checkboxes
3548
id: readiness
3649
attributes:

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Checklist
1212
- [ ] Added to index table in `README.md`
1313
- [ ] Considered alternatives and documented open questions
1414
- [ ] Applied appropriate status label (one of: draft, accepted, rejected, superseded, implemented)
15-
- [ ] Applied area/type labels as relevant (e.g., platform, backend, mobile, privacy; design, process)
15+
- [ ] Applied area/type labels as relevant (e.g., platform, backend, mobile, data-analytics, general, privacy; design, process)
1616

1717
Reviewers
1818
---------

.github/auto-assign.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,16 @@ filter:
3535
- ec18130
3636
- peyman-mohtashami
3737

38+
- label: "area: data-analytics"
39+
reviewers:
40+
- afolarin
41+
- Hsankesara
42+
43+
- label: "area: general"
44+
reviewers:
45+
- yatharthranjan
46+
- afolarin
47+
- mpgxvii
48+
- pvannierop
49+
3850

.github/labeler.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,10 @@ docs:
1212
"area: mobile":
1313
- rfcs/mobile/**
1414

15+
"area: data-analytics":
16+
- rfcs/data-analytics/**
17+
18+
"area: general":
19+
- rfcs/general/**
20+
1521

.github/labels.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,14 @@
4141
color: d93f0b
4242
description: Mobile apps and SDKs
4343

44+
- name: area: data-analytics
45+
color: 0e8a16
46+
description: Data pipelines, warehousing, BI, metrics, experimentation
47+
48+
- name: area: general
49+
color: 1f6feb
50+
description: Cross-cutting or domain-agnostic RFCs
51+
4452
- name: area: privacy
4553
color: 0b8769
4654
description: Security, privacy, compliance

.github/workflows/required-area-labels.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@ jobs:
1515
with:
1616
mode: minimum
1717
count: 1
18-
labels: "area: platform,area: backend,area: mobile"
18+
labels: "area: platform,area: backend,area: mobile,area: data-analytics,area: general"
1919
add_comment: true
2020
message: |
2121
This PR is missing an area label. Please add at least one of:
22-
area: platform | area: backend | area: mobile
22+
area: platform | area: backend | area: mobile | area: data-analytics | area: general
2323
2424

README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,11 @@ Quick start
99
-----------
1010
1. Fork this repository.
1111
2. Create a new branch for your RFC.
12-
3. Copy `rfcs/0000-template.md` to a new file `rfcs/NNNN-your-title.md` where `NNNN` is the next available RFC number. Place it under an area folder if applicable (e.g., `rfcs/platform/NNNN-your-title.md`).
12+
3. Copy `rfcs/0000-template.md` to a new file `rfcs/NNNN-your-title.md` where `NNNN` is the next available RFC number. Place it under an area folder if applicable (e.g., `rfcs/platform/NNNN-your-title.md`, `rfcs/backend/NNNN-your-title.md`, `rfcs/mobile/NNNN-your-title.md`, `rfcs/data-analytics/NNNN-your-title.md`, or `rfcs/general/NNNN-your-title.md`).
1313
4. Fill in all sections.
1414
5. Open a Pull Request using the provided template.
1515
6. Apply labels: one `status:` label and at least one `area:` label (see below). Labels are auto-suggested based on file paths.
16+
7. Link any related issues or discussions directly in the RFC body, such as Pre-RFC discussions (e.g., `RADAR-base/<repo>#<123>`) using the `Discussion` field in the RFC template.
1617

1718
Lifecycle
1819
---------
@@ -36,9 +37,9 @@ The RFC process is facilitated by the RADAR-base maintainers. See `CONTRIBUTING.
3637

3738
Areas and ownership
3839
-------------------
39-
- Area folders: `rfcs/platform/`, `rfcs/backend/`, `rfcs/mobile/`
40+
- Area folders: `rfcs/platform/`, `rfcs/backend/`, `rfcs/mobile/`, `rfcs/data-analytics/`, `rfcs/general/`
4041
- CODEOWNERS: PRs touching these folders auto-request domain reviewers
41-
- Labels: use one of `area: platform`, `area: backend`, `area: mobile`
42+
- Labels: use one of `area: platform`, `area: backend`, `area: mobile`, `area: data-analytics`, `area: general`
4243

4344
Status and label requirements
4445
-----------------------------

0 commit comments

Comments
 (0)