autometrics-dev / am

Autometrics Companion CLI app
Apache License 2.0
16 stars 2 forks source link

Merge `am_list` back into `am` #114

Closed gagbo closed 1 year ago

gagbo commented 1 year ago

Now that am_list (both the binary and the library) is stable, it's probably ready to be included in am entirely. There are 2 main things to take care of when integrating this:

In any case, it's probably going to have an impact on the release cycle for am, as adding/fixing support for specific languages must trigger a new version release as soon as possible for the dependent projects.

I think all the rest can be handled without too much issue, I can do that integration once those points are solved.

mellowagain commented 1 year ago

i think there is nothing stopping us from publishing the am library on crates.io, no? it's already setup this way as we have two crates in this repo actually, am (the cli tool) and autometrics-am (the library).

imo adjusting the release schedule also doesnt seem like a problem

for the subcommand, maybe something like am functions?

gagbo commented 1 year ago

I also need to remember to change the download location in the github action. I'll try to track the issue there as well

gagbo commented 1 year ago

Seems like it's going to be the plan: