README: Fix wrong git config record name

Change-Id: I0c1f005e82f92eb5d69b4812a7d8490a0664f1fb
Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
feature finish: Do not add links into merge msg

The problem is that the merge commit needs amending because of the order of
steps that is given. If you, however, need to resolve a merge conflict and push
manually, this amend step inserting the links will fail, because the merge
commit was already pushed upstream manually, which is not what the plugin counts
on. It would be necessary to use git push -f to make amend work, but that is too
dangerous. We decided to simply not append any links for now.

Change-Id: Ie2781dd2d56d509ffcf20d94a0157617a8c2ce3b
Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
feature start: Recognize and skip release stories

The plugin was completely ignoring that there may be some stories of the release
type, which were supposed to be skipped during feature start like chores are.

Change-Id: I289c67293b3c3420adf0d2431f3a924b1acf4347
Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature/ 52543807/feature-start-fails-on-release-story-type

Change-Id: Ic6967905a2d352d15f846f7ddf522a3a101a21be
PT-Story-URL: http://www.pivotaltracker.com/story/show/52543807
RB-Review-Request-URL: https://dev.salsitasoft.com/rb/r/4239/
Finished feature/ 52376723/feature-finish-fails-if-pushed-manually

PT-Story-URL: http://www.pivotaltracker.com/story/show/52376723
RB-Review-Request-URL: https://dev.salsitasoft.com/rb/r/4240/
feature finish: Be sure to fetch before merging

The issue was that must_be_uptodate was issuing git fetch origin develop, but
this command does not update remote-tracking branches. The result was that the
command was failing on push instead of merge.

This patch makes sure that we really fail as soon as possible, that is before
even trying to merge.

Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature 52376373/feature-finish-not-fetching.
release finish: Check reviews before submitting

The issue was that during release finish, the relevant stories were iterated
over and their reviews were submitted one by one. This create an inconsistent
situation once any of the stories is not submitted.

This fix makes the plugin first check for all reviews in one step, and then, if
everything is ok, submit them all at once so that there is no space for any
inconsistency.

Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature 52448949/release-finish-check-reviews-and-submit-all-at-once.
Move assert to the right place
Finished feature 52448949/release-finish-check-reviews-and-submit-all-at-once.
feature start: Handle rejected stories properly

Up until now it was not clear how rejected stories are being handled.

This patch adds explicit handling of rejected stories. They are offered during
feature start, but then it just performs checkout and PT story start. The the
branch does not exist, it prints a warning and fails since the story must exist
because someone must have called feature start in the past, and thus create the
feature branch.

Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature 51962659/handle-rejected-stories-properly.
feature finish: Make rbt error output available

Using subprocess.check_output for the call to rbt was not the best decision
since it just throws an exception in case the return code is not 0. In other
works, there is no way we can get out hand on the error output.

This patch makes the plugin call rbt using subprocess.Popen, using pipes to get
standard error output and print it to the user in case the return code is not 0.

Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature 52458397/feature-finish-improve-error-output.
Update README

Update README to mention release finish --ignore-missing-reviews

Signed-off-by: Ondrej Kupka <ondra.cap@gmail.com>
Finished feature 52232721/document-release-finish-better.
Fix typo

Change-Id: If297f9a767e237625875b871b6ab3a0a88e19b5b
Bump version to 1.0.1

Change-Id: I2c2b8c6f92571f3edb7b5b2dc4cb2529cee17bed
Finished release 1.0.1.