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

Use porter from the bin or the PATH #1860

Merged
merged 1 commit into from
Jan 12, 2022

Conversation

carolynvs
Copy link
Member

What does this change

First check if we have a local copy of porter and use that to generate a mixin feed. Otherwise assume that porter is installed in the GOPATH.

What issue does it fix

Fixes publishing binaries for porter v1. When I made a change for mixins it broke it for porter... 🤦‍♀️ Happy 2022!

Notes for the reviewer

N/A

Checklist

  • Unit Tests
  • Documentation
  • Schema (porter.yaml)

First check if we have a local copy of porter and use that to generate a
mixin feed. Otherwise assume that porter is installed in the GOPATH.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
@carolynvs
Copy link
Member Author

/azp run test-porter-release

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@carolynvs carolynvs marked this pull request as ready for review January 12, 2022 20:00
@carolynvs carolynvs merged commit ef3ab34 into getporter:release/v1 Jan 12, 2022
@carolynvs carolynvs deleted the fix-release branch January 12, 2022 20:02
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.

1 participant