Skip to content

[Stacked] Optimize commits - #8

Merged
south-mer merged 3 commits into
mercari-developfrom
optimize-commits
Jul 3, 2025
Merged

south-mer merged 3 commits into
mercari-developfrom
optimize-commits

Conversation

@south-mer

Copy link
Copy Markdown
Collaborator

What

Optimize commit fetching logic by combining it with comments/reviews

Why

To save request count and also GraphQL rate limit

hi120ki
hi120ki previously approved these changes Jul 3, 2025

@hi120ki hi120ki left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM for better GraphQL access

@south-mer
south-mer changed the base branch from otel to mercari-develop July 3, 2025 09:09
@south-mer
south-mer dismissed hi120ki’s stale review July 3, 2025 09:09

The base branch was changed.

@south-mer
south-mer requested a review from hi120ki July 3, 2025 09:09
@south-mer
south-mer merged commit 78b4999 into mercari-develop Jul 3, 2025
@south-mer
south-mer deleted the optimize-commits branch January 7, 2026 10:17
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