Closed
Bug 297744
Opened 19 years ago
Closed 19 years ago
Update "Changes" Page for 2.20rc1 and 2.18.2
Categories
(Bugzilla :: bugzilla.org, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mkanat, Assigned: mkanat)
References
()
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
wicked
:
review+
|
Details | Diff | Splinter Review |
This is pretty simple, usually. :-)
Assignee | ||
Updated•19 years ago
|
Assignee: nobody → mkanat
Assignee | ||
Comment 1•19 years ago
|
||
Attachment #188161 -
Flags: review?(wicked)
Assignee | ||
Updated•19 years ago
|
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Comment 2•19 years ago
|
||
Comment on attachment 188161 [details] [diff] [review]
Changes page
Usually yes.. but this time, I think you need to create 2.20 branch column too.
Assuming you plan to branch at the same time you release 2.20rc1 like I have
understood.
Attachment #188161 -
Flags: review?(wicked) → review-
Assignee | ||
Comment 3•19 years ago
|
||
(In reply to comment #2)
> (From update of attachment 188161 [details] [diff] [review] [edit])
> Usually yes.. but this time, I think you need to create 2.20 branch column too.
> Assuming you plan to branch at the same time you release 2.20rc1 like I have
> understood.
There's no 2.21 yet, so there's no "2.20 branch" needed, yet. That will be
necessary for rc2/2.20.
Comment 4•19 years ago
|
||
Comment on attachment 188161 [details] [diff] [review]
Changes page
OK, then it's probably next releases that need the 2.20 branch column created.
Assuming mkanat knows what he is doing with branches, this is good to go
otherwise.
Attachment #188161 -
Flags: review- → review+
Comment 5•19 years ago
|
||
If Max means "when 2.20rc1 is released, the 2.20 branch will be created (in
CVS)" then I think the Changes page needs a 2.20 branch column containing a link
to "2.20rc -> Today" changelog as appeared to happen when 2.18rc1 was released,
as far as I can tell.
If he doesn't, then ignore this...
Comment 6•19 years ago
|
||
I agree with Colin. AFAIK we will unfreeze when we release 2.20rc1. In this case
this patch is wrong.
Comment 7•19 years ago
|
||
(23:50:39) LpSolit: patches will be submitted independently on the trunk and the
2.20 branch. We then need a new column as Colin said
(23:51:00) LpSolit: else you would get a single query for both
(23:51:11) Zornreich: hmm..
(23:52:09) LpSolit: we need the BUGZILLA-2_20-BRANCH tag too
Comment 8•19 years ago
|
||
Comment on attachment 188161 [details] [diff] [review]
Changes page
Uhh, I'm even more confused now but I changed my mind back. Sorry for bugspam
and anybody more qualified that me can feel free to overrule my r-.
Attachment #188161 -
Flags: review+ → review-
Assignee | ||
Comment 9•19 years ago
|
||
OK, fixed it. I'll probably check this into the web site without r+, but you
can still look it over to make sure I didn't miss anything. :-)
Assignee | ||
Updated•19 years ago
|
Attachment #188161 -
Attachment is obsolete: true
Attachment #188634 -
Flags: review?(wicked)
Assignee | ||
Comment 10•19 years ago
|
||
OK, web site updates committed, commit message in bug 297590.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 11•19 years ago
|
||
Comment on attachment 188634 [details] [diff] [review]
v2
Live changes page looks and functions fine. It doesn't even have any validation
warnings. So this patch must be good, I'll give it a posthumous r+. :)
Attachment #188634 -
Flags: review?(wicked) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•