Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Disable push event for packages index generation#686

Closed
mangelajo wants to merge 3 commits intomainfrom
disable-packages-trigger
Closed

Disable push event for packages index generation#686
mangelajo wants to merge 3 commits intomainfrom
disable-packages-trigger

Conversation

@mangelajo
Copy link
Copy Markdown
Member

@mangelajo mangelajo commented Oct 3, 2025

Comment out the push event trigger for packages index.

Summary by CodeRabbit

  • Chores

    • Replaced automated CI triggers for package indexing with a manual workflow trigger; automated runs on pushes/tags are disabled (kept as commented reference).
  • No User Impact

    • No changes to features, performance, or UI.
    • Runtime behavior of the application is unchanged; only release automation is modified.

Comment out the push event trigger for packages index.
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 3, 2025

Deploy Preview for jumpstarter-docs ready!

Name Link
🔨 Latest commit 94bf012
🔍 Latest deploy log https://app.netlify.com/projects/jumpstarter-docs/deploys/68e80dcb2d148e00083d3cc8
😎 Deploy Preview https://deploy-preview-686--jumpstarter-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 3, 2025

Walkthrough

The GitHub Actions workflow .github/workflows/trigger-packages-index.yaml had its event triggers changed from push/tags to a manual workflow_dispatch; the previous push/tags configuration remains as a commented block. Job definitions were not modified.

Changes

Cohort / File(s) Summary
Workflow trigger change
.github/workflows/trigger-packages-index.yaml
Replaced on: push / tags trigger with on: workflow_dispatch; original push/tags block retained as a commented-out section. Jobs and steps unchanged.

Sequence Diagram(s)

sequenceDiagram
    participant Dev as Developer
    participant GitHub as GitHub Events
    participant Workflow as Packages Index Workflow

    rect rgba(0,128,96,0.06)
    note right of GitHub: Previous flow (before change)
    Dev->>GitHub: push / create tag
    GitHub->>Workflow: trigger workflow (push/tags)
    end

    rect rgba(0,96,192,0.06)
    note right of Dev: New flow (after change)
    Dev->>Workflow: manual "workflow_dispatch"
    Workflow->>Workflow: run jobs (unchanged)
    end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • bennyz

Poem

I twitch my whiskers, buttons bright,
No pushes wake the workflow's light.
A commented hush across the lane,
When humans click, I'll hop again. 🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly summarizes the main change by stating that the push event trigger for the packages index generation has been disabled, matching the alterations in the workflow file and avoiding vague or extraneous wording.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch disable-packages-trigger

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3033dd2 and 4143f55.

📒 Files selected for processing (1)
  • .github/workflows/trigger-packages-index.yaml (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/trigger-packages-index.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)
  • GitHub Check: Redirect rules - jumpstarter-docs
  • GitHub Check: Header rules - jumpstarter-docs
  • GitHub Check: Pages changed - jumpstarter-docs
  • GitHub Check: e2e
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.13)
  • GitHub Check: pytest-matrix (macos-15, 3.12)
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.12)
  • GitHub Check: pytest-matrix (macos-15, 3.13)
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.11)
  • GitHub Check: pytest-matrix (macos-15, 3.11)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6266878 and 3033dd2.

📒 Files selected for processing (1)
  • .github/workflows/trigger-packages-index.yaml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)
  • GitHub Check: Redirect rules - jumpstarter-docs
  • GitHub Check: Header rules - jumpstarter-docs
  • GitHub Check: Pages changed - jumpstarter-docs
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.11)
  • GitHub Check: pytest-matrix (macos-15, 3.13)
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.13)
  • GitHub Check: pytest-matrix (macos-15, 3.11)
  • GitHub Check: pytest-matrix (macos-15, 3.12)
  • GitHub Check: pytest-matrix (ubuntu-24.04, 3.12)
  • GitHub Check: e2e

Comment thread .github/workflows/trigger-packages-index.yaml
@kirkbrauer kirkbrauer enabled auto-merge (squash) October 9, 2025 19:36
@mangelajo
Copy link
Copy Markdown
Member Author

Not needed anymore, fixed by ncao :D

@mangelajo mangelajo closed this Oct 21, 2025
auto-merge was automatically disabled October 21, 2025 09:30

Pull request was closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants