Skip to content

Edit DOC: consistent imports (GH9886) part V #10941

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 1, 2015

Conversation

springcoil
Copy link
Contributor

Updated - but no idea why this is such a messed up branch.

@jreback jreback added the Compat pandas objects compatability with Numpy or Python functions label Aug 31, 2015
@jreback
Copy link
Contributor

jreback commented Aug 31, 2015

needs a rebase

@jorisvandenbossche
Copy link
Member

@springcoil I don't really understand how you manage to such a mess of your commits :-) (we had better looked at it together yesterday)

Some things:

  • if you start to work on something new (eg updating the imports in merging.rst), no rebase or something is needed. Just be sure to start your branch from the latest upstream master. Rebasing your branch is only needed once there are conflicts because other people have changed the same file. Which is not the case here at all.

  • To start a new branch from upstream master, just do:

    git checkout master
    git fetch upstream
    git merge --ff-only upstream/master
    git checkout -b new_branch
    
  • When you need to rebase (which will probably not be the case here if you did like above), simply do the following:

    git fetch upstream
    git checkout new_branch
    git rebase upstream/master
    

    and this should almost always work.

Don't close this PR, just try to clean up this branch and force push. To clean this up, do exactly the following:

git fetch upstream
git checkout merging
git rebase -i upstream/master
# then in overview of commits, remove all commits apart from the last one 'updating the docs' (the only commit with the actual changes I think), and leave that one as 'pick'
git push -f origin merging

If you do exactly as above, this should be fine

@jorisvandenbossche jorisvandenbossche added Docs and removed Compat pandas objects compatability with Numpy or Python functions labels Aug 31, 2015
@jorisvandenbossche jorisvandenbossche added this to the 0.17.0 milestone Aug 31, 2015
@springcoil
Copy link
Contributor Author

Ok will do that thanks.

On Mon, Aug 31, 2015 at 12:45 PM, Joris Van den Bossche <
[email protected]> wrote:

@springcoil https://github.com/springcoil I don't really understand how
you manage to such a mess of your commits :-) (we had better looked at it
together yesterday)

Some things:

if you start to work on something new (eg updating the imports in
merging.rst), no rebase or something is needed. Just be sure to start your
branch from the latest upstream master. Rebasing is only needed once there
are conflicts because other people have changed the same file. Which is not
the case here at all.

To start a new branch from upstream master, just do:

git checkout master
git fetch upstream
git merge --ff-only upstream/master
git checkout -b new_branch

When you need to rebase (which will probably not be the case here if
you did like above), simply do the following:

git fetch upstream
git checkout new_branch
git rebase upstream/master

and this should almost always work.

Don't close this PR, just try to clean up this branch and force push. To
clean this up, do exactly the following:

git fetch upstream
git checkout merging
git rebase -i upstream/master

then in overview of commits, remove all commits apart from the last one 'updating the docs' (the only commit with the actual changes I think), and leave that one as 'pick'

git push -f origin merging

If you do exactly as above, this should be fine


Reply to this email directly or view it on GitHub
#10941 (comment).

Peadar Coyle
Skype: springcoilarch
www.twitter.com/springcoil
peadarcoyle.wordpress.com

@springcoil
Copy link
Contributor Author

Thanks for the help - @jreback and @jorisvandenbossche Ok @jorisvandenbossche all the checks passed. Is this ready to be merged?

@jorisvandenbossche
Copy link
Member

Yep, thanks!

jorisvandenbossche added a commit that referenced this pull request Sep 1, 2015
  Edit DOC: consistent imports (GH9886) part V
@jorisvandenbossche jorisvandenbossche merged commit 92cb9e2 into pandas-dev:master Sep 1, 2015
@springcoil springcoil deleted the merging branch September 1, 2015 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants