Harden the Python data baseline - #1
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit:ced33cf3a2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| run: python -m compileall -q main.py examples tests | ||
| - name: Run Ruff lint checks | ||
| run: python -m ruff check main.py examples tests |
There was a problem hiding this comment.
Format the existing examples before enforcing Ruff
In the added Python quality workflow, this command scans all four pre-existing top-level example scripts in both matrix jobs. Running the exact command with Ruff 0.15.12 exits with status 1 because examples/01_csv_pandas_basics.py, 02_json_basics.py, 03_api_request_basics.py, and 04_ollama_local_api_basics.py each trigger I001 for an extra blank line after their imports, so every Ubuntu and Windows run stops before pytest. Format those files or adjust the lint configuration before making this check mandatory.
Useful? React with 👍 / 👎.
Uh oh!
There was an error while loading. Please reload this page.
Summary
This PR turns
python-data-basicsfrom a locally documented learning setup into a small, testable and cross-platform Python Data/BI baseline.Dependency and project model
pyproject.tomlas the dependency and tooling source of truthrequirements-core.txtBaseline and examples
main.pywith a deterministic environment and pandas baseline checkexamples/optional/with an explicit limitation and optional dependency groupNotebook hygiene
Quality assurance
Documentation
Verified GitHub-hosted execution
The final workflow run completed successfully on both matrix targets:
Both jobs successfully completed:
Safety and scope