Closed
Bug 933053
Opened 12 years ago
Closed 12 years ago
Remove content of Mozmill addon from README.md files
Categories
(Testing Graveyard :: Mozmill, defect)
Testing Graveyard
Mozmill
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gilles.leblanc, Unassigned)
Details
(Whiteboard: [mozmill-2.0.1])
Attachments
(1 file)
|
6.00 KB,
patch
|
davehunt
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:24.0) Gecko/20100101 Firefox/24.0 (Beta/Release)
Build ID: 20130911155223
Steps to reproduce:
Visit page https://github.com/mozilla/mozmill and page https://github.com/mozilla/mozmill/tree/master/mozmill.
README.md text is presented on these pages.
Actual results:
The text on both pages mention the Mozmill addon and include a link for addons.mozilla.org that point to a page that says: "This add-on has been removed by its author.".
Expected results:
The GitHub pages (sourced from the README.md) should not have pointed to the former addon page from the removed addon.
| Reporter | ||
Comment 1•12 years ago
|
||
The addon link on the README page leads to a page that says: "This add-on has
been removed by its author.". Furthermore, by searching on Google I was
able to find the following Mozmill Developpers Google Groups message
(https://groups.google.com/forum/#!topic/mozmill-dev/02HwCHWQFRA) that
states the addon was removed because it hadn't been updated in a long
time and support for it had already been dropped.
I originally submitted this as a pull request on GitHub where :davehunt asked me to create a bug for it here.
| Reporter | ||
Comment 2•12 years ago
|
||
developer.mozilla.org mentions using hg for creating the patch but since this is for a GitHub git repo I used git. If you prefer an hg patch let me know.
| Reporter | ||
Updated•12 years ago
|
Attachment #824992 -
Flags: review?(hskupin)
Attachment #824992 -
Flags: review?(dave.hunt)
Comment 3•12 years ago
|
||
Comment on attachment 824992 [details] [diff] [review]
patch
Review of attachment 824992 [details] [diff] [review]:
-----------------------------------------------------------------
This is great, thanks! Landed in:
https://github.com/mozilla/mozmill/commit/8d1d3bc64533b30eaa4c096cdb92d27317d083ed
Attachment #824992 -
Flags: review?(hskupin)
Attachment #824992 -
Flags: review?(dave.hunt)
Attachment #824992 -
Flags: review+
Updated•12 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 4•12 years ago
|
||
Thank you for such a prompt review!
Comment 5•12 years ago
|
||
Thanks for the patch! :)
Comment 6•12 years ago
|
||
This misses a backport to the hotfix-2.0 branch given that this is our active version.
Landed on hotfix-2.0:
https://github.com/mozilla/mozmill/commit/edf9b98fa251c3374ea0fa862a8db98fb1ea7a9a
Thanks Gilles for your contribution! If you are interested in more please let me know.
Summary: Mention of addon in README.md files for GitHub project → Remove content of Mozmill addon from README.md files
Whiteboard: [mozmill-2.0.1]
Version: 2.0 Branch → unspecified
| Assignee | ||
Updated•9 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•