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

Allow empty track name tags #55

Merged
merged 2 commits into from
Aug 3, 2021
Merged

Allow empty track name tags #55

merged 2 commits into from
Aug 3, 2021

Conversation

soruh
Copy link
Contributor

@soruh soruh commented Aug 3, 2021

  • I agree to follow the project's code of conduct.
  • I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Fixes #54

@urschrei
Copy link
Member

urschrei commented Aug 3, 2021

lgtm! Can you add a note to the changelog if you're happy, @soruh, and then we'll get this merged?

@soruh
Copy link
Contributor Author

soruh commented Aug 3, 2021

Sure, I wasn't completely sure where to add it, should I just add a new section for 0.8.3 and put it there?

@soruh
Copy link
Contributor Author

soruh commented Aug 3, 2021

Does something like this look okay?

@urschrei
Copy link
Member

urschrei commented Aug 3, 2021

Ah, sorry, should have been clearer. We usually add changes to an ## Unreleased section until we're ready to go, but this seems like an annoying oversight so I'm just gonna cut 0.8.3 after we merge. Thank you!

@urschrei
Copy link
Member

urschrei commented Aug 3, 2021

bors try

bors bot added a commit that referenced this pull request Aug 3, 2021
@soruh
Copy link
Contributor Author

soruh commented Aug 3, 2021

Ah, that makes sense, thank you.

@bors
Copy link
Contributor

bors bot commented Aug 3, 2021

try

Build succeeded:

@urschrei
Copy link
Member

urschrei commented Aug 3, 2021

bors r=lnicola

@bors
Copy link
Contributor

bors bot commented Aug 3, 2021

Build succeeded:

@bors bors bot merged commit 213c7a4 into georust:master Aug 3, 2021
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.

Panic for tracks with empty name
3 participants