Closed Bug 1480774 Opened 7 years ago Closed 7 years ago

Merge pushlog-legacy into pushlog

Categories

(Developer Services :: Mercurial: Pushlog, enhancement)

enhancement
Not set
normal

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: nobody → sheehan
Keywords: leave-open
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 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
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: