Skip to content

Make shell command graders platform-aware - #405

Closed
paulcam206 wants to merge 1 commit into
pinchbench:mainfrom
paulcam206:paulcam206-portable-task-grader
Closed

Make shell command graders platform-aware#405
paulcam206 wants to merge 1 commit into
pinchbench:mainfrom
paulcam206:paulcam206-portable-task-grader

Conversation

@paulcam206

Copy link
Copy Markdown

Summary

  • select PowerShell on Windows and bash on POSIX without relying on shell=True
  • keep git fixture setup platform-independent and surface missing shell infrastructure
  • normalize relative, absolute, and backslash-delimited command output paths
  • execute both embedded graders in behavior tests, including valid and invalid candidates

Testing

  • python -m pytest tests/test_task_shell_graders.py -q (10 passed, 6 subtests)
  • python -m pytest -q (42 passed, 6 subtests)
  • ruff check tests/test_task_shell_graders.py
  • python scripts/lint_argparse_help.py
  • python scripts/lint_manifest.py

Repo-wide ruff check . still reports pre-existing violations outside the changed files.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@paulcam206 paulcam206 closed this Aug 31, 2026
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