Summary

Warn about dirty directory when running 'rbt post' with Mercurial repository

Review Request #6082 — Created July 8, 2014 and submitted July 8, 2014, 2:47 p.m.

Information

Owner

bcran

Repository

RBTools

Branch

release-0.6.x

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

As with Git, the Mercurial backend only posts changesets and ignores uncommitted changes. Warn users about this, since they may expect to be able to post the same output as 'hg diff'.

Testing Done

Ran 'rbt post' both with and without uncommitted changes.
I'm not sure if we want the '--deleted' flag included, since it's files that are tracked but have been deleted - e.g. using 'rm file' instead of 'hg rm file'.

Issues

Description	From	Last Updated
Col: 80 E501 line too long (93 > 79 characters)	reviewbot	July 9, 2014, 2 p.m.

Description:

~		As with Git, the Mercurial backend only posts changesets and ignored uncommitted changes. Warn users about this, since they may expect to be able to post the same output as 'hg diff'.
	~	As with Git, the Mercurial backend only posts changesets and ignores uncommitted changes. Warn users about this, since they may expect to be able to post the same output as 'hg diff'.
-
-		I'm not sure if we want the '--deleted' flag included, since it's files that are tracked but have been deleted - e.g. using 'rm file' instead of 'hg rm file'.

Testing Done:

~		Ran 'rbt post' both with and without uncommitted changes.
	~	Ran 'rbt post' both with and without uncommitted changes.
	+	I'm not sure if we want the '--deleted' flag included, since it's files that are tracked but have been deleted - e.g. using 'rm file' instead of 'hg rm file'.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (5e4a1b1)

Fix it!

Tool: PEP8 Style Checker
Processed Files:
    rbtools/clients/mercurial.py



Tool: Pyflakes
Processed Files:
    rbtools/clients/mercurial.py

rbtools/clients/mercurial.py (Diff revision 1)
Show all issues
```
Col: 80
 E501 line too long (93 > 79 characters)
```