← Back to issue list

Declare instance paths as PurePath

View original Github issue

Metadata

Project
craft-providers
Number
#83
Type
pull request
State
merged
Author
cjwatson
Labels
—
Created
Updated
Closed

Current evaluation

Merged #83 in commit 411cec5 (implementation 11e96d3), changing instance-only path annotations from pathlib.Path to pathlib.PurePath.

Resolution: closed resolved

Reason: The approved PR was merged as commit 411cec5. Its implementation commit, 11e96d3, declares instance paths as PurePath because they are handled by the instance rather than accessed directly on the host filesystem.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Paths that are only handled on the instance side can be declared as `pathlib.PurePath` rather than the stronger type `pathlib.Path`, since this Python process doesn't need to (and shouldn't) do any direct filesystem access using them.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #83 in commit 411cec5 (implementation 11e96d3), changing instance-only path annotations from pathlib.Path to pathlib.PurePath.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR declaring instance-side paths as `pathlib.PurePath` instead of `pathlib.Path`, reflecting that the current process requires no direct filesystem access for these paths.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#135 Declare more instance paths as PurePath craft-providers merged Merged as commit 59ea19b, updating additional instance path annotations to PurePath so downstream consumers type-check correctly.
86%