No. Was correct use of rebase. If you had created new branch for this PR and then committed from a commit with work in progress.

e.g.  A -> B -> C (origin/master) ->D (wip) -> E (cpcloud:read-html-lxml-skips)