Skip to content

Remove local generator code - #4327

Open
nogates wants to merge 1 commit into
masterfrom
nogates/match-go-pr-4584
Open

nogates wants to merge 1 commit into
masterfrom
nogates/match-go-pr-4584

Conversation

@nogates

@nogates nogates commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Mirrors DataDog/datadog-api-client-go#4584 for the Java client:

  • removes the in-repository Python generator package, templates, tests, and generation script while retaining the OpenAPI snapshots under .generator/schemas
  • removes generator-only pre-commit hooks
  • documents the internal client-generation and pull-request regeneration workflow

Testing

  • DD_USE_GENERATED_TESTS=true mvn test (1,966 tests, 37 skipped)
  • pre-commit validate-config
  • pre-commit run docs --hook-stage manual --files README.md --show-diff-on-failure
  • pre-commit run check-case-conflict --all-files --show-diff-on-failure
  • git diff --check

Additional Notes

The plain cassette replay suite currently contains pre-existing response expectation drift and missing freeze files. The generated-test mode used by CI passes.

Review checklist

  • This PR includes all newly recorded cassettes for any modified tests. (Not applicable; no tests or cassettes changed.)

  • This PR does not rely on API client schema changes.

    • The CI should be fully passing.
  • Or, this PR relies on API schema changes and this is a Draft PR that includes tests.

    • Note: CI should not be run on this Draft PR, as it is expected to fail without the corresponding schema changes.

@nogates
nogates requested review from a team as code owners September 14, 2026 14:09
@github-actions github-actions Bot added the documentation Documentation related changes label Sep 14, 2026
@nogates nogates added the changelog/no-changelog Changes don't appear in changelog label Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog Changes don't appear in changelog documentation Documentation related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant