Skip to content

Wire pup CLI generation into the client generation pipeline - #745

Open
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
datadog-api-spec/generated/6183
Open

Wire pup CLI generation into the client generation pipeline#745
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
datadog-api-spec/generated/6183

Conversation

@dd-octo-sts

@dd-octo-sts dd-octo-sts Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@dd-octo-sts
dd-octo-sts Bot requested a review from a team as a code owner August 19, 2026 09:49
@dd-octo-sts dd-octo-sts Bot added the changelog/no-changelog Changes don't appear in changelog label Aug 19, 2026
@datadog-datadog-prod-us1-2

This comment has been minimized.

Sign in to view
platinummonkey
platinummonkey requested changes Aug 19, 2026
View reviewed changes

@platinummonkey platinummonkey left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this will break some things, looks like we need to work on how it handles these scopes

Comment thread src/auth/types.rs
@dd-octo-sts
dd-octo-sts Bot force-pushed the datadog-api-spec/generated/6183 branch 5 times, most recently from 1679471 to 2ffd02e Compare August 24, 2026 11:37
@dd-octo-sts
dd-octo-sts Bot force-pushed the datadog-api-spec/generated/6183 branch from 2ffd02e to 2263481 Compare August 24, 2026 11:49
@MNsharahily

MNsharahily commented Aug 30, 2026

Copy link
Copy Markdown

Pipelines

✨ Unblock PR with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

CI | Check, Test & Coverage — 🔧 Needs a code fix, caused by this PR
View in Datadog · View in GitHub Actions

Compilation error in src/generated.rs:14:18: unused variable cfg. Add _cfg to silence this warning or use #[allow(unused_variables)].

📋 Copy prompt for your agent

CI on my pull request is failing. Help me find and fix the root cause of each failing job below — they were flagged as caused by changes in this PR, so focus on the diff. For each job, explain the failure and propose a fix.

Branch: datadog-api-spec/generated/6183

CI | Check, Test & Coverage
Commit: bf354204beef136f994996f3f80b6b0e2e5f0366
Error (code / build):
Compilation error in src/generated.rs:14:18: unused variable `cfg`. Add `_cfg` to silence this warning or use `#[allow(unused_variables)]`.
CI job: https://github.com/DataDog/pup/actions/runs/32462682278/job/96712678107

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: bf35420 | Docs | View more details | Give us feedback!

@MNsharahily MNsharahily mentioned this pull request Aug 30, 2026
Closed
@MNsharahily

MNsharahily commented Aug 30, 2026

Copy link
Copy Markdown

Pipelines
✨ Unblock PR with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

CI | Check, Test & Coverage — 🔧 Needs a code fix, caused by this PR
View in Datadog · View in GitHub Actions

Compilation error in src/generated.rs:14:18: unused variable cfg. Add _cfg to silence this warning or use #[allow(unused_variables)].

📋 Copy prompt for your agent

CI on my pull request is failing. Help me find and fix the root cause of each failing job below — they were flagged as caused by changes in this PR, so focus on the diff. For each job, explain the failure and propose a fix.

Branch: datadog-api-spec/generated/6183

CI | Check, Test & Coverage
Commit: bf354204beef136f994996f3f80b6b0e2e5f0366
Error (code / build):
Compilation error in src/generated.rs:14:18: unused variable `cfg`. Add `_cfg` to silence this warning or use `#[allow(unused_variables)]`.
CI job: https://github.com/DataDog/pup/actions/runs/32462682278/job/96712678107

Useful? React with 👍 / 👎
This comment will be updated automatically if new data arrives.
🔗 Commit SHA: bf35420 | Docs | View more details | Give us feedback!

@MNsharahily

MNsharahily commented Aug 30, 2026

Copy link
Copy Markdown

See DataDog/datadog-api-spec#6183 Test branch datadog-api-spec/test/ulysse.mavrocordatos/AAWF-1384/wire-pup-generation

@MNsharahily

MNsharahily commented Aug 30, 2026

Copy link
Copy Markdown

See DataDog/datadog-api-spec#6183 Test branch datadog-api-spec/test/ulysse.mavrocordatos/AAWF-1384/wire-pup-generation

@MNsharahily MNsharahily mentioned this pull request Aug 31, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@platinummonkey platinummonkey platinummonkey requested changes
@jack-edmonds-dd jack-edmonds-dd jack-edmonds-dd left review comments

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

changelog/no-changelog Changes don't appear in changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MNsharahily @platinummonkey @jack-edmonds-dd