Skip to content

docs: clarify primary type labels are issues-only (PRs use ownership labels) (#278)#279

Merged
SamuelHassine merged 1 commit into
mainfrom
chore/labels-pr-clarification
Jun 7, 2026
Merged

docs: clarify primary type labels are issues-only (PRs use ownership labels) (#278)#279
SamuelHassine merged 1 commit into
mainfrom
chore/labels-pr-clarification

Conversation

@SamuelHassine

Copy link
Copy Markdown
Member

Summary

Clarifies the shared label taxonomy so it can never be misread (by a human or an automated reviewer):

  • Primary type labels (feature / bug / documentation) are applied to issues only, alongside the GitHub issue Type.
  • Pull requests do not carry a primary type label — the type: title prefix conveys the type. PRs should instead carry an ownership label (filigran team / community) to differentiate the author, plus any useful area/workflow labels.

Updated .github/LABELS.md, .github/labels.yml, and the CONTRIBUTING conventions block.

Closes #278

@SamuelHassine SamuelHassine merged commit 4c899de into main Jun 7, 2026
0 of 3 checks passed
@SamuelHassine SamuelHassine deleted the chore/labels-pr-clarification branch June 7, 2026 21:02
@github-actions github-actions Bot added the filigran team Item from the Filigran team. label Jun 7, 2026
@codecov

codecov Bot commented Jun 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.34%. Comparing base (0a48f46) to head (1cfef09).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #279   +/-   ##
=======================================
  Coverage   70.34%   70.34%           
=======================================
  Files          49       49           
  Lines        1966     1966           
=======================================
  Hits         1383     1383           
  Misses        583      583           
Flag Coverage Δ
connectors 70.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: clarify primary type labels are issues-only (PRs use ownership labels)

2 participants