mirror of
https://github.com/microsoft/BCQuality.git
synced 2026-08-06 17:36:53 +01:00
Add Gotcha warning: taskNo vs taskId confusion at commit time
Users refer to tasks by taskNo in conversation (e.g. 'opgave 51'). Commit messages must use taskId (e.g. 8738) — different field. Added explicit Gotcha block and sharpened step 3 in How to find. Approved by: mid (Michael Dieringer)
This commit is contained in:
parent
34d8f372f8
commit
e11c1fd16c
1 changed files with 8 additions and 3 deletions
|
|
@ -1,4 +1,4 @@
|
|||
---
|
||||
---
|
||||
name: commit-message-must-include-bc-task-id
|
||||
description: >
|
||||
Every commit message must begin with the BC task ID in [#id] format,
|
||||
|
|
@ -47,11 +47,16 @@ The sub-task has two numbers — do not confuse them:
|
|||
Always use `taskId` in commit messages. It is unambiguous across all projects
|
||||
and repos.
|
||||
|
||||
> **Gotcha:** Users and conversations refer to tasks by `taskNo` — e.g. "opgave 51"
|
||||
> or "task 42". This is the natural shorthand and is correct for conversation.
|
||||
> But `taskNo` is NOT what goes in the commit message. Always look up `taskId`
|
||||
> from the MCP response before committing — they are different fields.
|
||||
|
||||
## How to find the taskId before committing
|
||||
|
||||
1. Get the current branch: `git branch --show-current`
|
||||
2. Find the linked project via BC MCP (see `[[bc-mcp-find-active-task-for-branch]]`)
|
||||
3. Read `taskId` from the matching active task
|
||||
3. In the MCP response, read the **`taskId`** field — NOT `taskNo`
|
||||
4. Prefix every commit on this branch with `[#taskId]`
|
||||
|
||||
If no task exists for the branch, create one first (see bc-mcp.agent.md
|
||||
|
|
@ -60,4 +65,4 @@ create-task workflow) or ask the project manager to register the work.
|
|||
## Scope
|
||||
|
||||
All commits that reach the main branch — feature, fix, test, chore, docs.
|
||||
Merge commits and auto-generated commits (renovate, al-go) are exempt.
|
||||
Merge commits and auto-generated commits (renovate, al-go) are exempt.
|
||||
Loading…
Add table
Add a link
Reference in a new issue