WIP release notes for 2.0

Review Request #4861 — Created Oct. 30, 2013 and submitted

Information

Review Board
master

Reviewers

WIP release notes for 2.0

This change adds release notes for almost everything that's gone into the master
branch so far. There are a few things that are intentionally left out because
they're not ready for user consumption.

Built releasenotes docs and checked it out.

Description From Last Updated

That will have to be updated now.

chipx86chipx86

Missing a trailing period.

chipx86chipx86

Hmm, you shouldn't need this format. I always do: * Blah blah blah. * Blah blah..

chipx86chipx86

"request"

chipx86chipx86

Best to do Markdown_ and then have a block below like: .. _Markdown: http://.... Easier to reuse, and no links …

chipx86chipx86

Should be consistent with using "Markdown".

chipx86chipx86

We should link to the page describing GitHub's extensions.

chipx86chipx86

Can we make this a little more explicit about how this was old behavior? Maybe just put "previously" in there.

chipx86chipx86

"thumbnails"

chipx86chipx86

Can you use :guilabel: for these?

chipx86chipx86

This made it into 1.7.16, and is documented there.

chipx86chipx86

Hmm, I believe this is in 1.7.5. Certainly, 1.7.x releases already do this.

chipx86chipx86

Can you link to this resource? Should be able to do: :ref:`Repository resource <webapi2.0-repository-resource>`

chipx86chipx86

Same with Repository List resource.

chipx86chipx86

Same with these resources.

chipx86chipx86

We'll need these filled in for the beta.

chipx86chipx86

Should link this resource.

chipx86chipx86

"JavaScript"

chipx86chipx86

These tags are on all pages we ship. I think it's confusing to have them in the release notes.

chipx86chipx86

Can we have literal quotes around that path?

chipx86chipx86

"one level of children" is a bit confusing to me. I don't remember what's in this hook, so I have …

chipx86chipx86

We should have a blurb about this for the beta.

chipx86chipx86

"a consolidated database representation" is kind of overly technical-sounding and doesn't mean anything to anybody who doesn't already know how …

chipx86chipx86

"have to make" or "has to make"? "have" sounds odd here.

chipx86chipx86

"JavaScript"

chipx86chipx86

We weren't using this jsmin. We were using a separate Python-based jsmin rewrite that did not include this term. The …

chipx86chipx86

Should use jsmin_ and have a .. _jsmin: http:// link below.

chipx86chipx86

"JavaScript"

chipx86chipx86

Should use UglifyJS_ and .. _UglifyJS: http://.

chipx86chipx86

Given the above, it's not worth talking about the license here. Better to say it offers improvements over the version …

chipx86chipx86

Can you use literal quotes for these fields?

chipx86chipx86

Here, too: issue_opened, issue_status, open, null.

chipx86chipx86

Should probably use :guilabel: here.

chipx86chipx86

"offter"?

chipx86chipx86

Should use literal quotes around /r/ here and in the description.

chipx86chipx86

We're missing at least two names (Po-Chien Lin and Alessandro Menti). Should go through and make sure everyone is listed.

chipx86chipx86

Let's not actually link this yet. I don't want to confuse people who view the release notes. We can just …

chipx86chipx86

Two blank lines.

chipx86chipx86

1.7.18.

chipx86chipx86

Now that 1.7.18 has this, we can remove it from these release notes.

chipx86chipx86
chipx86
  1. We should have a 2.0 beta 1 set of release notes that are ready for pushing this week. I made some notes about a couple things that still need to be filled in for this, and things that should be removed.

  2. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    That will have to be updated now.

  3. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     
     
     
     

    This isn't ready for alpha.

  4. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     
     
     
     

    Nor this.

  5. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    Missing a trailing period.

  6. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     

    Hmm, you shouldn't need this format. I always do:

    * Blah blah
      blah.
    
    * Blah blah..
    
  7. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "request"

  8. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Best to do Markdown_ and then have a block below like:

    .. _Markdown: http://....
    

    Easier to reuse, and no links in the text.

  9. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should be consistent with using "Markdown".

  10. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    We should link to the page describing GitHub's extensions.

    1. There's not really a great place to link. GitHub's page is 99% about features that we don't support (linking to github issues! linking to github users! ...). I think our own docs are a better overview.

    2. Yeah, that's fair.

  11. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    Can we make this a little more explicit about how this was old behavior? Maybe just put "previously" in there.

  12. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "thumbnails"

  13. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    Can you use :guilabel: for these?

  14. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    This made it into 1.7.16, and is documented there.

  15. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Hmm, I believe this is in 1.7.5. Certainly, 1.7.x releases already do this.

  16. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Can you link to this resource? Should be able to do:

    :ref:`Repository resource <webapi2.0-repository-resource>`
    
    1. Why are these dropped?

    2. The highlighting goofed. Looks like there's a bug. That SHOULD have read: :ref:`Repository resource <webapi2.0-repository-resource>`

  17. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Same with Repository List resource.

  18. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Same with these resources.

  19. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     

    We'll need these filled in for the beta.

  20. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should link this resource.

  21. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "JavaScript"

  22. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    These tags are on all pages we ship. I think it's confusing to have them in the release notes.

  23. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Can we have literal quotes around that path?

  24. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     

    "one level of children" is a bit confusing to me. I don't remember what's in this hook, so I have no idea what this means.

  25. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     

    We should have a blurb about this for the beta.

  26. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "a consolidated database representation" is kind of overly technical-sounding and doesn't mean anything to anybody who doesn't already know how this works. How about:

    Starting in Review Board 1.7, newly uploaded diffs of files would be stored in the database only once, saving storage space if that particular file was used repeatedly in parent diffs or remained unchanged across several iterations of a change.
    
  27. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "have to make" or "has to make"? "have" sounds odd here.

    1. "have" is correct when "web browsers" is plural. That said, I'll try to make this less clunky.

  28. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "JavaScript"

  29. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     
     
     

    We weren't using this jsmin. We were using a separate Python-based jsmin rewrite that did not include this term.

    The main reason we're leaving the Python jsmin is because it's largely unmaintained and does not offer some of the benefits of UglifyJS.

  30. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should use jsmin_ and have a .. _jsmin: http:// link below.

  31. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "JavaScript"

  32. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should use UglifyJS_ and .. _UglifyJS: http://.

  33. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Given the above, it's not worth talking about the license here. Better to say it offers improvements over the version of jsmin we were using.

  34. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Can you use literal quotes for these fields?

  35. docs/releasenotes/2.0.txt (Diff revision 1)
     
     
     
     

    Here, too: issue_opened, issue_status, open, null.

  36. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should probably use :guilabel: here.

  37. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    "offter"?

  38. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    Should use literal quotes around /r/ here and in the description.

  39. docs/releasenotes/2.0.txt (Diff revision 1)
     
     

    We're missing at least two names (Po-Chien Lin and Alessandro Menti). Should go through and make sure everyone is listed.

    1. The rest of the list was created from git logs. It's just I missed these two because I dropped localization in one big blob.

  40. docs/releasenotes/index.txt (Diff revision 1)
     
     
     
     
     
     
     
     

    Let's not actually link this yet. I don't want to confuse people who view the release notes. We can just keep it in the tree instead.

    1. Even if this is called 2.0-beta1 ?

    2. Totally fine with it as 2.0 beta 1. Just don't want to show a final 2.0 yet.

  41. docs/releasenotes/index.txt (Diff revision 1)
     
     
     
     

    Two blank lines.

  42. 
      
david
chipx86
  1. 
      
  2. docs/releasenotes/2.0-beta-1.txt (Diff revision 2)
     
     

    1.7.18.

  3. docs/releasenotes/2.0-beta-1.txt (Diff revision 2)
     
     
     
     
     
     
     
     
     
     
     
     

    Now that 1.7.18 has this, we can remove it from these release notes.

  4. 
      
david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Loading...