Pathlib and perf

[amilcarlucas]

No description provided.

[Copilot]

Pull Request Overview

This pull request modernizes the codebase by replacing os.path and os module calls with pathlib.Path operations, providing better performance and more readable code.

Key changes:

• Replaces all os.path operations with equivalent pathlib.Path methods
• Removes unused os module imports
• Improves code readability and maintainability through modern Python path handling

[Copilot]

The function name get_directory_name_from_full_path suggests it should return a directory name, but Path.name returns the final component of the path (which could be a file). The original code used os.path.basename(os.path.split(normalized_path)[1]) which had different behavior. This change may break functionality if callers expect directory-specific behavior.

Suggested change

	# Use pathlib for cleaner path operations
	return Path(full_path).name
	# Return the name of the parent directory, even if the path ends with a file
	return Path(full_path).parent.name

[Copilot]

The cwd parameter expects a string path, but Path(__file__).parent returns a Path object. This should be converted to string: cwd=str(Path(__file__).parent)

Suggested change

	cwd=Path(__file__).parent,
	cwd=str(Path(__file__).parent),

[Copilot]

The open() function expects a string path, but git_hash_file is a Path object. Convert to string: with open(str(git_hash_file), encoding=\"utf-8\") as file:

Suggested change

	with open(git_hash_file, encoding="utf-8") as file:
	with open(str(git_hash_file), encoding="utf-8") as file:

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
8376	6443	77%	73%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
ardupilot_methodic_configurator/backend_filesystem.py	73%	🟢
TOTAL	73%	🟢

updated for commit: d783fcd by action🐍

[github-actions]

Test Results

1 743 tests   1 719 ✅  52s ⏱️
    1 suites      1 💤
    1 files       23 ❌

For more details on these failures, see this check.

Results for commit d783fcd.

♻️ This comment has been updated with latest results.