Document the skip keyword argument of @testitem - #40
Merged
Conversation
`skip` accepts either a `Bool` literal or an arbitrary expression. The expression is evaluated in the *test process*, immediately before the test item would have run, so checks like `VERSION < v"1.11"` or `Sys.iswindows()` see the environment the tests actually run in rather than the one that discovered them. Defaults to `false`. The macro remains a no-op marker; parsing lands in TestItemDetection.jl and the value is carried through JuliaWorkspaces.jl. Part of stream S1 (Discovery) of the ReTestItems feature-delta plan (A.10 item 5). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This was referenced Aug 13, 2026
davidanthoff
marked this pull request as ready for review
August 13, 2026 23:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of stream S1 (Discovery) of the ReTestItems feature-delta plan — A.10 item 5, "
skipas an item kwarg".What
Documents
skipin the@testitemdocstring, and adds two examples. The macro itself stays a no-op marker; nothing executable changes in this repo.Semantics documented:
skipaccepts aBoolliteral or an arbitrary expression evaluating to aBool.skipan item keyword rather than a coordinator-side filter:VERSION < v"1.11"andSys.iswindows()must see the worker's environment, not the environment that discovered the item (we support--julia-cmdpointing anywhere, and CI matrices routinely run several Julia versions).false.Companion PRs (land in this order)
skipinfind_test_detail!, returnoption_skip.option_skiponTestItemDetail, plus stable test item ids.Execution of
skip(TestItemControllers / TestItemApp) is a later wave.Testing
Pkg.test()green (4/4). No test items exercise the docstring; behaviour is tested in TestItemDetection.jl and JuliaWorkspaces.jl.🤖 Generated with Claude Code