Skip to content

fix(grpc-proto): reorder authors in pyproject.toml - #400

Merged
CatherineSue merged 1 commit into
mainfrom
fix/grpc-proto-authors
Feb 10, 2026
Merged

CatherineSue merged 1 commit into
mainfrom
fix/grpc-proto-authors

Conversation

@CatherineSue

@CatherineSue CatherineSue commented Feb 10, 2026 •

Copy link
Copy Markdown
Member

Description

Problem

The PyPI page for smg-grpc-proto shows Simo Lin as the first author, but Chang Su should be listed first.

Solution

Reorder the authors list in grpc_client/python/pyproject.toml to put Chang Su first.

Changes

  • Reorder authors in pyproject.toml: Chang Su, Simo Lin, Keyang Ru

Test Plan

Metadata-only change. The updated author order will be reflected on PyPI when the next version is published.

Checklist
  • cargo +nightly fmt passes
  • cargo clippy --all-targets --all-features -- -D warnings passes
  • (Optional) Documentation updated

Summary by CodeRabbit

  • Chores
    • Updated project authors list metadata ordering.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Summary of Changes

Hello @CatherineSue, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses an issue where the PyPI page for smg-grpc-proto displayed an incorrect author order. The change ensures that the pyproject.toml file accurately lists Chang Su as the first author, which will be reflected on PyPI upon the next package publication.

Highlights

  • Author Order Correction: The order of authors in grpc_client/python/pyproject.toml has been adjusted to correctly reflect Chang Su as the primary author, followed by Simo Lin and Keyang Ru.
Changelog
  • grpc_client/python/pyproject.toml
    • Reordered the authors list to place Chang Su first, followed by Simo Lin and Keyang Ru.
Activity
  • No human activity has occurred on this pull request since its creation.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

@github-actions github-actions Bot added dependencies Dependency updates grpc gRPC client and router changes labels Feb 10, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request reorders the authors in the pyproject.toml file. The change correctly reflects the intention described in the pull request, which is to list Chang Su as the first author. The modification is straightforward and I see no issues with it.

@coderabbitai

coderabbitai Bot commented Feb 10, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The authors list in grpc_client/python/pyproject.toml is reordered, with Simo Lin moved from the first position to the second position, after Chang Su. No authors are added or removed; only their sequence is updated.

Changes

Cohort / File(s) Summary
Metadata Reordering
grpc_client/python/pyproject.toml
Authors list reordered: Simo Lin moved from first to second position, resulting in sequence: Chang Su, Simo Lin, Keyang Ru.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The authors shuffle in their place,
Simo hops with newfound grace,
Chang leads on, Keyang follows through,
A little dance, a fresh review! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/grpc-proto-authors

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

@CatherineSue
CatherineSue merged commit 73adf1e into main Feb 10, 2026
6 of 7 checks passed
@CatherineSue
CatherineSue deleted the fix/grpc-proto-authors branch February 10, 2026 21:45
@coderabbitai coderabbitai Bot mentioned this pull request Feb 10, 2026
2 of 3 tasks
ppraneth pushed a commit that referenced this pull request Feb 18, 2026
Signed-off-by: ppraneth <pranethparuchuri@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency updates grpc gRPC client and router changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant