search

octokit / source-generator

6 stars 3 forks source link

Use kiota list command to install dependencies #33

Closed kfcampbell closed 8 months ago

kfcampbell commented 8 months ago

Fixes #17.

Instead of hard-coding dependencies and their versions, this PR uses the kiota list command to get them.

:warning: Note: :warning: in a very "production" application, this should use structs rather than interface{}s and it should be unit-tested, which it's not currently. @nickfloyd do you think we're at an appropriate stage where I should sharpen it like so? Is this just me being lazy? Are we in enough flux that I shouldn't do so yet and instead file an issue to tackle it after the alpha release?

nickfloyd commented 8 months ago

@nickfloyd do you think we're at an appropriate stage where I should sharpen it like so? Is this just me being lazy? Are we in enough flux that I shouldn't do so yet and instead file an issue to tackle it after the alpha release?

I still think we are grinding away at the concept here - though, thanks to Kiota and all of our previous work we have a solid direction, we are still ideating on how we want things to look and feel... I say we am to make "it" then come back around and make it better - I'm using this same approach with auth and headers - get it working then try it out, then get it working better.