Skip to content

Conversation

@cwperks
Copy link
Member

@cwperks cwperks commented Jun 9, 2025

Description

This is a quality of life PR to drastically reduce the number of merge conflicts and prevent maintainers from having to constantly resolve conflicts only in the CHANGELOG file.

By setting the gitattribute merge=union, this tells git to automatically take the union of both sides of the conflicts without maintainers having to manually determine how to perform the merge.

Related Issues

Resolves opensearch-project/OpenSearch#18475

Related to https://about.gitlab.com/blog/gitlab-reduced-merge-conflicts-by-90-percent-with-changelog-placeholders/

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…conflicts

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@codecov
Copy link

codecov bot commented Jun 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.18%. Comparing base (69916f4) to head (9bdebb8).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5384      +/-   ##
==========================================
+ Coverage   72.15%   72.18%   +0.02%     
==========================================
  Files         382      382              
  Lines       23697    23697              
  Branches     3644     3644              
==========================================
+ Hits        17099    17106       +7     
+ Misses       4804     4795       -9     
- Partials     1794     1796       +2     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cwperks cwperks merged commit 6444856 into opensearch-project:main Jun 9, 2025
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Minimize merge conflicts in CHANGELOG

3 participants