aws / codecatalyst-blueprints

Apache License 2.0
48 stars 18 forks source link

fix: QOL improvement that defaults the merge strategy of blueprints to prefer-new #557

Closed alexforsyth closed 1 month ago

alexforsyth commented 1 month ago

Issue

Blueprints use three way merge as the default merge strategy. This is kinda annoying in practice because it causes various merge conflicts where there arent meant to be merge conflicts. Or does other annoying things like creating full page merge conflicts on linter errors.

Description

This PR changes the default behavior to be three way merge BUT with a default preference towards the newer code. Customers can still overwrite this by building their own merge strategy or applying one of the existing strategies on the repo construct.

Testing

How was this change tested?

Additional context

Add any other context about the PR here.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.