Paths

Table of Contentst

Differential D2569

Resolve VCS hashes to commit URL
ClosedPublic
Actions

Authored by dereckson on Mar 6 2022, 02:54.

Tags

None

Referenced Files

	F12439803: D2569.id.diff
	Thu, Nov 6, 09:56

	F12430223: D2569.id6490.diff
	Wed, Nov 5, 13:50

	Unknown Object (File)
	Wed, Nov 5, 11:35

	Unknown Object (File)
	Wed, Nov 5, 11:32

	Unknown Object (File)
	Wed, Nov 5, 00:48

	Unknown Object (File)
	Tue, Nov 4, 06:58

	Unknown Object (File)
	Mon, Nov 3, 09:48

	Unknown Object (File)
	Mon, Nov 3, 09:47

Subscribers

None

Details

Reviewers

Commits

rRESOLVEHASH34835825f734: Resolve VCS hashes to commit URL

Summary

Provide a small utility to find an URL representation
matching a hash, querying code review systems like Phabricator
or Gerrit, then public code hosting like GitHub or GitLab.

Test Plan

Run commands and code documented in README + make test

Diff Detail

Repository

rRESOLVEHASH Resolve hash

Lint

Lint Errors

Severity	Location	Code	Message
Error	src/resolvehash/search/__init__.py:8	F401	flake8 F401
Error	src/resolvehash/search/vcs.py:28	E501	flake8 E501
Error	src/resolvehash/search/vcs.py:32	E501	flake8 E501
Error	src/resolvehash/search/vcs.py:35	E501	flake8 E501
Error	src/resolvehash/search/vcs.py:40	E501	flake8 E501
Error	src/resolvehash/vcs/phabricator.py:58	E501	flake8 E501
Error	tests/test_resolvehash.py:35	E501	flake8 E501
Warning	src/resolvehash/resolvehash.py:	CHMOD1	Invalid Executable

Unit

No Test Coverage

Branch

main

Build Status

Buildable 4026
Build 4278: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 6 2022, 02:54

dereckson created this revision.

Harbormaster completed remote builds in B4025: Diff 6489.Mar 6 2022, 02:54

Provide the symbolink link from ${PREFIX}/bin/resolve-hash to ${PYCACHE}/**/resolvehash/resolvehash.py

Ready to release.

Harbormaster completed remote builds in B4026: Diff 6490.Mar 6 2022, 10:16

In D2569#37892, @dereckson wrote:

Provide the symbolink link from ${PREFIX}/bin/resolve-hash to ${PYCACHE}/**/resolvehash/resolvehash.py

Done, with command line specific code moved to bin/resolve-hash instead.

dereckson accepted this revision.Mar 6 2022, 10:18

This revision is now accepted and ready to land.Mar 6 2022, 10:18

This revision was landed with ongoing or failed builds.Mar 6 2022, 10:19

Closed by commit rRESOLVEHASH34835825f734: Resolve VCS hashes to commit URL (authored by dereckson). · Explain Why

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rRESOLVEHASH34835825f734: Resolve VCS hashes to commit URL.

dereckson mentioned this in P301 bin/resolve-hash.Mar 6 2022, 13:01

Revision Contents
Changeset List

Path

Size

4 lines

29 lines

8 lines

24 lines

43 lines

100 lines

bin/

32 lines

dev-requirements.txt

1 line

14 lines

requirements.txt

2 lines

32 lines

src/

resolvehash/

6 lines

54 lines

search/

8 lines

62 lines

vcs/

6 lines

38 lines

22 lines

22 lines

87 lines

tests/

test_resolvehash.py

40 lines

Diff 6490

.arcconfig

Loading...

.arclint

Loading...

.gitignore

Loading...

LICENSE

Loading...

Makefile

Loading...

README.md

Loading...

bin/resolve-hash

Loading...

dev-requirements.txt

Loading...

pyproject.toml

Loading...

requirements.txt

Loading...

setup.cfg

Loading...

src/resolvehash/init.py

Loading...

src/resolvehash/resolvehash.py

Loading...

src/resolvehash/search/init.py

Loading...

src/resolvehash/search/vcs.py

Loading...

src/resolvehash/vcs/init.py

Loading...

src/resolvehash/vcs/gerrit.py

Loading...

src/resolvehash/vcs/github.py

Loading...

src/resolvehash/vcs/gitlab.py

Loading...

src/resolvehash/vcs/phabricator.py

Loading...

tests/test_resolvehash.py

Loading...