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

Give an option to hide merge commits in a branch view in the UI #7441

Closed
nicholasjng opened this issue Feb 6, 2024 · 3 comments · Fixed by #8142
Closed

Give an option to hide merge commits in a branch view in the UI #7441

nicholasjng opened this issue Feb 6, 2024 · 3 comments · Fixed by #8142
Assignees

Comments

@nicholasjng
Copy link
Contributor

Similarly to GitHub metrics I guess. We switched our transaction model to a merge of an ephemeral branch, which creates merge commits on success. When going through a branch history, these commits clutter the view, so it would be great to have a filtered view where we can focus on the actual additions.

cc @maxmynter

@talSofer
Copy link
Contributor

Thanks for opening this request @nicholasjng! Are you using lakectl or the lakeFS UI to view the commit history?

If you had a --no-merges flag as part of the lakectl log command, would that solve your case?

@talSofer talSofer added the good first issue Good for newcomers label Feb 18, 2024
@nicholasjng
Copy link
Contributor Author

I'm viewing it in the UI. --no-merges sounds like a good idea (assuming it means everything with maximum 1 parent, like in git log), for a UI view of commits, should not be difficult to filter by that condition, then.

@arielshaqed
Copy link
Contributor

Also the same filtering is equally possible in the UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants