Skip to content

Add dedicated path type throughout the codebase#6819

Closed
ayushjariyal wants to merge 5 commits into
aiidateam:mainfrom
ayushjariyal:issue_6814
Closed

Add dedicated path type throughout the codebase#6819
ayushjariyal wants to merge 5 commits into
aiidateam:mainfrom
ayushjariyal:issue_6814

Conversation

@ayushjariyal
Copy link
Copy Markdown
Contributor

issue #6814

In this PR, I introduce a unified PathType alias in aiida/common/types.py to replace inconsistent path-like annotations.
This improves consistency and readability.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.30%. Comparing base (660fec7) to head (0a6e54b).
⚠️ Report is 292 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6819      +/-   ##
==========================================
- Coverage   78.31%   78.30%   -0.01%     
==========================================
  Files         566      567       +1     
  Lines       42762    42771       +9     
==========================================
+ Hits        33484    33486       +2     
- Misses       9278     9285       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@danielhollas danielhollas changed the title Adding Dedicated path througout the codebase Add dedicated path type throughout the codebase Apr 22, 2025
Copy link
Copy Markdown
Collaborator

@khsrali khsrali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ayushjariyal
Overall looks good!
Please also unify TransportPath with this new path.

@ayushjariyal ayushjariyal requested a review from khsrali April 23, 2025 18:24
@GeigerJ2 GeigerJ2 self-assigned this Aug 19, 2025
@khsrali khsrali closed this Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants