microsoft / microsoft/BCQuality
Prise instead of Info, or as an extra type
Nobody has claimed this yet.
- Dominant language
- AL
- Stars
- 212
- Forks
- 121
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 33
Description
In the contract introduced here, we have four types of comments: Blocker, Major, Minor, Info.
Once again, this is enough for just reviewing the code created by other AI Agents, but if we are talking about interaction with human developers, the possibility to point out a job well done, some exceptional solutions introduced, sounds simply better.
And I must say - it is quite well received by the developers.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the contract introduced by the change referenced in the issue and inspect where the four comment types are defined and consumed. Clarify whether the goal is to replace Info or add a separate positive type, then update the contract and its affected validation or documentation so the selected behavior is consistently represented.
Written by the indexing model from the issue text.
Assessment
- Domain
- developer-experience
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100