Summary

Add draft release notes for RBTools 0.7

Review Request #6720 — Created Jan. 1, 2015 and submitted Jan. 7, 2015, 1:22 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Commit

7c8faf6...

Reviewers

Groups

rbtools

People

Description

This change has draft relnotes for everything that is present in master but not
release-0.6.x.

Testing Done

Issues

Description	From	Last Updated
We should link to the aliases docs. (Will need to define a reference point in rbt/aliases.rst along with this).	chipx86	Jan. 6, 2015, 4:19 p.m.
"command lines".	chipx86	Jan. 3, 2015, 2:22 p.m.
We should mention that the Windows installer is based on work by Bruce Cran, and make sure he's in the …	chipx86	Jan. 3, 2015, 2:23 p.m.
"There are now installers ..."	chipx86	Jan. 3, 2015, 2:23 p.m.
Let's not say "less-technical users." I don't think people will take kindly to that. Really, we want people to use …	chipx86	Jan. 3, 2015, 2:23 p.m.
"include an"	chipx86	Jan. 3, 2015, 2:23 p.m.
"config file"	chipx86	Jan. 3, 2015, 2:23 p.m.
This should also mention that the patch is by Barret.	chipx86	Jan. 3, 2015, 2:24 p.m.
To get the options to link correctly, can you add this below the section and before the bullet points: :program: …	chipx86	Jan. 3, 2015, 2:25 p.m.
"Added" and "command line"	chipx86	Jan. 3, 2015, 2:25 p.m.
This is only one half of the command. I would say something like: "The new rbt land command is a …	chipx86	Jan. 3, 2015, 2:25 p.m.
This is now optional and doesn't happen by default.	chipx86	Jan. 3, 2015, 2:26 p.m.
"Added"	chipx86	Jan. 3, 2015, 2:27 p.m.
Missing period.	chipx86	Jan. 3, 2015, 2:27 p.m.
"Enhanced"	chipx86	Jan. 3, 2015, 2:27 p.m.
"unapplied"	chipx86	Jan. 3, 2015, 2:28 p.m.
"Added" and "command line"	chipx86	Jan. 3, 2015, 2:28 p.m.
"command line" and "config file"	chipx86	Jan. 3, 2015, 2:28 p.m.
We should mention that this only happens on RB 2.0+.	chipx86	Jan. 3, 2015, 2:29 p.m.
Can we move the second sentence into its own paragraph, like we did above for rbt land?	chipx86	Jan. 3, 2015, 2:29 p.m.
Can you move the :command: bit to the next line? I don't remember if ReST technically allows these to span …	chipx86	Jan. 3, 2015, 2:29 p.m.
Can we put "The old behavior ..." in a new paragraph? Helps it stand out.	chipx86	Jan. 3, 2015, 2:29 p.m.
Can we mention that this also works with the new -C?	chipx86	Jan. 3, 2015, 2:30 p.m.
This somehow reads a bit oddly. Maybe: "Subversion repositories configured to require user credentials for each command could not be …	chipx86	Jan. 3, 2015, 2:30 p.m.
"config file"	chipx86	Jan. 3, 2015, 2:30 p.m.
Same comment about "The old behavior"	chipx86	Jan. 3, 2015, 2:31 p.m.
Let's link to Power Pack. Also, we should drop the "the".	chipx86	Jan. 6, 2015, 4:37 p.m.
No period.	chipx86	Jan. 3, 2015, 2:31 p.m.
No period.	chipx86	Jan. 3, 2015, 2:31 p.m.
No period.	chipx86	Jan. 3, 2015, 2:31 p.m.
Sorry, I had this wrong. Should be: .. program:: rbt patch	chipx86	Jan. 7, 2015, 1:17 p.m.
Can we make the "only supported" sentence its own paragraph after this one, to match the other entries that do …	chipx86	Jan. 7, 2015, 1:13 p.m.

Tool: PEP8 Style Checker
Ignored Files:
    docs/releasenotes/0.7.rst



Tool: Pyflakes
Ignored Files:
    docs/releasenotes/0.7.rst

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

We should link to the aliases docs. (Will need to define a reference point in rbt/aliases.rst along with this).

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"command lines".
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

We should mention that the Windows installer is based on work by Bruce Cran, and make sure he's in the Contributors list.

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"There are now installers ..."
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

Let's not say "less-technical users." I don't think people will take kindly to that. Really, we want people to use the installers anyway for these platforms, as it's guaranteeing a particular environment, making our lives easier too.

How about just "This simplifies distribution and installation on these platforms."

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"include an"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"config file"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
This should also mention that the patch is by Barret.
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

To get the options to link correctly, can you add this below the section and before the bullet points:

:program: rbt diff

Same with other sections.

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"Added" and "command line"
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

This is only one half of the command.

I would say something like:

"The new rbt land command is a quick and easy way to push a change that has been reviewed on Review Board to the upstream repository. The change may be in a local branch or stored as a patch on Review Board."

I'd also make the "When running against ..." a separate paragraph.

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
This is now optional and doesn't happen by default.
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"Added"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Missing period.
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"Enhanced"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"unapplied"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"Added" and "command line"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"command line" and "config file"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
We should mention that this only happens on RB 2.0+.
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

Can we move the second sentence into its own paragraph, like we did above for rbt land?

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

Can you move the :command: bit to the next line? I don't remember if ReST technically allows these to span multiple lines, but it definitely messes up syntax highlighting here.

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

Can we put "The old behavior ..." in a new paragraph? Helps it stand out.

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Can we mention that this also works with the new -C?
```

docs/releasenotes/0.7.rst (Diff revision 1)

The issue has been resolved. Show all issues

This somehow reads a bit oddly. Maybe:

"Subversion repositories configured to require user credentials for each command could not be used with RBTools. These credentials can now be provided using the new --svn-username and ..."

docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
"config file"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Same comment about "The old behavior"
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
Let's link to Power Pack. Also, we should drop the "the".
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
No period.
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
No period.
```
docs/releasenotes/0.7.rst (Diff revision 1)
The issue has been resolved. Show all issues
```
No period.
```

Commit:

3ae9ee699cb4e0ecb0c882f7b7552534551ae86f

55e89b83ea3459b89842df17cc9e4812cea2b54a

Diff:

Revision 2 (+259)

Show changes

	docs/rbtools/rbt/aliases.rst
	docs/releasenotes/0.7.rst
	docs/releasenotes/index.rst

Tool: PEP8 Style Checker
Ignored Files:
    docs/rbtools/rbt/aliases.rst
    docs/releasenotes/index.rst
    docs/releasenotes/0.7.rst



Tool: Pyflakes
Ignored Files:
    docs/rbtools/rbt/aliases.rst
    docs/releasenotes/index.rst
    docs/releasenotes/0.7.rst

docs/releasenotes/0.7.rst (Diff revision 2)

The issue has been resolved. Show all issues

Sorry, I had this wrong. Should be:

.. program:: rbt patch

docs/releasenotes/0.7.rst (Diff revision 2)

The issue has been resolved. Show all issues

Can we make the "only supported" sentence its own paragraph after this one, to match the other entries that do that?

Commit:

55e89b83ea3459b89842df17cc9e4812cea2b54a

7c8faf6ac61f114141d1fe36ee16a97690866329

Diff:

Revision 3 (+260)

Show changes

	docs/rbtools/rbt/aliases.rst
	docs/releasenotes/0.7.rst
	docs/releasenotes/index.rst

Tool: PEP8 Style Checker
Ignored Files:
    docs/rbtools/rbt/aliases.rst
    docs/releasenotes/index.rst
    docs/releasenotes/0.7.rst



Tool: Pyflakes
Ignored Files:
    docs/rbtools/rbt/aliases.rst
    docs/releasenotes/index.rst
    docs/releasenotes/0.7.rst

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (786aa63)