Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 18, 2025

Compiler-dependent tests should gracefully skip when llvmlite or backend bits aren’t available.

  • Test robustness
    • Wrap architecture-agnostic suite with a module-level skip when compiler deps are missing.
    • Use string-based annotations in compiler tests to avoid NameErrors during collection without llvmlite.
  • Clarity
    • Keep type expectations while preventing optional dependency import failures.
# Module-level guard
try:
    from eigenscript.compiler.codegen.llvm_backend import LLVMCodeGenerator
except ImportError:
    pytest.skip("Compiler dependencies not installed", allow_module_level=True)
Original prompt

Look for areas of improvement


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Dec 18, 2025
Co-authored-by: InauguralPhysicist <217690737+InauguralPhysicist@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate potential areas of improvement Guard compiler-dependent tests for optional backends Dec 18, 2025
@InauguralPhysicist InauguralPhysicist marked this pull request as ready for review December 18, 2025 23:53
@InauguralPhysicist InauguralPhysicist merged commit 2cf2d9d into main Dec 18, 2025
5 checks passed
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