chore: align with Filigran Conventional Commits & label conventions (#168)#169
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Aligns the repository’s contribution workflow with Filigran-wide conventions: Conventional Commits-style titles (with (#issue) for PRs), a shared label taxonomy/palette, and updated issue templates.
Changes:
- Added contribution guidelines describing Filigran title/label conventions.
- Introduced canonical shared label taxonomy (
.github/LABELS.md) and machine-readable label manifest (.github/labels.yml). - Updated GitHub issue templates to prefill titles with
feat:/fix:and apply shared triage/type labels.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
CONTRIBUTING.md |
New contributor guidance, including Filigran title/label conventions. |
.github/labels.yml |
Adds canonical Filigran shared label palette (name/color/description). |
.github/LABELS.md |
Documents the title + label taxonomy and how to apply labels consistently. |
.github/ISSUE_TEMPLATE/feature_request.md |
Prefills feature request titles (feat:) and applies feature + needs triage. |
.github/ISSUE_TEMPLATE/bug_report.md |
Prefills bug report titles (fix:) and applies bug + needs triage. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+29
to
+36
| * **Titles** — All commit, pull request and issue titles follow the | ||
| [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) | ||
| specification with a GitHub issue reference: | ||
| `type(scope?)!?: description (#issue)` (e.g. | ||
| `feat(api): add bulk export endpoint (#1234)`). The description starts with a | ||
| lowercase letter and has no trailing period; preserve acronyms and proper | ||
| nouns. Types: `feat`, `fix`, `chore`, `docs`, `style`, `refactor`, `perf`, | ||
| `test`, `build`, `ci`, `revert`. |
Comment on lines
+14
to
+16
| Every commit, pull request and issue title follows the | ||
| [Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/) | ||
| specification, with a GitHub issue reference appended: |
| <!-- | ||
| - Why is this solution needed? | ||
| - What value or benefits will the end-users gain from this change? | ||
| - Is this change a technical improvement? just for internal use, or does it impact users as well? |
| ## Environment | ||
| 1. OS (where OpenAEV server runs): { e.g. Mac OS 10, Windows 10, Ubuntu 16.4, etc. } |
| <!-- Please describe what actually happened. --> | ||
| ## Additional information |
Comment on lines
+42
to
+45
| * **GitHub reference** — Pull request titles **must** end with the related issue | ||
| reference, e.g. `(#1234)` (the PR title becomes the squash-merge commit). Every | ||
| pull request must be linked to an issue. Enforcement is preventive and applied | ||
| at the organization level; **Renovate** pull requests are exempt. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Aligns this repository with the Filigran-wide contribution conventions (Conventional Commits +
(#issue)), the shared label palette and issue/PR templates.What changed
github/ISSUE_TEMPLATE/bug_report.md.github/ISSUE_TEMPLATE/feature_request.md.github/LABELS.md.github/labels.ymlCONTRIBUTING.mdThe shared label colors are also synced to the canonical palette.
Closes #168