Skip to content

add instructions about TestEnv.jl to Sienna.md#586

Open
luke-kiernan wants to merge 3 commits into
mainfrom
lk/claude-test-env
Open

add instructions about TestEnv.jl to Sienna.md#586
luke-kiernan wants to merge 3 commits into
mainfrom
lk/claude-test-env

Conversation

@luke-kiernan

Copy link
Copy Markdown
Contributor

No description provided.

@luke-kiernan luke-kiernan requested review from Copilot and jd-lara and removed request for Copilot June 18, 2026 01:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Sienna agent/contribution guidance to document a TestEnv.jl-based test workflow alongside the existing --project=test commands, plus additional troubleshooting guidance.

Changes:

  • Adds shell one-liners for running all tests or a specific test file using TestEnv.activate().
  • Adds a note about avoiding environment-merge issues when using TestEnv.jl.
  • Extends troubleshooting with an “Environment issues” section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .claude/Sienna.md Outdated
Comment thread .claude/Sienna.md Outdated
Comment thread .claude/Sienna.md Outdated
Comment thread .claude/Sienna.md Outdated
luke-kiernan and others added 2 commits June 17, 2026 19:18
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.86%. Comparing base (5f7f677) to head (83cfb3f).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #586      +/-   ##
==========================================
- Coverage   78.88%   78.86%   -0.02%     
==========================================
  Files          75       75              
  Lines        6660     6649      -11     
==========================================
- Hits         5254     5244      -10     
+ Misses       1406     1405       -1     
Flag Coverage Δ
unittests 78.86% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@luke-kiernan

luke-kiernan commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

Turns out using TestEnv; TestEnv.activate(); push!(ARGS, "test_file_name"); include("test/runtests.jl") doesn't work. Will revise

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