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 component history by marking snaps prior to tag-from-scope as hidden #9281

Merged
merged 4 commits into from
Oct 31, 2024

Conversation

davidfirst
Copy link
Member

Here is the issue this PR attempt to resolve.
When components are developed on lanes and merged back, the component history in bit log often displays redundant snap-tag pairs for each merge, where each snap holds meaningful changes, but the subsequent tag, when created by tag-from-scope (_tag), duplicates these without adding new information. This redundancy leads to:

  • Cluttered history views (bit log and bit blame), making it hard to track meaningful version updates.
  • Misleading file histories and “blame” data, as changes are always attributed to snaps instead of tags.
  • Loss of meaningful commit messages, with only the final snap message from the lane appearing in the main history after a merge.

Solution:
The hidden property is added to the Version object to filter out specific snaps after tagging, specifically when using the _tag command (or tag-from-scope). This command is commonly used when merging lanes and often results in redundant snap-tag pairs.
When _tag creates a tag from the last snap on a lane, it copies the same file configuration and contents, leading to duplicate entries in the component history. By setting hidden = true on the snap after the tag is created, only the tag remains visible in bit log and bit blame, while the snap (containing redundant data) is hidden. This prevents clutter in history views and ensures that only the most meaningful tags appear in version tracking commands.

@davidfirst davidfirst enabled auto-merge (squash) October 31, 2024 21:09
@davidfirst davidfirst merged commit 50e57bb into master Oct 31, 2024
11 checks passed
@davidfirst davidfirst deleted the tag-from-scope-hidden branch October 31, 2024 22:12
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.

2 participants