JuliaAI / MLJLinearModels.jl

Generalized Linear Regressions Models (penalized regressions, robust regressions, ...)
MIT License
80 stars 13 forks source link

dev branch woes #83

Closed ablaom closed 3 years ago

ablaom commented 3 years ago

@tlienart

I tried pushing something to dev but discovered post facto that it was not in sync with master. There are a lot of merge conflicts to resolve when I try to sync. It looks like a number of PR's may have been merged directly onto master.

I propose deleting dev and starting over, trusting to current master. There appear to be no commits on master since last patch.

Can you confirm this is okay, thanks.

DilumAluthge commented 3 years ago

In the future, it might be easier to set dev as the default branch for this repo. That way, when someone opens a PR, GitHub will set dev as the target branch of the PR by default.

This might help reduce the frequency of people accidentally merging PRs onto master when they should have merged into dev.

Instructions here: https://docs.github.com/en/free-pro-team@latest/github/administering-a-repository/changing-the-default-branch#changing-the-default-branch

tlienart commented 3 years ago

@ablaom sure 👍🏼

I'll try to clean up some of that this morning (and merge #83)

tlienart commented 3 years ago

alright, I cleaned up old stale branches, brought in #83 (update to TagBot) and synced the dev branch. Let me know if there's something more I can do!

tlienart commented 3 years ago

ah and following @DilumAluthge 's suggestion I made dev the default branch.

tlienart commented 3 years ago

looks like I have a bit more work to do some tests are failing

tlienart commented 3 years ago

all fixed.