Skip to content

CI: Fix nightly build#114

Merged
amotl merged 1 commit intomainfrom
fix-ci-202602
Feb 8, 2026
Merged

CI: Fix nightly build#114
amotl merged 1 commit intomainfrom
fix-ci-202602

Conversation

@amotl
Copy link
Member

@amotl amotl commented Feb 8, 2026

Problem

ValueError: Failed to load fragment 'https://www.example.org/': [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1081)

References

@amotl
CI: Fix nightly build
777673e 
ValueError: Failed to load fragment 'https://www.example.org/':
[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to
get local issuer certificate (_ssl.c:1081)
@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026 •
edited
Loading

Walkthrough

A single test function was updated to use a different remote URL endpoint (https://httpbin.org/uuid) and its corresponding assertion was adjusted to validate the expected response content.

Changes

Cohort / File(s) Summary
Test Updates
tests/test_prompt.py		 Modified test_fragment_remote_success to load from httpbin.org/uuid instead of example.org and updated assertion to expect "uuid" in response instead of "Example Domain".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰 A test hops to a newer spring,
Where UUID dances in the JSON ring,
From example's old domain it fled,
To httpbin's fresh endpoint instead.
Simple paths make testing bright! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a CI/nightly build issue by updating a test to use a different endpoint that doesn't cause SSL certificate verification failures.
Description check ✅ Passed The description is directly related to the changeset, explaining the SSL certificate error that prompted the test endpoint change from example.org to httpbin.org.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-ci-202602

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
tests/test_prompt.py (1)

62-67: Reasonable fix for the SSL issue, but consider the long-term reliability of httpbin.org.

The switch from example.org to httpbin.org/uuid fixes the immediate CI failure. However, httpbin.org is a third-party service that has experienced availability issues in the past, which could cause flaky CI failures again.

Consider whether this integration test could use a locally spawned HTTP server (e.g., pytest-httpserver) instead, to avoid any external dependency. That said, if the goal is specifically to validate real-world TLS + redirect behavior, an external endpoint is reasonable.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amotl amotl mentioned this pull request Feb 8, 2026
Merged
@amotl amotl marked this pull request as ready for review February 8, 2026 16:12
@amotl amotl merged commit d1910fc into main Feb 8, 2026
11 checks passed
@amotl amotl deleted the fix-ci-202602 branch February 8, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@amotl