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

README: Renamed default branch #808

Merged
merged 1 commit into from
May 14, 2021
Merged

README: Renamed default branch #808

merged 1 commit into from
May 14, 2021

Conversation

olleolleolle
Copy link
Contributor

Summary

This is a start of "get the default branch name" used everywhere.

Details

A name change was started, this keeps it going.

Motivation and Context

It was mentioned in cucumber/common#1507

How Has This Been Tested?

Not tested yet!

I searched, and found more places than this, but this is a start.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Internal change (refactoring, test improvements, developer experience or update of dependencies)

Checklist:

  • I've added tests for my code
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@olleolleolle olleolleolle requested a review from mvz May 14, 2021 15:35
@mvz
Copy link
Contributor

mvz commented May 14, 2021

Can you also adjust the GitHub actions, please? No build is triggered when merging, currently.

@mvz
Copy link
Contributor

mvz commented May 14, 2021

No build is triggered for pull requests, either. Really, this is all going in the wrong order 😞

@mvz
Copy link
Contributor

mvz commented May 14, 2021

I'm going to go ahead and merge this. Then we can tackle the rest later.

@mvz mvz merged commit 920c0f9 into main May 14, 2021
@mvz mvz deleted the readme-mentions-main branch May 14, 2021 19:04
@olleolleolle
Copy link
Contributor Author

Sorry I was away from a big screen, hope things turn out OK!

@mvz
Copy link
Contributor

mvz commented May 15, 2021

Yes, everything seems to work now, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants