Skip to content

Conversation

@warjiang
Copy link
Contributor

@warjiang warjiang commented Jul 3, 2025

TLDR

there is no need to run all ci tasks as the original task in the scenario of forked repo, so I want to submit the PR, only allow these kinds of ci task work in the original repo.

Dive Deeper

for some schedule task like:

  • Gemini Scheduled PR Triage 🚀
  • Gemini Scheduled Issue Triage
  • Generate Weekly Community Report 📊
    there is no need to run these task periodly in the forked repo, it will turn out too many failed result.

for Gemini Automated Issue Triage task, it only make sense in the google-gemini/gemini-cli repo. So I want to

Reviewer Test Plan

Testing Matrix

🍏 🪟 🐧
npm run
npx
Docker
Podman - -
Seatbelt - -

Linked issues / bugs

@warjiang warjiang requested a review from a team as a code owner July 3, 2025 04:05
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@warjiang
Copy link
Contributor Author

warjiang commented Jul 3, 2025

I've suffered from ci task error, especially for period task:
image

@warjiang
Copy link
Contributor Author

warjiang commented Jul 3, 2025

@jerop could you help me review the changes?

image

@ali-aljufairi
Copy link
Contributor

@warjiang you can just disable actions in the forked repo but yeah I also agree it would be better if we dont have to

@warjiang
Copy link
Contributor Author

warjiang commented Jul 3, 2025

@warjiang you can just disable actions in the forked repo but yeah I also agree it would be better if we dont have to

+1

@jackwotherspoon jackwotherspoon changed the title feat(ci): disable schedule job in forked repo ci: disable schedule job in forked repo Jul 3, 2025
@jackwotherspoon jackwotherspoon changed the title ci: disable schedule job in forked repo ci: disable scheduled jobs in forked repo Jul 3, 2025
@jackwotherspoon
Copy link
Collaborator

While as mentioned this can be disabled on forked repos by the owner of the fork, it is probably best to have it disabled by default on our end.

Thanks for this @warjiang 👍 👏

@warjiang
Copy link
Contributor Author

warjiang commented Jul 8, 2025

@jackwotherspoon Hi, can we move forward this PR

@mattKorwel
Copy link
Collaborator

Thanks for getting this in. This was definitely on my list of things to update. Appreciate the contribution.

@jackwotherspoon jackwotherspoon added this pull request to the merge queue Jul 8, 2025
Merged via the queue into google-gemini:main with commit a34cc61 Jul 8, 2025
4 checks passed
yulin0629 pushed a commit to yulin0629/gemini-cli that referenced this pull request Jul 8, 2025
JunYang-tes pushed a commit to JunYang-tes/gemini-cli.nvim that referenced this pull request Jul 12, 2025
involvex pushed a commit to involvex/gemini-cli that referenced this pull request Sep 11, 2025
reconsumeralization pushed a commit to reconsumeralization/gemini-cli that referenced this pull request Sep 19, 2025
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.

Optimize GitHub Actions PR Triage Scheduler Frequency

4 participants