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

Merge cassandra-gocql-driver/trunk #387

Conversation

sylwiaszunejko
Copy link
Collaborator

No description provided.

@sylwiaszunejko
Copy link
Collaborator Author

I am not sure why there is only merge commit present. I used git merge command and expected to have something like in this PR #148 with commits from upstream and then merge commit. I am not sure why it looks differently here

@roydahan
Copy link
Collaborator

Did you try rebase?

@dkropachev
Copy link
Collaborator

I am not sure why there is only merge commit present. I used git merge command and expected to have something like in this PR #148 with commits from upstream and then merge commit. I am not sure why it looks differently here

Can you please do it again with rebase, it should preserve original commits

@Lorak-mmk
Copy link

With merge it should still show all the commits.

@sylwiaszunejko
Copy link
Collaborator Author

sylwiaszunejko commented Jan 28, 2025

@dkropachev @roydahan I am not sure if rebasing is a correct solution in this case, it means a lot of conflicts and changing commits from long ago as previously AFAIK merging upstream was done using git merge

@dkropachev
Copy link
Collaborator

@dkropachev @roydahan I am not sure if rebasing is a correct solution in this case, it means a lot of conflicts and changing commits from long ago as previously AFAIK merging upstream was done using git merge

What you describe is normal rebase process, you need to evaluate each commit and understand whether we need it or not, and how to change it to get it through.

@sylwiaszunejko
Copy link
Collaborator Author

@dkropachev @roydahan I am not sure if rebasing is a correct solution in this case, it means a lot of conflicts and changing commits from long ago as previously AFAIK merging upstream was done using git merge

What you describe is normal rebase process, you need to evaluate each commit and understand whether we need it or not, and how to change it to get it through.

sure, but as it was never done before it is a really time consuming process and I am not sure it is worth the effort tbh @Lorak-mmk WDYT?

@sylwiaszunejko
Copy link
Collaborator Author

I am closing this PR in favour of #388

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.

4 participants