Skip to content

feat(resource): Enable the use of multiple paths as input - #22

Merged
papazof merged 1 commit into
mainfrom
feat_resource_list_input
Jun 15, 2025
Merged

papazof merged 1 commit into
mainfrom
feat_resource_list_input

Conversation

@papazof

@papazof papazof commented Jun 15, 2025

Copy link
Copy Markdown
Contributor

Enable the use of both single and multiple path resources:

  • Extend the Resource initializer to handle lists of paths
  • Correct the CList class to properly handle lists of str objects

@papazof
papazof marked this pull request as ready for review June 15, 2025 13:50
@papazof
papazof requested a review from ananos June 15, 2025 13:52

@ananos ananos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks @papazof!

Enable the use of both single and multiple path resources:
- Extend the `Resource` initializer to handle lists of paths
- Correct the `CList` class to properly handle lists of `str` objects

PR: #22
Signed-off-by: Kostis Papazafeiropoulos <papazof@gmail.com>
Reviewed-by: Anastassios Nanos <ananos@nubificus.co.uk>
Approved-by: Anastassios Nanos <ananos@nubificus.co.uk>
@github-actions
github-actions Bot force-pushed the feat_resource_list_input branch from fdcf6f8 to de83bf4 Compare June 15, 2025 14:42
@github-actions

Copy link
Copy Markdown

Coverage report diff

Filename Stmts Miss Cover
_c_types/wrappers/clist.py +6 0 +2.34%
resource.py +3 0 +1.21%
TOTAL +9 0 +0.19%

@papazof
papazof merged commit 320e413 into main Jun 15, 2025
@papazof
papazof deleted the feat_resource_list_input branch June 15, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants