Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix duplicate nested function declaration and handle merging column headers #25

Closed
wants to merge 2 commits into from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Aug 10, 2023

Description

This PR fixes the compilation errors caused by a duplicate nested function declaration in merger.go and modifies the combine function to handle merging similarly named column headers to the same column.

Summary of Changes

  • Removed the duplicate nested function declaration of combine in merger.go.
  • Modified the existing combine function to handle merging columns with matching headers.
  • Updated the logic within the combine function to correctly merge similarly named column headers.
  • Ensured that the TestCombine test case passes after making these changes.

Fixes #19.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/fix-merge-column-headers

To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.

sweep-ai bot added 2 commits August 10, 2023 20:26
…ed the existing `combine` function body to handle merging similarly named column headers
@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Aug 10, 2023
@sweep-ai sweep-ai bot closed this Aug 10, 2023
@sweep-ai sweep-ai bot deleted the sweep/fix-merge-column-headers branch August 10, 2023 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: make all similarly named column headers merge to the same column
0 participants