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

Generate formulae.brew.sh data process is broken #49840

Closed
chenrui333 opened this issue Feb 6, 2020 · 1 comment · Fixed by Homebrew/formulae.brew.sh#244
Closed

Generate formulae.brew.sh data process is broken #49840

chenrui333 opened this issue Feb 6, 2020 · 1 comment · Fixed by Homebrew/formulae.brew.sh#244
Assignees

Comments

@chenrui333
Copy link
Member

@chenrui333 chenrui333 commented Feb 6, 2020

Broken since the latest node-build PR merge, https://github.com/Homebrew/homebrew-core/runs/428749667

brew formula-analytics  --days-ago=365 --json --os-version > _data/analytics/os-version/365d.json
brew formula-analytics  --days-ago=30 --json --all-core-formulae-json --build-error > _data/analytics/build-error/homebrew-core/30d.json
Error: Options --json and --all-core-formulae-json are mutually exclusive.
rake aborted!
@issyl0

This comment has been minimized.

Copy link
Member

@issyl0 issyl0 commented Feb 6, 2020

The brew formula-analytics command was switched to use CLI::Parser in Homebrew/homebrew-formula-analytics#129.

@issyl0 issyl0 self-assigned this Feb 6, 2020
issyl0 added a commit to Homebrew/formulae.brew.sh that referenced this issue Feb 6, 2020
- In Homebrew/homebrew-formula-analytics#129, `brew formula-analytics` switched to use `CLI::Parser`. While doing so, `--json` and `--all-core-formulae-json` were set to be [conflicting](https://github.com/Homebrew/homebrew-formula-analytics/blob/master/cmd/formula-analytics.rb#L34).
- This caused an issue where the GitHub Action to generate the data for these formulae.brew.sh pages [couldn't run](Homebrew/homebrew-core#49840) because it was using both of the arguments.
- Instead, make this only use `--all-core-formulae-json` as it's the more specific one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.