Skip to content

feat(autograder): add live-API, pandas-transform, and architecture-overview checks#11

Merged
lassebenni merged 3 commits into
mainfrom
feat/autograder-live-api-transform-checks
Jun 18, 2026
Merged

feat(autograder): add live-API, pandas-transform, and architecture-overview checks#11
lassebenni merged 3 commits into
mainfrom
feat/autograder-live-api-transform-checks

Conversation

@lassebenni

Copy link
Copy Markdown
Collaborator

Summary

Three new automated checks, each corresponding to a minimum requirement that was previously unverified:

  • HTTP library import (requests / httpx / aiohttp) — critical: a CSV-only pipeline with no API call would have passed before
  • pandas real transform operations (.groupby, .agg, pd.to_datetime, .dt., .assign, .merge, .fillna, .dropna) — critical: pd.DataFrame(records) straight to storage would have passed before
  • Architecture overview in README (mermaid block, ASCII arrows → ↓, or ## Architecture / ## Data flow heading): no check existed for this documentation requirement

Also includes minor cleanup: type annotations, from __future__ import annotations, consistent helper style, clearer check labels.

Test plan

  • Open a student repo that uses requests + real pandas transforms + has a README architecture section — confirm all new checks pass
  • Open a stub repo with no API import — confirm HTTP library check fails as critical
  • Open a repo with only pd.DataFrame(records) — confirm transform check fails as critical

@lassebenni lassebenni merged commit 57a9062 into main Jun 18, 2026
4 checks passed
@lassebenni lassebenni deleted the feat/autograder-live-api-transform-checks branch June 18, 2026 07:47
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