Skip to content

docs: document review-guidelines skill for code review#700

Merged
factory-nizar merged 1 commit intomainfrom
docs/review-guidelines-skill
Mar 3, 2026
Merged

docs: document review-guidelines skill for code review#700
factory-nizar merged 1 commit intomainfrom
docs/review-guidelines-skill

Conversation

@factory-nizar
Copy link
Contributor

Updates the code review docs to replace the manual prompt editing section with the new .factory/skills/review-guidelines/SKILL.md approach.

Ref: Factory-AI/droid-action#49

@factory-droid
Copy link
Contributor

factory-droid bot commented Mar 3, 2026

Droid finished @factory-nizar's task —— View job


Review complete.

  • PR diff reviewed (docs only)
  • No P0/P1 issues found

Replace the manual workflow prompt editing section with the new
.factory/skills/review-guidelines/SKILL.md approach.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@factory-nizar factory-nizar force-pushed the docs/review-guidelines-skill branch from a8f355b to 150e337 Compare March 3, 2026 17:56
Copy link
Contributor

@factory-droid factory-droid bot left a comment

Choose a reason for hiding this comment

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

Reviewed the diff; the docs update looks consistent with existing skill path conventions (.factory/skills/<name>/SKILL.md). No P0/P1 issues found.

@factory-nizar factory-nizar merged commit 01742a8 into main Mar 3, 2026
3 checks passed
@factory-nizar factory-nizar deleted the docs/review-guidelines-skill branch March 3, 2026 19:22
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.

2 participants