Skip to content

Conversation

@ultmaster
Copy link
Contributor

This pull request simplifies the test_run_with_http_tracer test in tests/test_tracer.py by skipping all HTTP tracer tests unconditionally, rather than skipping them only for specific agent types.

Testing changes:

  • Updated the test_run_with_http_tracer function to skip all tests for the HTTP tracer due to ongoing issues on GitHub Actions, instead of skipping only for certain agent implementations.
@ultmaster ultmaster changed the title Disable HTTP tracer tests Aug 5, 2025
@ultmaster ultmaster marked this pull request as ready for review August 5, 2025 12:05
Copilot AI review requested due to automatic review settings August 5, 2025 12:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request disables HTTP tracer tests that were causing issues on GitHub Actions and bumps the project version to 0.1.1 with an updated description.

  • Simplifies HTTP tracer test skipping by unconditionally skipping all tests instead of selectively skipping specific agent types
  • Bumps version from 0.1 to 0.1.1 across project files
  • Updates project description and adds CI badges to README

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
tests/test_tracer.py Replaces selective agent-specific test skipping with unconditional skip for all HTTP tracer tests
pyproject.toml Updates version to 0.1.1 and enhances project description
agentlightning/init.py Updates package version constant to match new version
README.md Adds CI status badges and updates tagline description
Comment on lines 847 to 848
import httpdbg.hooks.all

Copy link

Copilot AI Aug 5, 2025

Choose a reason for hiding this comment

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

The import statement import httpdbg.hooks.all is now unreachable due to the unconditional pytest.skip() call above it. Consider removing this import or moving it inside a conditional block if the test might be re-enabled in the future.

Suggested change
import httpdbg.hooks.all
Copilot uses AI. Check for mistakes.
@ultmaster ultmaster merged commit befec0e into main Aug 5, 2025
8 checks passed
@ultmaster ultmaster deleted the disable-http-tracer-tests branch October 28, 2025 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants