Summary: | [webkitscmpy] Document package usage | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||
Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aakash_jain, dewei_zhu, slewis, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jonathan Bedard
2021-02-16 12:09:22 PST
Created attachment 420541 [details]
Patch
Comment on attachment 420541 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=420541&action=review > Tools/Scripts/libraries/webkitscmpy/README.md:7 > +webkitcorepy along with fasteners, monotonic, whichcraft, and xmltodict. nit: Think this would read better as bulleted list than as a sentence > Tools/Scripts/libraries/webkitscmpy/README.md:11 > +The `git-webkit` command exposes much of the utility of the `webkitscmpy` library to the command line. Most notably: nit: the utility of webktscmpy seems circular. We haven't defined what the utility is yet. Maybe replace with what the utility of webkitscmpy is > Tools/Scripts/libraries/webkitscmpy/README.md:15 > +`git-webkit checout <ref>`: Move the current local repository to the provided git ref, Subversion revision or identifier. typo checkout > Tools/Scripts/libraries/webkitscmpy/README.md:17 > +`git-webkit canoniclize`: Standardize commit authorship and put identifiers into the commit message. typo canonicalize. I think that's an awkward name for a command > Tools/Scripts/libraries/webkitscmpy/README.md:21 > +The `webkitscmpy` library provides a repository abstraction for both local and remote repositories. Those abstractions nit: Instead of Those abstractions can be instantiated in code like so how about: To instantiate a repository object. > Tools/Scripts/libraries/webkitscmpy/README.md:28 > +remote.Scm.from_url('https://svn.webkit.org/repository/webkit') nit: Would this make more sense if it was assigned to a variable? > Tools/Scripts/libraries/webkitscmpy/README.md:33 > +be missing from certain implementations. For example, remote repositories do not have a `checkout` command available. nit how about While the abstraction layer is consistent for all implementations not all implementation support every feature. For example, ... > Tools/Scripts/libraries/webkitscmpy/README.md:35 > +Each repository abstraction keeps a record contributors, which can be primed and passed into the repository object: nit: record of contributors? list of contributor records? Created attachment 420554 [details]
Patch for landing
Committed r272939: <https://commits.webkit.org/r272939> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420554 [details]. |