Skip to content

test: cover Xquik queries and fix CLI docs#347

Open
kriptoburak wants to merge 1 commit into
fabien0102:mainfrom
kriptoburak:codex/xquik-fetcher-query-test
Open

test: cover Xquik queries and fix CLI docs#347
kriptoburak wants to merge 1 commit into
fabien0102:mainfrom
kriptoburak:codex/xquik-fetcher-query-test

Conversation

@kriptoburak

@kriptoburak kriptoburak commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • add generator coverage for a real Xquik tweet-search operation
  • verify required and optional query parameter types plus fetcher URL generation
  • fix the getting-started command to invoke the published @openapi-codegen/cli package
  • correct 2 nearby README typos

Independent repository fix

The README previously instructed users to run npx openapi-codegen, but this repository publishes the CLI as @openapi-codegen/cli. The corrected command now matches the initialization command and package name.

Validation

  • pnpm test -- --run (242 tests)
  • pnpm check
  • pnpm build

The branch is rebuilt on current main as 1 signed commit.

@kriptoburak
kriptoburak force-pushed the codex/xquik-fetcher-query-test branch from 1421a22 to c694598 Compare July 20, 2026 00:49
@kriptoburak kriptoburak changed the title test: cover Xquik search fetcher generation test: cover Xquik queries and fix CLI docs Jul 20, 2026
@kriptoburak

Copy link
Copy Markdown
Author

I refreshed this PR on current main as one signed commit. In addition to the Xquik generator fixture, it now fixes the README command that referenced the wrong unscoped CLI package and corrects two nearby typos. All 242 tests, repository checks, and builds pass.

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.

1 participant