Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

is_dirty supports path. Fixes #482.#496

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Byron merged 1 commit intogitpython-developers:masterfromhaizaar:master
Aug 2, 2016

Conversation

haizaar
Copy link
Contributor

No description provided.

lorenzo-w reacted with rocket emoji
@haizaarhaizaar mentioned this pull requestAug 1, 2016
@ByronByron added this to thev2.0.8 - Bugfixes milestoneAug 2, 2016
@Byron
Copy link
Member

@haizaar Thanks a lot ! The PR already looks good to me. The only thing I was wondering is whether we should start supporting multiple paths right away. It's just thatgit status already does that, at least when looking at it's usage spec:

git status [<options>...] [--] [<pathspec>...]

What do you think ?

@haizaar
Copy link
ContributorAuthor

Hi Sebastian,

I would rather wait until someone requests it. Particularly, because the
project is in maintenance mode, we better only add features that are driven
by user demand.

path argument can be extended in the future to accept list of paths as
well.

On 2 Aug 2016 06:57, "Sebastian Thiel"notifications@github.com wrote:

@haizaarhttps://github.com/haizaar Thanks a lot ! The PR already looks
good to me. The only thing I was wondering is whether we should start
supporting multiple paths right away. It's just that git status already
does that, at least when looking at it's usage spec:

git status [...] [--] [...]

What do you think ?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#496 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADjWaSZizeq8Q0_c595kKwnpePfrwEIks5qbsAYgaJpZM4JZf5s
.

@ByronByron merged commite866c4a intogitpython-developers:masterAug 2, 2016
@Byron
Copy link
Member

Thanks for your contribution, it's much appreciated !

@haizaar
Copy link
ContributorAuthor

Thanks. When do you plan a new PyPi release?

On 2 Aug 2016 8:54 AM, "Sebastian Thiel"notifications@github.com wrote:

Thanks for your contribution, it's much appreciated !


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#496 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADjWSCEaAlx34gTcsrKyVRRwXM5adVXks5qbtuGgaJpZM4JZf5s
.

@Byron
Copy link
Member

v2.0.8 was just released to pypi.

@haizaar
Copy link
ContributorAuthor

Thank you!

On 3 Aug 2016 07:50, "Sebastian Thiel"notifications@github.com wrote:

v2.0.8 was just released to pypi.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#496 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADjWVPxlHc9CMVep2dLBP6eCJo8JrB6ks5qcB4fgaJpZM4JZf5s
.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
@haizaar@Byron

[8]ページ先頭

©2009-2025 Movatter.jp