Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should $(GitDefaultBranch) now default to main rather than master #167

Closed
dellis1972 opened this issue Aug 13, 2021 · 4 comments · Fixed by #225
Closed

Should $(GitDefaultBranch) now default to main rather than master #167

dellis1972 opened this issue Aug 13, 2021 · 4 comments · Fixed by #225

Comments

@dellis1972
Copy link

GitHub has changed its default branch from master to main.
Should GitDefaultBranch be updated to match?

@kzu
Copy link
Member

kzu commented Aug 20, 2021

Perhaps. It would be a breaking change for folks and for branch commit count from fork point (if the branch remains 'master'), so perhaps this should go with a notice and a major version change?

@dellis1972
Copy link
Author

Or issue a warning now, and then change in a few months along with a major version change?

@kzu hope you are doing well btw :D

@dellis1972
Copy link
Author

I can make the required changes for the warning if you like and do a PR.

@kzu
Copy link
Member

kzu commented Aug 24, 2021

Hey @dellis1972! Thanks for dropping by 🤟 .

So perhaps the logic for the warning would be: you don't have a GitDefaultBranch property set (meaning next major will break you). Should we warn/info if you have it set to main, saying it won't be necessary down the road? Or no warning at all in that case? (I kinda dislike it when projects duplicate values that equal their defaults, since that's just clutter).

Maybe by the following major version when the new default is introduced, the former info (that you're setting a value that will not be needed later on), becomes a warning instead that you don't need to set it?

kzu added a commit that referenced this issue Feb 6, 2023
It's been a while and most repos I encounter nowadays have switched.

Update docs, change to new default.

Closes #167
@kzu kzu closed this as completed in #225 Feb 6, 2023
kzu added a commit that referenced this issue Feb 6, 2023
It's been a while and most repos I encounter nowadays have switched.

Update docs, change to new default.

Closes #167
@devlooped devlooped locked and limited conversation to collaborators Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants