Skip to content

docs(rules): take the highest ranked Goal your lead gives you - #136

Merged
zolotokrylin merged 1 commit into
mainfrom
docs/goal-priority-source
Jul 30, 2026
Merged

docs(rules): take the highest ranked Goal your lead gives you#136
zolotokrylin merged 1 commit into
mainfrom
docs/goal-priority-source

Conversation

@zolotokrylin

@zolotokrylin zolotokrylin commented Jul 30, 2026

Copy link
Copy Markdown
Member

DEV-110 told contributors to pick a Goal by their own read of impact and urgency, and to ask their lead if unsure. For anyone working across several repos that is a guess, and it ignores that a lead may already have decided the order.

The rule now takes the ranked order as the answer where one is given, and keeps the impact-and-urgency fallback for contributors who have none. It says nothing about where that order is kept, which is deliberate: that is the lead's business, not the contribution model's.

@holdex

holdex Bot commented Jul 30, 2026

Copy link
Copy Markdown

Time Submission Status

Member # Time Running Total Status Last Update
zolotokrylin 7min ✅ Submitted Jul 30, 2026, 4:01 AM

Submit or update total time with:

@holdex pr submit-time 2h

Add time on top of previous submission with:

@holdex pr add-time 1h30m

See available commands to help comply with our Guidelines.

@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

DEV-110 now directs readers to select the next Goal from their ranked Priorities issue, with a fallback for repositories without one.

Changes

Goal ownership guidance

Layer / File(s) Summary
Goal selection instructions
docs/rules/DEV-110.md
Adds Priorities-issue selection and a Goal: issue fallback with impact, urgency, and lead consultation guidance.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main documentation change about choosing the next ranked Goal, though it is a bit imprecise about the source.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/goal-priority-source

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@zolotokrylin
zolotokrylin force-pushed the docs/goal-priority-source branch from f5d337f to 1520b85 Compare July 30, 2026 04:00
@zolotokrylin zolotokrylin changed the title docs(rules): take the next Goal from a ranked priorities list docs(rules): take the highest ranked Goal your lead gives you Jul 30, 2026

@zolotokrylin zolotokrylin left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@holdex pr submit-time 7m

@zolotokrylin
zolotokrylin merged commit 0d14176 into main Jul 30, 2026
3 checks passed
@zolotokrylin
zolotokrylin deleted the docs/goal-priority-source branch July 30, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant