This is an archive of the discontinued Mercurial Phabricator instance.

phabstatus: allprecursors -> allpredecessors
ClosedPublic

Authored by ryanmce on Aug 17 2017, 1:36 PM.

Reviewers

simonfar

Group Reviewers

Restricted Project

Commits

Summary

Renamed upstream. Eliminate more develwarns.

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.

I've just noticed the typo in the title - can you fix all the titles before landing?

This revision is now accepted and ready to land.Aug 17 2017, 3:16 PM

ryanmce retitled this revision from phabstatus: allprecurors -> allpredecessors to phabstatus: allprecursors -> allpredecessors.Aug 18 2017, 6:07 AM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			hgext3rd/phabstatus.py (2 lines)

	except (IndexError, KeyError, ValueError, TypeError):			except (IndexError, KeyError, ValueError, TypeError):
	# We got no result back, or it did not contain all required fields			# We got no result back, or it did not contain all required fields
	return "Error"			return "Error"

	local = ctx.hex()			local = ctx.hex()
	if local == remote:			if local == remote:
	return "sync"			return "sync"
	elif count == 1:			elif count == 1:
	precursors = list(obsutil.allprecursors(repo.obsstore,			precursors = list(obsutil.allpredecessors(repo.obsstore,
	[ctx.node()]))			[ctx.node()]))
	hashes = [repo.unfiltered()[h].hex() for h in precursors]			hashes = [repo.unfiltered()[h].hex() for h in precursors]
	# hashes[0] is the current			# hashes[0] is the current
	# hashes[1] is the previous			# hashes[1] is the previous
	if len(hashes) > 1 and hashes[1] == remote:			if len(hashes) > 1 and hashes[1] == remote:
	return "sync"			return "sync"
	else:			else:
	return "unsync"			return "unsync"