Skip to content

Fix Homebrew install command for pup#36

Merged
platinummonkey merged 2 commits into
mainfrom
fix/brew-install-readme
Feb 11, 2026
Merged

Fix Homebrew install command for pup#36
platinummonkey merged 2 commits into
mainfrom
fix/brew-install-readme

Conversation

@angristan
Copy link
Copy Markdown
Contributor

@angristan angristan commented Feb 11, 2026

pup is already a brew formula, so there is a conflict and brew install pup installs the upstream pup by default:

~ brew tap datadog/pack
➜  ~ brew install pup
Warning: Treating pup as a formula. For the cask, use datadog/tap/pup or specify the `--cask` flag. To silence this message, use the `--formula` flag.
Warning: pup 0.4.0 is already installed and up-to-date.
To reinstall 0.4.0, run:
  brew reinstall pup
➜  ~ brew info pup
Warning: Treating pup as a formula. For the cask, use datadog/tap/pup or specify the `--cask` flag. To silence this message, use the `--formula` flag.
==> pup ✔: stable 0.4.0 (bottled), HEAD
Parse HTML at the command-line
https://github.com/EricChiang/pup
Deprecated because it is not maintained upstream! It was disabled on 2025-02-24.
Installed
/opt/homebrew/Cellar/pup/0.4.0 (6 files, 4.3MB) *
  Poured from bottle using the formulae.brew.sh API on 2026-02-11 at 14:07:44
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/p/pup.rb
License: MIT
==> Dependencies
Build: go ✔, gox ✘
==> Options
--HEAD
        Install HEAD version
==> Downloading https://formulae.brew.sh/api/formula/pup.json
==> Analytics
install: 134 (30 days), 367 (90 days), 1,739 (365 days)
install-on-request: 134 (30 days), 367 (90 days), 1,739 (365 days)
build-error: 0 (30 days)

Using brew install datadog/pack/pup works:

~ brew uninstall pup
Warning: Treating pup as a formula. For the cask, use datadog/tap/pup or specify the `--cask` flag. To silence this message, use the `--formula` flag.
Warning: Formula postgresql was renamed to postgresql@14.
Warning: Formula postgresql was renamed to postgresql@14.
Uninstalling /opt/homebrew/Cellar/pup/0.4.0... (6 files, 4.3MB)
Warning: Formula postgresql was renamed to postgresql@14.
➜  ~ brew install datadog/pack/pup
==> Fetching downloads for: pup
✔︎ Formula pup (0.7.0)                                                                                                                                   Verified    184.5KB/184.5KB
==> Installing pup from datadog/pack
==> go build -ldflags=-s -w
🍺  /opt/homebrew/Cellar/pup/0.7.0: 9 files, 21.6MB, built in 2 seconds
==> Running `brew cleanup pup`...
Disable this behaviour by setting `HOMEBREW_NO_INSTALL_CLEANUP=1`.
Hide these hints with `HOMEBREW_NO_ENV_HINTS=1` (see `man brew`).
➜  ~ pup --help
Pup is a Go-based command-line wrapper that provides easy interaction
with Datadog APIs. It supports both API key and OAuth2 authentication.

@angristan angristan requested a review from a team as a code owner February 11, 2026 13:15
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c44a48e57d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 11, 2026

📊 Test Coverage Report

Overall Coverage: 80.4% Coverage

Threshold: 80% ✅

Coverage by Package
## Coverage by Package

- github.com/DataDog/pup/pkg/auth/callback/server.go:40: 81.2%
- github.com/DataDog/pup/pkg/auth/dcr/client.go:28: 100.0%
- github.com/DataDog/pup/pkg/auth/dcr/types.go:24: 100.0%
- github.com/DataDog/pup/pkg/auth/oauth/client.go:22: 100.0%
- github.com/DataDog/pup/pkg/auth/oauth/pkce.go:24: 85.7%
- github.com/DataDog/pup/pkg/auth/storage/factory.go:53: 94.7%
- github.com/DataDog/pup/pkg/auth/storage/keychain.go:44: 42.9%
- github.com/DataDog/pup/pkg/auth/storage/storage.go:58: 71.4%
- github.com/DataDog/pup/pkg/auth/types/types.go:23: 100.0%
- github.com/DataDog/pup/pkg/client/client.go:32: 94.4%
- github.com/DataDog/pup/pkg/config/config.go:22: 100.0%
- github.com/DataDog/pup/pkg/formatter/formatter.go:31: 100.0%
- github.com/DataDog/pup/pkg/useragent/useragent.go:32: 100.0%
- github.com/DataDog/pup/pkg/util/time.go:20: 95.8%

## Summary

total:								(statements)		80.4%

📈 Coverage Status: ✅ PASSED - Coverage meets minimum threshold

Updated for commit e3029a4

@platinummonkey platinummonkey merged commit e596ec5 into main Feb 11, 2026
4 checks passed
@platinummonkey platinummonkey deleted the fix/brew-install-readme branch February 11, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants