Skip to content

username changed#345

Closed
mdabucse wants to merge 2 commits intoabhimehro:mainfrom
mdabucse:make-change
Closed

username changed#345
mdabucse wants to merge 2 commits intoabhimehro:mainfrom
mdabucse:make-change

Conversation

@mdabucse
Copy link

#335 I made the changes based on the requirements

@trunk-io
Copy link

trunk-io bot commented Feb 18, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 18, 2026
@gemini-code-assist
Copy link

Summary of Changes

Hello @mdabucse, 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 a minor documentation issue by correcting the git clone command in the README.md file. The change ensures that users can accurately clone the repository using the correct GitHub username, improving the clarity and usability of the project's setup instructions.

Highlights

  • Documentation Update: Updated the git clone command in README.md to reflect the correct repository owner username, changing it from a placeholder your-username to abhimehro.
Changelog
  • README.md
    • Updated the git clone command to use the correct GitHub username abhimehro.
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.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Message that will be displayed on users' first pull request

Copy link

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

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 updates the repository URL in the README.md to use a specific username instead of a placeholder. While this makes the command work out-of-the-box for cloning the main repository, it could be confusing for users who follow the instructions to fork the repository for using GitHub Actions. My suggestion is to use a more explicit placeholder like YOUR_USERNAME to guide users to clone their own fork, ensuring a smoother setup process for all use cases described in the documentation.

Thanks for the suggestion! The clone URL under "Quick start" is intentionally set to the original repository owner (abhimehro) so users can directly copy-paste and clone the repo. Users who want to use GitHub Actions are separately instructed to fork the repo in the "Configure GitHub Actions" section below, so the distinction is already clear.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@mdabucse
Copy link
Author

@abhimehro
The Development Partner / copilot-setup-steps check fails because the workflow uses the pull_request event, which gives a read-only GITHUB_TOKEN for PRs from forks. The createComment() call at line 55 of
copilot-setup-steps.yml
gets a 403 Forbidden error. This is not related to my README change — it would fail for any fork-based PR. The repo owner may want to switch this workflow to use pull_request_target or make it a non-required check.

Copy link
Owner

@abhimehro abhimehro left a comment

Choose a reason for hiding this comment

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

Hey @mdabucse, thanks for opening this and for the clear write-up on the CI failure!

On the README change:
Gemini's note is worth taking seriously. Using your-username (or YOUR_USERNAME) as a placeholder is the safer pattern, especially since the docs walk contributors through forking first. If someone clones directly from abhimehro:main instead of their fork, they'll hit push permission errors down the line. Want to swap it back to a placeholder with a short note clarifying the fork-first flow?

On the failing check:
You're right, this isn't on you. The copilot-setup-steps workflow uses pull_request, which restricts the token to read-only for fork-based PRs. The createComment() call at line 55 will 403 every time for external contributors. I'll look at migrating it to pull_request_target, or marking it non-required so it doesn't block otherwise clean PRs.

Next steps:

  • Swap the clone URL to a placeholder (or add an inline comment pointing contributors to substitute their username)
  • I'll handle the workflow fix on my end

Appreciate you flagging the CI issue proactively! That kind of context saves a lot of back and forth.

@abhimehro
Copy link
Owner

Swapped it back to a placeholder with a short note clarifying the fork-first flow in Pull Request #361

@abhimehro abhimehro closed this Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants