← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~xaav/loggerhead/export-tarball into lp:loggerhead

 

> I reverted the change in 455, because I couldn't download tarballs via a web
> browser.

Right, specifically turning it back on by default.  I think that's fine.

I'm just testing it interactively in a browser and it does seem to work ok, which is great.  

[fix] There is one bug which is that the tarball link always gets you the tarball from the current version of the branch, even if you're looking at an older version, which I think people would find confusing.  Could you try to fix that?

My small changes are in lp:~mbp/loggerhead/export-tarball
-- 
https://code.launchpad.net/~xaav/loggerhead/export-tarball/+merge/66408
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~xaav/loggerhead/export-tarball into lp:loggerhead.


References