Closed
Bug 1480774
Opened 7 years ago
Closed 7 years ago
Merge pushlog-legacy into pushlog
Categories
(Developer Services :: Mercurial: Pushlog, enhancement)
Developer Services
Mercurial: Pushlog
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sheehan, Assigned: sheehan)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
We currently have two "pushlog" extensions living in version-control-tools: `pushlog` and `pushlog-legacy`. `pushlog-legacy` currently holds the code related to generating pushlog feeds, and `pushlog` holds the bulk of the extension code (wire protocol extensions, database interactions, etc). We should move `pushlog-legacy` into the `pushlog` extension so we have a single source of truth for this crucial feature.
Additionally, we could clean up the code in `pushlog-legacy` and have the feeds make better use of Mercurial's template layer. For now, let's get the functionality moved under one roof.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sheehan
Assignee | ||
Updated•7 years ago
|
Keywords: leave-open
Assignee | ||
Comment 1•7 years ago
|
||
This commit merges `pushlog-legacy` into `pushlog`. The `pushlog-feed.py`
file is renamed to simply `feed.py`, and `test-hgweb.t` in
`pushlog-legacy` is renamed to `test-hgweb-feed.t`. Most other
changes required a simple path change, such as the new feed extension
path and the new location of the `helpers.sh` test macros. We also
update the docs to reference the new extension locations.
There is still some work to be done merging the two extensions into
one, so we don't need to specify both `pushlog-feed` and `pushlog`
in hgrc files. We will iterate on this in the future - having a
single home for pushlog is the goal of this commit.
Comment 2•7 years ago
|
||
Comment on attachment 8997444 [details]
pushlog: merge `pushlog-feed` into `pushlog` (Bug 1480774) r?gps
Gregory Szorc [:gps] has approved the revision.
https://phabricator.services.mozilla.com/D2726
Attachment #8997444 -
Flags: review+
Pushed by cosheehan@mozilla.com:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/73e0bf282bfb
pushlog: merge `pushlog-feed` into `pushlog` r=gps
Assignee | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•