Skip to content

Link developer Python tutorial from SDK README - #162

Open
karlwaldman wants to merge 2 commits into
mainfrom
codex/python-readme-tutorial-20261031
Open

karlwaldman wants to merge 2 commits into
mainfrom
codex/python-readme-tutorial-20261031

Conversation

@karlwaldman

@karlwaldman karlwaldman commented Oct 31, 2026 •

Copy link
Copy Markdown
Member

Advances OilpriceAPI/website-clean#2702. The SDK README now links the already deployed Python tutorial beside its reproducible examples. The tutorial returned HTTP 200 and visibly covers authentication, source timestamps, error recovery, and a daily refresh pattern. git diff --check passed. Documentation only; no package behavior changes.

@coderabbitai

coderabbitai Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 80b0439f-070a-4a0d-9678-ee004fb86b51
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@karlwaldman karlwaldman added the risk:elevated Elevated: auto-merges after an adversarial review APPROVE label Oct 31, 2026
@karlwaldman

Copy link
Copy Markdown
Member Author

APPROVE — exact-head adversarial review found no remaining customer-facing claim or integration defect.

Reviewed the complete README-only diff at 022f18cdde751e06226f3f674ca41f9633ea75f6 in a detached worktree, the PR discussion (no earlier blocking review), and the linked website-clean#2702 scope. The first draft also linked the not-yet-deployed getting-started page and claimed its corrected response shape was live; that claim was removed in the current head. The sole remaining target is the deployed Python tutorial: a cache-busted production GET returned 200, and its rendered text covers API-key authentication, source timestamps, error recovery, and a daily refresh pattern. git diff --check passed. This change touches only README.md; it changes no SDK runtime, auth, billing, API keys, or email behavior.

Required CI on this exact head: Live API tests plus Python 3.8, 3.9, 3.10, 3.11, and 3.12 test jobs all passed (6/6 jobs). No README-specific unit test exists or is needed for this link; the live target and content were checked directly.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:elevated Elevated: auto-merges after an adversarial review APPROVE

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant