labgrid-project / labgrid-project/labgrid

Replace string based path handling with pathlib.Path

Open
#738 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue
Dominant language
Python
Stars
528
Forks
278
Avg merge
2d 19h
Merged PRs (30d)
4

Description

We handle serveral paths within labgrid as strings and should switch to a pathlib.Path based handling. Since python3.6 all the os functions can handle this natively, this should remove some stringifications which were required for python 3.5.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the serveral path-handling sites in labgrid and the os calls currently receiving stringified paths. Audit how these paths flow through the library, then identify the stringifications that were only needed for Python 3.5. Done means the relevant handling uses pathlib.Path and remains compatible with Python 3.6 os functions.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
embedded-iot
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.