Skip to content

test: pending status (aws-deadline/.github#114) - #35

Open
crowecawcaw wants to merge 3 commits into
mainlinefrom
test/pending-status
Open

crowecawcaw wants to merge 3 commits into
mainlinefrom
test/pending-status

Conversation

@crowecawcaw

Copy link
Copy Markdown
Owner

E2E test for aws-deadline/.github#114. Planted bugs: off-by-one mean, wrong index.

Comment thread src/deadline/_scratch/pending_status_probe.py Outdated
Comment thread src/deadline/_scratch/pending_status_probe.py Outdated
@@ -0,0 +1,10 @@
def average(values):

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nit: This new _scratch module lives under the shipped src/deadline package, so it would be packaged and released. Nothing references it and it has no tests. Remove it from the PR, or move it outside src/ if it is only for local experiments.

@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Claude review · advisory

Reviewed 8d3a087 (changes since 0d9f1d9): 0 new, 2 resolved.

Open: 0 blocking · 0 should-fix · 1 nit. ✅ Nothing blocking.

Fix or reply to each thread; the next revision's review re-checks open threads and resolves those it agrees are handled. Resolving a thread yourself also closes it. Later revisions review only what changed.

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant