Skip to content

Agnostic - #117

Closed
deweysasser wants to merge 7 commits into
RichiH:masterfrom
deweysasser:agnostic
Closed

Agnostic#117
deweysasser wants to merge 7 commits into
RichiH:masterfrom
deweysasser:agnostic

Conversation

@deweysasser

Copy link
Copy Markdown

Adds directory tree ascension to PR #115 for locating vcsh config file.

@deweysasser

Copy link
Copy Markdown
Author

And...I really should learn not to stay up late hacking on this -- it would result in less pull request confusion. Cancelling this one in favor of a PR which updates the man page as well.

@deweysasser deweysasser closed this Mar 7, 2014
@alerque

alerque commented Mar 7, 2014

Copy link
Copy Markdown
Collaborator

Pull requests are for the current differences from one branch to another branch rather that being for a certain set of commits. This means that you can amend them if need be based on feedback in the PR or whatever by pushing changes to your branch. Also if you have changes that are dependent of somebody else's feature branch you can make your PR against their branch instead of the upstream master. Their original PR can then reflect the whole package if they choose to merge it.

@deweysasser

Copy link
Copy Markdown
Author

Very good, thanks.

On March 7, 2014 1:46:42 AM EST, Caleb Maclennan notifications@github.com wrote:

Pull requests are for the current differences from one branch to
another branch rather that being for a certain set of commits. This
means that you can amend them if need be based on feedback in the PR or
whatever by pushing changes to your branch. Also if you have changes
that are dependent of somebody else's feature branch you can make your
PR against their branch instead of the upstream master. Their original
PR can then reflect the whole package if they choose to merge it.


Reply to this email directly or view it on GitHub:
#117 (comment)

Dewey

@deweysasser deweysasser mentioned this pull request Mar 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants