Skip to content

feat: add cherry-pick option and methods with tests#170

Merged
seokju-na merged 5 commits intotoss:mainfrom
racgoo:feat/cherrypick
Dec 24, 2025
Merged

feat: add cherry-pick option and methods with tests#170
seokju-na merged 5 commits intotoss:mainfrom
racgoo:feat/cherrypick

Conversation

@racgoo
Copy link
Copy Markdown
Contributor

@racgoo racgoo commented Dec 13, 2025

Summary

Add cherry-pick support to the 'Repository' with tests.

Changes

  • Add 'cherrypick' and 'cherrypick_commit' methods
  • Add tests covering cherry-pick behavior

Notes

  • 'cherrypick_commit' is provided for inspecting the resulting 'Index' of a cherry-pick operation without mutating the repository.
  • The 'dryRun' option on 'cherrypick' simulates applying the cherry-pick to detect conflicts and feasibility, whereas 'cherrypick_commit' returns the computed index that would result from the operation.
  • Both approaches run without side effects, but serve different purposes: validation ('dryRun') vs result inspection ('cherrypick_commit').

Closes #169

@racgoo racgoo requested a review from seokju-na as a code owner December 13, 2025 09:52
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 13, 2025

@racgoo is attempting to deploy a commit to the Toss Team on Vercel.

A member of the Team first needs to authorize it.

@racgoo
Copy link
Copy Markdown
Contributor Author

racgoo commented Dec 19, 2025

Hi @seokju-na ,
I’ve opened this PR, but the GitHub Actions didn’t run due to permission limitations.
Could you please re-run the CI for this PR?

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 22, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
es-git Ready Ready Preview, Comment Dec 22, 2025 10:16am

Copy link
Copy Markdown
Collaborator

@seokju-na seokju-na left a comment

Choose a reason for hiding this comment

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

Thanks!

@seokju-na seokju-na merged commit 44a2a2d into toss:main Dec 24, 2025
27 checks passed
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.

cherrypick

2 participants