Skip to content

chore: UPDATE biosequence, message and geopandas datasets use of os.PathLike#1336

Merged
merelcht merged 3 commits intokedro-org:mainfrom
juanchodpg2:feat/issue_1316_biosequence_message_geopandas
Mar 2, 2026
Merged

chore: UPDATE biosequence, message and geopandas datasets use of os.PathLike#1336
merelcht merged 3 commits intokedro-org:mainfrom
juanchodpg2:feat/issue_1316_biosequence_message_geopandas

Conversation

@juanchodpg2
Copy link
Copy Markdown
Contributor

@juanchodpg2 juanchodpg2 commented Feb 27, 2026

Description

Solves part of #1316. Updates the filepath type hint from str to str | os.PathLike for the following Datasets:

BioSequence
Email
Geopandas

Development notes

Unit tests working for the changes implemented.

Checklist

  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Updated jsonschema/kedro-catalog-X.XX.json if necessary
  • Added a description of this change in the relevant RELEASE.md file
  • Added tests to cover my changes
  • Received approvals from at least half of the TSC (required for adding a new, non-experimental dataset)

Signed-off-by: Juan David Patino Guerra <juan.patino.guerra@ing.com>
@juanchodpg2 juanchodpg2 force-pushed the feat/issue_1316_biosequence_message_geopandas branch from 95b6d06 to 9da9810 Compare February 27, 2026 19:19
@merelcht merelcht changed the title UPDATE biosequence, message and geopandas datasets use of os.PathLike chore: UPDATE biosequence, message and geopandas datasets use of os.PathLike Feb 28, 2026
Copy link
Copy Markdown
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your contribution @juanchodpg2

Signed-off-by: Merel Theisen <merel.theisen@quantumblack.com>
@merelcht merelcht enabled auto-merge (squash) February 28, 2026 12:15
@merelcht merelcht requested review from SajidAlamQB and lrcouto March 2, 2026 12:20
Copy link
Copy Markdown
Contributor

@SajidAlamQB SajidAlamQB left a comment

Choose a reason for hiding this comment

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

Thank you, @juanchodpg2! ⭐

@merelcht merelcht merged commit 7ea4de8 into kedro-org:main Mar 2, 2026
16 checks passed
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