Skip to content

Conversation

@alice-sawatzky
Copy link

@alice-sawatzky alice-sawatzky commented Aug 10, 2020

add an endpoint at /api/repos to list repos

  • figure out routing??
  • add handler
  • add tests
  • document endpoint

Relies on chartmuseum/storage#48, so once that merges we should revert 5ca02f6

Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

@alice-sawatzky
Copy link
Author

retrigger

Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
…t placeholder

Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
@alice-sawatzky alice-sawatzky marked this pull request as ready for review August 18, 2020 23:58
@jdolitsky jdolitsky changed the base branch from master to main September 13, 2020 20:00
@jdolitsky
Copy link
Contributor

Since this is a large feature, going to slate it for v0.14 so we can release a v0.13. Thanks again for working on this

@alice-sawatzky
Copy link
Author

alice-sawatzky commented Sep 21, 2020

no problem! I was learning go while writing this, so please let me know if there's any issues i've missed and i'll be happy to fix (and learn from it!)

Especially because I really butchered your HTTP router in the process of doing this 😛

Signed-off-by: Alice Sawatzky <alice.sawatzky@farmersedge.ca>
@jdolitsky
Copy link
Contributor

closing as outdated

@jdolitsky jdolitsky closed this Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants