Skip to content

Change patch --dry-run behavior#415

Open
FoamyGuy wants to merge 1 commit intoadafruit:mainfrom
FoamyGuy:dry_run_change
Open

Change patch --dry-run behavior#415
FoamyGuy wants to merge 1 commit intoadafruit:mainfrom
FoamyGuy:dry_run_change

Conversation

@FoamyGuy
Copy link
Copy Markdown
Contributor

@FoamyGuy FoamyGuy commented Apr 8, 2026

This makes 3 changes to the patch command behavior:

  • --dry-run now makes the changes from the patch locally. It used to only check whether the patch applied but not make any local changes
  • New argument --commit-changes added to create a commit from the local changes.
  • New argument --push-changes added to push commits that were created prior

This new functionality makes the following workflow possible:

  1. Apply a patch with adabot using --dry-run
  2. Make any other manual changes desired to local repos
  3. Create commits from local changes
  4. Push the commits

I intend to use this functionality to apply the ruff version update patch from #410. The newer version of ruff introduced new or modified rules that some libraries fail (PASS: 211, FAIL: 198). This workflow will allow us to fix the ruff failures before commiting and pushing the changes.

…ents used for commiting and pushing separately.
@FoamyGuy FoamyGuy requested a review from a team April 8, 2026 14:49
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.

1 participant