Skip to content

Split PR skill #125

@ericelliott

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:

"""

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