redhat-et / copilot-ops

copilot-ops is a CLI tool that boosts up any DevOps repo to a ninja level of Artificially Intelligent Ops Repo
Apache License 2.0
34 stars 13 forks source link

Use the go-gpt3 package #97

Closed RobotSail closed 2 years ago

RobotSail commented 2 years ago

This PR aims to close #87 by replacing our openai package with the upstream go-gpt3 package. This way, other people besides us can contribute to and benefit from the go-gpt3 repository, and our pull-requests there will also benefit other users. We also eliminate some amount of technical debt with this transition, since there is now less to maintain within our own repository.

This change also adds a minor refactor which moves config-related stuff into a separate package underneath pkg/cmd.

codecov-commenter commented 2 years ago

Codecov Report

Merging #97 (013791c) into main (2dd6622) will decrease coverage by 6.86%. The diff coverage is 65.78%.

@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   73.74%   66.88%   -6.87%     
==========================================
  Files           7        7              
  Lines         598      625      +27     
==========================================
- Hits          441      418      -23     
- Misses        135      188      +53     
+ Partials       22       19       -3     
Impacted Files Coverage Δ
pkg/cmd/config/config.go 0.00% <ø> (ø)
pkg/filemap/filemap.go 53.88% <0.00%> (-5.80%) :arrow_down:
pkg/cmd/utils.go 78.76% <72.72%> (-0.61%) :arrow_down:
pkg/cmd/edit.go 83.01% <100.00%> (+3.47%) :arrow_up:
pkg/cmd/generate.go 86.25% <100.00%> (+1.01%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2dd6622...013791c. Read the comment docs.

RobotSail commented 2 years ago

We'll probably want PR #73 to merge before this one, and we can pull out all of the refactors that happened with moving config into a sub-package.