Skip to content

Comments

Revert part of "docs: Update code review documentation with PR review workflow"#311

Merged
xingyaoww merged 3 commits intomainfrom
revert-302-openhands/update-code-review-docs
Feb 8, 2026
Merged

Revert part of "docs: Update code review documentation with PR review workflow"#311
xingyaoww merged 3 commits intomainfrom
revert-302-openhands/update-code-review-docs

Conversation

@xingyaoww
Copy link
Contributor

@xingyaoww xingyaoww commented Feb 8, 2026

Reverts #302

It accidentally includes content from Joe's branch that's not ready yet - this PR revert the parts that are not yet ready

@xingyaoww
Copy link
Contributor Author

@OpenHands please revert all document except sdk/guides/github-workflows/pr-review.mdx

@openhands-ai
Copy link

openhands-ai bot commented Feb 8, 2026

I'm on it! xingyaoww can track my progress at all-hands.dev

Co-authored-by: openhands <openhands@all-hands.dev>
@openhands-ai
Copy link

openhands-ai bot commented Feb 8, 2026

Done! I've updated PR #311 to keep the changes to sdk/guides/github-workflows/pr-review.mdx from PR #302 while reverting all other documents.

Summary:

The PR now reverts all documents from PR #302 except sdk/guides/github-workflows/pr-review.mdx as requested.

View full conversation

Co-authored-by: openhands <openhands@all-hands.dev>
@xingyaoww xingyaoww changed the title Revert "docs: Update code review documentation with PR review workflow" Revert part of "docs: Update code review documentation with PR review workflow" Feb 8, 2026
@xingyaoww xingyaoww merged commit 541464f into main Feb 8, 2026
4 checks passed
@xingyaoww xingyaoww deleted the revert-302-openhands/update-code-review-docs branch February 8, 2026 18:44
Copy link
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

🍰

Sunday fun!

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.

3 participants