Skip to content

feat: implement jobScript pre-built script support (mirrors SLURM plugin)#23

Draft
Copilot wants to merge 3 commits into
mainfrom
copilot/borrow-jobscripturl-behavior
Draft

feat: implement jobScript pre-built script support (mirrors SLURM plugin)#23
Copilot wants to merge 3 commits into
mainfrom
copilot/borrow-jobscripturl-behavior

Conversation

Copilot AI commented Apr 14, 2026

Copy link
Copy Markdown
Contributor
  • Implement jobScript handling in SubmitHandler (handles.py)
  • Add 9 unit tests for the new jobScript path in tests/test_handles.py
  • Fix flake8 E501 (line too long) lint issues introduced in tests/test_handles.py
  • All 152 tests pass

Copilot AI changed the title [WIP] Implement borrow jobscriptURL behavior from slurm plugin feat: implement jobScript pre-built script support (mirrors SLURM plugin) Apr 14, 2026
Copilot AI requested a review from dciangot April 14, 2026 11:29
@dciangot

Copy link
Copy Markdown
Member

@copilot fix lint issues without breaking e2e tests

Copilot AI commented Apr 14, 2026

Copy link
Copy Markdown
Contributor Author

@copilot fix lint issues without breaking e2e tests

Fixed in 294dee0 — resolved the 4 E501 (line too long) issues introduced in tests/test_handles.py. The 3 pre-existing E501 violations (lines 849, 956, 1070) were already present before my changes and are unchanged.

@dciangot dciangot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

Borrow jobscriptURL behavior from slurm plugin

2 participants