Skip to content

Conversation

Copy link

Copilot AI commented Feb 2, 2026

Review comment requested opening a GitHub issue to track incorrect pytest.approx usage in test_exponential_backoff.py.

Response provided

Replied to comment explaining lack of issue-creation permissions and provided complete issue details:

  • Root cause: Commit bfc692c converted self.assertAlmostEqual(x, y) to pytest.approx(x, y) instead of assert x == pytest.approx(y)
  • Affected lines: 11, 14, 16, 18, 20, 22, 37, 45, 50
  • Example fix: pytest.approx(mock_time(), 0.5)assert mock_time() == pytest.approx(0.5)

Awaiting clarification on whether to fix assertions in this PR or have user create issue manually.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Feb 2, 2026
@jacalata jacalata closed this Feb 2, 2026
Copilot AI changed the title [WIP] Address feedback from review on release v0.39 PR Respond to review feedback: cannot create GitHub issues programmatically Feb 2, 2026
@jacalata jacalata deleted the copilot/sub-pr-1737-again branch February 2, 2026 06:47
Copilot AI requested a review from jacalata February 2, 2026 06:47
Copilot stopped work on behalf of jacalata due to an error February 2, 2026 06: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.

2 participants