← Back to issue list

utils: refactor os utils to use pathlib

View original Github issue

Metadata

Project
craft-parts
Number
#264
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #264 in commit 3a326f90, refactoring get_bin_paths(), get_include_paths(), and get_library_paths() to use pathlib instead of os.path.

Resolution: closed resolved

Reason: PR #264 was approved and merged as commit 3a326f90, replacing os.path usage with pathlib in the three path utility functions.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Update `get_bin_paths()`, `get_include_paths()`, `get_library_paths()` to use pathlib. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #264 in commit 3a326f90, refactoring get_bin_paths(), get_include_paths(), and get_library_paths() to use pathlib instead of os.path.
stealth/space-bunny-alpha — closed resolved Maintainer PR #264 by cmatsuoka merged into main as 3a326f90 (commit 84390514), converting get_bin_paths(), get_include_paths(), and get_library_paths() from os.path to pathlib (+40/-38, 2 files).
stealth/space-bunny-alpha — closed resolved Merged #264 via commit 3a326f90, refactoring get_bin_paths(), get_include_paths(), and get_library_paths() to use pathlib instead of os.path.
qwen/qwen3.6-35b-a3b — — Merged refactoring that updates get_bin_paths, get_include_paths, and get_library_paths to use pathlib. Approved by one reviewer, passed CI checks, and modified two files.
qwen3.6-35b-a3b-mtp-q6 — — Merged refactoring of get_bin_paths, get_include_paths, and get_library_paths to use pathlib. Maintainer Claudio Matsuoka authored the change, which was applied directly without comments or revisions.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1424 refactor: use pathlib internally craft-parts merged Merged PR #1424 as commit cfdabafc, replacing internal os.path calls with pathlib across 95 files while preserving the public API and migration-state serialization.
76%