Skip to content

Conversation

@tbruyelle
Copy link
Contributor

Since we are forcing the updage of changelog.md on every PR, this leads to many conflicts when it's time to merge.

By forcing the merge strategy of this file to union, it decreases the chance of conflicts by ~90%. The downside is sometimes the merge gets wrong (repeated lines) so the dev should have a look at the result.

Since we are forcing the updage of changelog.md on every PR, this leads
to many conflicts when it's time to merge.

By forcing the merge strategy of this file to `union`, it decreases the
chance of conflicts by ~90%. The downside is sometimes the merge gets
wrong (repeated lines) so the dev should have a look at the result.
@tbruyelle tbruyelle merged commit b2bb5d6 into develop Sep 7, 2022
@tbruyelle tbruyelle deleted the chore/changelog-merge-strategy branch September 7, 2022 13:28
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
Since we are forcing the updage of changelog.md on every PR, this leads
to many conflicts when it's time to merge.

By forcing the merge strategy of this file to `union`, it decreases the
chance of conflicts by ~90%. The downside is sometimes the merge gets
wrong (repeated lines) so the dev should have a look at the result.
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.

3 participants