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

Install instructions are incorrect #20

Closed
mefellows opened this issue May 30, 2023 · 8 comments · Fixed by #22, #23 or #24
Closed

Install instructions are incorrect #20

mefellows opened this issue May 30, 2023 · 8 comments · Fixed by #22, #23 or #24

Comments

@mefellows
Copy link

This line in the docs is incorrect:

✗ pact-plugin-cli -y install https://github.com/austek/pact-avro-plugin/releases/latest

2023-05-30T07:23:56.024139Z ERROR pact_plugin_cli: error - Downloading manifest file from GitHub

I think the release version needs to be pinned for that to work.

We should instead get it added to the registry so people can just go pact-plugin-cli install -y avro.

@uglyog how does one go about doing that? Or is that something we're reserving for "official" plugins?

@austek
Copy link
Owner

austek commented May 30, 2023

Actually, instructions are correct but the published artifacts are incorrect, the manifest file is, currently, packaged within the tar.gz file, where as pact-plugin-cli expects it to be outside

@mefellows
Copy link
Author

hmm I'm still getting the following:

✗ ~/.pact/bin/pact-plugin-cli -y install https://github.com/austek/pact-avro-plugin/releases/latest
2023-05-30T23:37:42.975134Z ERROR pact_plugin_cli: error - Downloading manifest file from GitHub
Error: ExitCode(unix_exit_status(1))
➜  pact-go git:(example/avro) ✗ ~/.pact/bin/pact-plugin-cli -y install https://github.com/austek/pact-avro-plugin/releases/tag/v0.0.3

2023-05-30T23:37:50.242627Z ERROR pact_plugin_cli: error - Downloading manifest file from GitHub
Error: ExitCode(unix_exit_status(1))

@rholshausen
Copy link

pact-plugin-cli has commands for managing the registry files, and they get checked into GitHub.

@mefellows
Copy link
Author

Ah, so they do. Should we add Avro there or is that reserved for "official" ones?

@rholshausen
Copy link

What are "official" ones?

I can add it, but I was envisioning a plugin author creating a PR with the updated index entries, which we can then merge. This is all undocumented.

@rholshausen
Copy link

@mefellows
Copy link
Author

What are "official" ones?

I don't know, was just trying to understand the process - thanks for the docs, I think that will be super helpful. I've updated the plugin workshop too.

@YOU54F
Copy link
Contributor

YOU54F commented Jul 12, 2023

oh that is nice Ron, will get that in our OSS update this month!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants