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

Make whipper not break on missing release type #398

Merged
merged 5 commits into from
May 3, 2019

Conversation

Freso
Copy link
Member

@Freso Freso commented May 2, 2019

Fix for #396

@Freso Freso force-pushed the bugfix/issue-396-no-mb-release-group-type branch from 7fb848b to 9615405 Compare May 2, 2019 16:29
See #396

Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
@Freso Freso force-pushed the bugfix/issue-396-no-mb-release-group-type branch from 9615405 to 8209927 Compare May 2, 2019 16:35
Fixes #396

Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
@@ -1,5 +1,5 @@
# -*- Mode: Python; test-case-name: whipper.test.test_common_mbngs -*-
# vi:si:et:sw=4:sts=4:ts=4
# vi:si:et:sw=4:sts=4:ts=4:set fileencoding=utf-8
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The encoding is required to be defined due to the “Ö” in the comment to the new test.

whipper/common/mbngs.py Outdated Show resolved Hide resolved
@Freso Freso marked this pull request as ready for review May 2, 2019 16:43
Freso added 2 commits May 3, 2019 02:38
Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
@Freso Freso requested review from JoeLametta and MerlijnWajer May 3, 2019 00:48
Signed-off-by: Frederik “Freso” S. Olesen <[email protected]>
@Freso Freso force-pushed the bugfix/issue-396-no-mb-release-group-type branch from b4aec85 to bf8daf1 Compare May 3, 2019 00:49
@JoeLametta JoeLametta merged commit a9b6e3c into develop May 3, 2019
@JoeLametta
Copy link
Collaborator

Merged, thanks!

@Freso Freso deleted the bugfix/issue-396-no-mb-release-group-type branch May 3, 2019 08:37
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