Skip to content
This repository has been archived by the owner on Jun 26, 2024. It is now read-only.

Support provider: github #17

Closed
wilsonehusin opened this issue Mar 18, 2022 · 3 comments · Fixed by #56
Closed

Support provider: github #17

wilsonehusin opened this issue Mar 18, 2022 · 3 comments · Fixed by #56
Labels
enhancement New feature or request ux Ensures a good time when using the product
Milestone

Comments

@wilsonehusin
Copy link
Member

wilsonehusin commented Mar 18, 2022

Currently, programs need to specify complete URL template.

This is helpful to support all sources. However, most programs come from GitHub releases. Would be nice to have something simplified and auto-detect the common pattern (or lookup from GitHub API), such that the config manifest can look simpler like:

name: myprogram
version: 0.1.0
provider: github
path: myorg/myprogram
overlay:
  Host: https://mygithubenterprise.com  # optional, defaults to "https://github.com"

While the lockfile manifest should remain the same.

(Or at least I don't think there is a need to change it)

@wilsonehusin wilsonehusin added enhancement New feature or request ux Ensures a good time when using the product labels Mar 18, 2022
@wilsonehusin
Copy link
Member Author

wilsonehusin commented Mar 18, 2022

Maybe we should do this in baby steps — I think I would be quite happy to have something like this:

name: myprogram
version: 0.1.0
provider: github
path: myorg/myprogram/_/.tar.gz

Which would generate the URL:

https://github.com/myorg/myprogram/releases/download/v0.1.0/myprogram_0.1.0_Linux_arm64.tar.gz

Alternatively, shove them all in overlay section:

name: myprogram
version: 0.1.0
provider: github
path: myorg/myprogram
overlay:
  Host: "github.com"
  Tag: "v{{ .Version }}"
  Separator: "_"
  Ext: ".tar.gz"
  ArchiveNameVersion: "{{ .Version }}"

@wilsonehusin wilsonehusin added the brainstorm Not yet implementable (e.g. design phase or blocked) label Mar 18, 2022
@wilsonehusin wilsonehusin added this to the v0.1.0 milestone Mar 30, 2022
@wilsonehusin
Copy link
Member Author

Relevant-ish: #35

@wilsonehusin
Copy link
Member Author

The structure of program configuration was re-done in #54, which now should make it closer to the comment above.

@wilsonehusin wilsonehusin removed the brainstorm Not yet implementable (e.g. design phase or blocked) label Apr 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request ux Ensures a good time when using the product
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant