Differential D15877
When observing a repository, switch to "importing" mode on a large discovery in an empty repository Authored by epriestley on May 10 2016, 2:05 PM. Tags None Referenced Files
Subscribers None
Details When hosting a repository, we currently have a heuristic that tries to detect when you're doing an initial import: if you push more than 7 commits to an empty repository, it counts as an import and we disable mail/feed/etc. Do something similar for observed repositories: if the repository is empty and we discover more than 7 commits, switch to import mode until we catch up. This should align behavior with user expectation more often when juggling hosted vs imported repositories.
Diff Detail
Event TimelineComment Actions Ah, I think this caused the bad unit test in the rest of the series -- specifically, in the unit tests we create a repository and mark it ephemeral (to prevent anything from updating it), but this now causes it to update. There's no real reason to mark it ephemeral (this was just a sanity check sort of thing, I believe), so I'll just remove that, which should fix this and all subsequent changes. Comment Actions That fix seems to work, I'll cascade it down the chain and sort things out in the morning. |