fix: allow configurable request timeout#25
Conversation
|
Quick status for review: this PR is mergeable from GitHub's side and intentionally scoped to the linked apidrift-cli issue. Validation already included in the PR body:
I also added the suggested GSSoC scoring labels in the PR body so maintainers can apply them directly if the change is accepted. |
|
Hi @tanmayjoddar, this PR is still open, mergeable, and clean against the current base branch. I rechecked the repo test baseline locally ( |
|
Fresh validation pass on the current branch:\n- node --experimental-vm-modules node_modules/jest/bin/jest.js --runInBand -> 3 suites / 13 tests passed\n- node --check src/core/fetcher.js\n- git diff --check\n\nThis remains clean and mergeable from my side; no GitHub checks are configured on the branch, so local validation is the current proof. |
Summary
APIDRIFT_TIMEOUT_MSfor slower APIs while keeping the default at 10000 msValidation
npm cinpm test -- --runInBand(13 passed)node --check src/core/fetcher.jsnode --check tests/core/fetcher.test.jsgit diff --checkSuggested labels:
gssoc,dx,good first issue.Suggested GSSoC labels
If accepted for scoring, this should fit
gssoc,level:beginner,dx,type:feature,type:testing,quality:cleanbased on the linked issue labels and the focused validation in this PR.