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

publish_repo and publish_drop did not parse prefix correctly #78

Merged
merged 1 commit into from
Mar 19, 2016

Conversation

edarzins
Copy link
Contributor

"publish_repo" and publish_drop did not parse the prefix correctly, so repos could only be published to the default prefix ".", and repos could not be dropped

@@ -106,7 +108,7 @@ def publish_repo(names, publish_options={})
end

if publish_options[:prefix]
uri = uri + publish_options[:prefix]
uri = uri + "/#{publish_options[:prefix]}"

Choose a reason for hiding this comment

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

Use self-assignment shorthand +=.

@edarzins edarzins changed the title publish_repo and publish_drop did parse prefix correctly publish_repo and publish_drop did not parse prefix correctly Mar 18, 2016
@sepulworld
Copy link
Owner

Thank you for the bug fix! I will release a new gem with this patch

sepulworld added a commit that referenced this pull request Mar 19, 2016
publish_repo and publish_drop did not parse prefix correctly
@sepulworld sepulworld merged commit 01fe666 into sepulworld:master Mar 19, 2016
@sepulworld
Copy link
Owner

https://rubygems.org/gems/aptly_cli/versions/0.2.3 -- new Gem released

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.

3 participants