Skip to content

Split PR skill #125

Description

@ericelliott

I'm experimenting with a new way to break large PRs down into smaller PRs. We should refine this and turn it into a skill for aidd Framework:

"""
We are trying to merge changes from the main PR below.

However, this branch contains way too much. Our task is to spilt it into smaller pieces we can merge incrementally without breaking what's already working.

  1. Merge in latest main and resolve any merge conflicts (ask before changing anything that might break anything)
  2. Some of this has already been done. You need to account for that. See "Reference" below.
  3. Look for opportunities to break out smaller utilities. Look at existing churn in the branch and see if splitting files that are larger than 200 LoC into separate modules might make it a little more resilient to fixes without breaking things. Watch out for any shared mutable state which might complicate things and think about how we may refactor to eliminate brittle patterns.
  4. Please refer to the JavaScript rules in $projectRoot/ai/rules/javascript and make sure that we are obeying the constraints there.

Reference:

  • Main PR:

"""

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions