I think the code works well but it might leave unnecessary pinned nodes
around. This patch makes it more conservative so that only a movement
(update or bookmark) on an obsolete commit would write new pinned nodes.
Details
Details
- Reviewers
durham stash - Group Reviewers
Restricted Project - Commits
- rFBHGX5fb55777b826: hiddenoverride: be more conservative about pinning nodes
Diff Detail
Diff Detail
- Repository
- rFBHGX Facebook Mercurial Extensions
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
I'm pushing this since it seems helpful to solve some weird cases that people have reported. That might prevent another rollback on Friday's release.
hgext3rd/fbamend/hiddenoverride.py | ||
---|---|---|
115 | Smth is wrong with a comment |
Smth is wrong with a comment