search

axiomhq / cli

The power of Axiom on the command line.
https://axiom.co
MIT License
42 stars 11 forks source link

fix(deps): update to most recent 'axiom-go' #135

Closed lukasmalkmus closed 1 year ago

lukasmalkmus commented 2 years ago

This updates to latest axiom-go and drops non-public routes.

codecov[bot] commented 2 years ago

Codecov Report

Base: 14.10% // Head: 15.29% // Increases project coverage by +1.19% :tada:

Coverage data is based on head (728e5a8) compared to base (dd5c88e). Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #135 +/- ## ========================================== + Coverage 14.10% 15.29% +1.19% ========================================== Files 10 10 Lines 539 791 +252 ========================================== + Hits 76 121 +45 - Misses 445 652 +207 Partials 18 18 ``` | [Impacted Files](https://codecov.io/gh/axiomhq/cli/pull/135?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq) | Coverage Δ | | |---|---|---| | [internal/cmd/root/root.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-aW50ZXJuYWwvY21kL3Jvb3Qvcm9vdC5nbw==) | `0.00% <ø> (ø)` | | | [pkg/terminal/io.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-cGtnL3Rlcm1pbmFsL2lvLmdv) | `0.00% <0.00%> (ø)` | | | [internal/cmd/root/help.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-aW50ZXJuYWwvY21kL3Jvb3QvaGVscC5nbw==) | `0.00% <0.00%> (ø)` | | | [pkg/terminal/epipe\_other.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-cGtnL3Rlcm1pbmFsL2VwaXBlX290aGVyLmdv) | `0.00% <0.00%> (ø)` | | | [internal/cmd/root/help\_topic.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-aW50ZXJuYWwvY21kL3Jvb3QvaGVscF90b3BpYy5nbw==) | `100.00% <0.00%> (ø)` | | | [internal/config/config.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-aW50ZXJuYWwvY29uZmlnL2NvbmZpZy5nbw==) | `6.14% <0.00%> (+0.87%)` | :arrow_up: | | [pkg/terminal/color.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-cGtnL3Rlcm1pbmFsL2NvbG9yLmdv) | `5.26% <0.00%> (+1.00%)` | :arrow_up: | | [cmd/axiom/main.go](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq#diff-Y21kL2F4aW9tL21haW4uZ28=) | `19.10% <0.00%> (+1.06%)` | :arrow_up: | | ... and [2 more](https://codecov.io/gh/axiomhq/cli/pull/135/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=axiomhq)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

SollyzDev commented 2 years ago

looks legit but it seems we also need to update the integration test.

lukasmalkmus commented 2 years ago

@schehata Fixed the tests but we need to release a new version of axiom-go first before we update and merge this PR. Keeping in draft for now.

lukasmalkmus commented 2 years ago

This needs https://github.com/axiomhq/axiom-go/pull/136#discussion_r959398450.

lukasmalkmus commented 1 year ago

This is now ready and uses axiom-go@v0.12.0.