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

Repos to publish fix #82

Merged
merged 2 commits into from
Mar 30, 2016
Merged

Conversation

edarzins
Copy link
Contributor

Bug fix. The "parse_names" function created "repos_to_publish", but did not return it!

@@ -57,6 +57,7 @@ def parse_names(names, label_type)
repos_to_publish << { label_type.to_s => k }
end
end
return repos_to_publish

Choose a reason for hiding this comment

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

Redundant return detected.

@sepulworld sepulworld merged commit 055feb8 into sepulworld:master Mar 30, 2016
@sepulworld
Copy link
Owner

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.

None yet

3 participants