2012/2/15 mark florisson <markflorisso...@gmail.com>: > On 15 February 2012 09:37, Stefan Behnel <stefan...@behnel.de> wrote: >> Hi, >> >> I'd like to suggest that instead of overwriting pull requests and all of >> their comments on github by pushing replaced commits over them, it would be >> better to keep any existing discussions accessible by rejecting the current >> pull request and creating a new one. >> >> Does that make sense for everyone? >> >> Stefan >> _______________________________________________ >> cython-devel mailing list >> cython-devel@python.org >> http://mail.python.org/mailman/listinfo/cython-devel > > Hm, rebasing + force pushing is very common for pull requests, does > that delete the inline (or all) comments?
Yeah, vanished comments are really annoying on the other hand it's really hard to avoid rebases and forced pusing. Forced pushing (via git rebase -i HEAD~N) is rather useful to cleanup commits: fix typos, obvious bugs and so on. -- vitja. _______________________________________________ cython-devel mailing list cython-devel@python.org http://mail.python.org/mailman/listinfo/cython-devel
