saintsystems / odata-client-php

OData Client Library for PHP
MIT License
141 stars 103 forks source link

Separate commits for different tasks #144

Closed cweiske closed 11 months ago

cweiske commented 11 months ago

I just upgraded from 0.6 to 0.7.1 and things broke. During debugging I wanted to see the changes in ODataRequest::execute(), did a git blame and found commit c34498a1c01a3ddac2ff5b5045f72da5cc2d27b9.

Unfortunately this commit mixes up

It would be really nice if you would split those changes in the future in separate commits (e.g. via git add -p) so that it's easier to see what changed. Looking at the commits is the only thing we can do as long as there are no upgrade guides, so they are important.

anderly commented 11 months ago

@cweiske, you can see the separate commits via the PR, but these were squashed with the PR merge.