pizza-cli icon indicating copy to clipboard operation
pizza-cli copied to clipboard

Bug: `--period` flag is broken

Open jpmcb opened this issue 7 months ago • 2 comments

Describe the bug

The period flag seems to have broken at some point:

❯ pizza insights contributors https://github.com/open-sauced/app -p 45
Error: error while calling 'ContributorsServiceAPI.FindAllChurnPullRequestContributors' with repository 501028599': 400 Bad Request
error while calling 'ContributorsServiceAPI.NewPullRequestContributors' with repository 501028599': 400 Bad Request
error while calling 'ContributorsServiceAPI.FindAllRecentPullRequestContributors' with repository 501028599': 400 Bad Request
error while calling 'ContributorsServiceAPI.FindAllRepeatPullRequestContributors' with repository 501028599: 400 Bad Request

Usage:
  pizza insights contributors url... [flags]

Flags:
  -f, --file string    Path to yaml file containing an array of git repository urls
  -h, --help           help for contributors
  -p, --period int32   Number of days, used for query filtering (default 30)

Global 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")
  -o, --output string       The formatting for command output. One of: (table, yaml, csv, json) (default "table")

Steps to reproduce

  1. call the insights command with a repo and the -p / --period flag defined
  2. Notice the 400, bad request responses

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

Contributing Docs

  • [X] I agree to follow this project's Contribution Docs

jpmcb avatar Nov 06 '23 21:11 jpmcb