Closed sroberti closed 3 years ago
@sroberti, could you rebase your PR onto the master, there's something odd, like it has 34 pages while the master has 43 pages. I suspect, based on the diff, that merging it will go just fine (maybe I'll try that local) but just in case, maybe you can rebase instead, then either force push on that same branch or create a new PR.
Never mind, @sroberti, it works fine locally so I'm merging. In the future, my recommendation is when you have a long branch that hasn't been shared yet (no PR yet) you can rebase onto the master before creating a PR, that results into a cleaner history.
Thanks @sroberti, very appreciated. Could you in a subsequent PR include your code on the nomadic approach and reference it in the report?