Skip to content

Conversation

@gitlost
Copy link
Contributor

@gitlost gitlost commented Jan 8, 2018

Closes #51

Adds version arg to check_git_package_name() and passes to new function get_raw_git_version() that returns what should be used in the raw github request (with limited accuracy - eg suffixes such as ^0 are just stripped).

Needed to change an existing test to use the tag name v0.1.0 instead of the version 0.1.0.

@gitlost gitlost changed the title Issue 51 wrong branch Fix package name is retrieved from wrong branch. Jan 8, 2018
@gitlost gitlost added bug command:package-install Related to 'package install' command labels Jan 8, 2018
@gitlost gitlost added this to the 1.0.11 milestone Jan 8, 2018
@schlessera
Copy link
Member

@gitlost We have a separate organization called wp-cli-test to put test repositories in, so that they don't depend on a single person's personal Github account.

I've sent an invite to you so you can create the test package you've referenced in this PR in that organization instead.

@gitlost gitlost requested a review from a team January 8, 2018 22:37
@schlessera schlessera merged commit 1a3eefa into master Jan 13, 2018
@schlessera schlessera deleted the issue-51-wrong-branch branch January 13, 2018 11:22
schlessera added a commit that referenced this pull request Jan 5, 2022
Fix package name is retrieved from wrong branch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug command:package-install Related to 'package install' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package name is retrieved from wrong branch

3 participants