Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Trim down CLI - remove unused, defunct commands #93

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

jpmcb
Copy link
Member

@jpmcb jpmcb commented Aug 26, 2024

Description

Removes unused, defunct commands:

  • Deletes bake code entirely
  • Deletes repo-query code entirely
  • Removes the show command from the root command (but the infrastructure of the code is still there)

Related Tickets & Documents

Closes: #89
Closes: #90
Closes: #92

Mobile & Desktop Screenshots/Recordings

Commands are gone:

❯ just build
Building for local arch
❯ ./build/pizza -h

A command line utility for insights, metrics, and all things OpenSauced

Usage:
  pizza <command> <subcommand> [flags]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  help        Help about any command
  insights    Gather insights about git contributors, repositories, users and pull requests
  login       Log into the CLI application via GitHub
  version     Displays the build version of the CLI

Flags:
      --beta                Shorthand for using the beta OpenSauced API endpoint ("https://beta.api.opensauced.pizza"). Supersedes the
                            '--endpoint' flag
      --disable-telemetry   Disable sending telemetry data to OpenSauced
  -e, --endpoint string     The API endpoint to send requests to (default "https://api.opensauced.pizza")
  -h, --help                help for pizza
  -o, --output string       The formatting for command output. One of: (table, yaml, csv, json) (default "table")

Use "pizza [command] --help" for more information about a command.

Steps to QA

  1. Build via just build
  2. Verify commands are gone via ./build/pizza

Tier (staff will fill in)

  • Tier 1
  • Tier 2
  • Tier 3
  • Tier 4

Signed-off-by: John McBride <john@opensauced.pizza>
@jpmcb jpmcb requested a review from a team August 26, 2024 19:50
Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹

@jpmcb jpmcb merged commit 7ddd4b9 into beta Aug 26, 2024
21 checks passed
@jpmcb jpmcb deleted the remove-unused-cmds branch August 26, 2024 20:58
open-sauced bot pushed a commit that referenced this pull request Aug 26, 2024
## [1.2.0-beta.2](v1.2.0-beta.1...v1.2.0-beta.2) (2024-08-26)

### 🍕 Features

* Trim down CLI - remove unused, defunct commands ([#93](#93)) ([7ddd4b9](7ddd4b9))
open-sauced bot pushed a commit that referenced this pull request Aug 30, 2024
## [1.2.0](v1.1.1...v1.2.0) (2024-08-30)

### 📝 Documentation

* updated comment for LoadConfig ([ab5206b](ab5206b))

### ✅ Tests

* added setup and teardown ([aba6310](aba6310))
* added tests for LoadConfig ([2a5f85d](2a5f85d))
* made tests parallel ([2b3d8ca](2b3d8ca))

### 🍕 Features

* added .sauced.yaml and updated CODEOWNERS file ([#109](#109)) ([dfc56cb](dfc56cb))
* added built at to version command ([#94](#94)) ([9960fc0](9960fc0))
* Codeowners generation ([#95](#95)) ([79cf8a2](79cf8a2))
* now generate codeowners checks in user root but also repository root folder for .sauced.yaml ([a0298b1](a0298b1))
* pizza login success page style refresh ([#112](#112)) ([9357dac](9357dac))
* Refactors Auth code into api/ directory ([#105](#105)) ([d851499](d851499))
* Skip semantic-release docker build in favor of buildx building ([d782974](d782974))
* Trim down CLI - remove unused, defunct commands ([#93](#93)) ([7ddd4b9](7ddd4b9))
* Upgrade Go module to use Go 1.22 ([#96](#96)) ([690b6e9](690b6e9))
* Use justfile vs. makefile ([#84](#84)) ([8f38eaf](8f38eaf))

### 🐛 Bug Fixes

* escape non-standard characters in the filename path ([#106](#106)) ([418951f](418951f))
* mark endpoint and beta flags as hidden ([#113](#113)) ([6aa250f](6aa250f))
* move output flag to insights command ([#115](#115)) ([be7f8cd](be7f8cd))
* update supabase keys for auth ([#80](#80)) ([247c431](247c431))
* use repository fullname to fetch contributors ([#77](#77)) ([5326875](5326875))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants