Skip to content

feat: add input to allow pushing to fork#268

Merged
grahamc merged 1 commit intoDeterminateSystems:mainfrom
squat:allow_pushing_to_fork
Apr 9, 2026
Merged

feat: add input to allow pushing to fork#268
grahamc merged 1 commit intoDeterminateSystems:mainfrom
squat:allow_pushing_to_fork

Conversation

@squat
Copy link
Copy Markdown
Contributor

@squat squat commented Apr 9, 2026

Description

This commit fixes #267: it defines a new input for the GH action to
allow pushing the branch to a fork of the parent repository and creating
the PR from that fork to the parent.

Checklist
  • [] Tested functionality against a test repository (see "How to test changes")
  • Added or updated relevant documentation (leave unchecked if not applicable)

Summary by CodeRabbit

  • New Features
    • Introduced new optional push-to-fork input parameter that allows specifying a target fork repository for push operations during pull request creation workflows. This provides enhanced flexibility for fork-based contribution workflows and enables seamlessly pushing changes to alternative repositories while maintaining full pull request capabilities and functionality for automation needs.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7573673d-9850-41ca-9aae-ec82f9ba58da

📥 Commits

Reviewing files that changed from the base of the PR and between c58fe46 and 5db1c1d.

📒 Files selected for processing (1)
  • action.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • action.yml

📝 Walkthrough

Walkthrough

Added a new optional push-to-fork input parameter to the action configuration. This parameter accepts an owner/repo-fork formatted value and forwards it to the underlying peter-evans/create-pull-request action step.

Changes

Cohort / File(s) Summary
Action Input Configuration
action.yml
Added optional push-to-fork input parameter and configured it to be passed through to the peter-evans/create-pull-request action step, enabling users to push PR branches to a fork repository.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A fork in the road, both safe and sound,
With least privilege, security's found,
A simple param, so easy to pass,
Protecting the repo with utmost class! 🔒

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding a new input parameter to enable pushing to a fork, which is the primary modification in the PR.
Linked Issues check ✅ Passed The PR fulfills issue #267 by adding the push-to-fork input and forwarding it to peter-evans/create-pull-request, enabling fork-based PR creation for improved security.
Out of Scope Changes check ✅ Passed All changes in the PR are directly related to implementing the push-to-fork feature; no out-of-scope modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

This commit fixes DeterminateSystems#267: it defines a new input for the GH action to
allow pushing the branch to a fork of the parent repository and creating
the PR from that fork to the parent.
Copy link
Copy Markdown

@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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.direnv/flake-inputs/dkw3lzhrskwpi1nchpwhhz12yjq4i2f0-source:
- Line 1: Remove the committed .direnv artifacts and prevent them from being
re-committed: delete the tracked files `.direnv/bin/nix-direnv-reload` and
`.direnv/flake-profile-*.rc` from git history (or at least from the current
commit) using git rm --cached (or git rm) for those paths, add `.direnv/` to
`.gitignore`, commit the removal and .gitignore change, and push the branch;
ensure no other .direnv files remain tracked (verify with git ls-files | grep
.direnv) before creating the PR update.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4bb142ac-438f-484d-a826-63cbe90b0cf5

📥 Commits

Reviewing files that changed from the base of the PR and between c92eec5 and c58fe46.

📒 Files selected for processing (6)
  • .direnv/bin/nix-direnv-reload
  • .direnv/flake-inputs/dkw3lzhrskwpi1nchpwhhz12yjq4i2f0-source
  • .direnv/flake-inputs/s7bvgynzfisna3jdq2cr4jx73xnhxs27-source
  • .direnv/flake-profile-a5d5b61aa8a61b7d9d765e1daf971a9a578f1cfa
  • .direnv/flake-profile-a5d5b61aa8a61b7d9d765e1daf971a9a578f1cfa.rc
  • action.yml

Comment thread .direnv/flake-inputs/dkw3lzhrskwpi1nchpwhhz12yjq4i2f0-source Outdated
@squat squat force-pushed the allow_pushing_to_fork branch from c58fe46 to 5db1c1d Compare April 9, 2026 01:40
Copy link
Copy Markdown
Member

@grahamc grahamc left a comment

Choose a reason for hiding this comment

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

This is so great. Thank you so much!

@grahamc grahamc enabled auto-merge April 9, 2026 02:08
@grahamc grahamc disabled auto-merge April 9, 2026 02:08
@grahamc grahamc merged commit ff43f16 into DeterminateSystems:main Apr 9, 2026
3 checks passed
@squat squat deleted the allow_pushing_to_fork branch April 19, 2026 09:14
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.

Allow Pushing to Fork

2 participants