Unknown option 'rebase' error #1528

Activity

1. stsewd commented on Jan 3, 2023

   

   stsewd

   on Jan 3, 2023

   Contributor

   More actions

   Options should be passed as kwargs, the argument from pull() is a git reference (like a branch name).

   haven't tested if this works

   repository.remotes['origin'].pull(rebase='')  # or rebase=True?
   
2. sgwilliams-ebsco commented on Jan 3, 2023

   

   sgwilliams-ebsco

   on Jan 3, 2023

   Author

   More actions

   Thanks for the quick response! We'll test out the kwargs approach and report back.
3. sgwilliams-ebsco commented on Jan 3, 2023

   

   sgwilliams-ebsco

   on Jan 3, 2023

   Author

   More actions

   We've confirmed that

   repository.remotes['origin'].pull(rebase='') 
   

   and

   repository.remotes['origin'].pull(rebase=True)
   

   both work.

   I wonder if this should have a been a bump to the major version since it changes the behavior of several of the core git methods. Admittedly, we were passing the options to the pull command incorrectly.

   Anyway, our tests are passing again.

   Thanks for the help!
4. sgwilliams-ebsco

   closed this as completedon Jan 3, 2023
5. Byron commented on Jan 4, 2023

   

   Byron

   on Jan 4, 2023

   Last edited by Byron

   Member

   More actions

   I wonder if this should have a been a bump to the major version since it changes the behavior of several of the core git methods.

   It's the trade-off between not causing surprises during automated upgrades and protecting unsuspecting uses of GitPython from harm. For critical fixes GitPython will typically choose a patch release even if these have the potential to break some users.