Skip to content

Use consistent _parquet suffix in GDF parquet driver functions#21

Merged
nathanjmcdougall merged 3 commits intodevelopfrom
copilot/fix-driver-function-suffixes
Feb 25, 2026
Merged

Use consistent _parquet suffix in GDF parquet driver functions#21
nathanjmcdougall merged 3 commits intodevelopfrom
copilot/fix-driver-function-suffixes

Conversation

Copy link
Contributor

Copilot AI commented Feb 23, 2026

Both GDF parquet driver functions now use the _parquet suffix consistently.

Changes

  • drivers/gdf/filetypes/parquet.py: Renamed pin_read_gdf_geoparquetpin_read_gdf_parquet
  • drivers/gdf/dispatch.py: Updated imports and call sites to match

Both functions now follow the same _parquet suffix:

pin_read_gdf_parquet(...)   # renamed from pin_read_gdf_geoparquet
pin_write_gdf_parquet(...)  # unchanged
Original prompt

This section details on the original issue you should resolve

<issue_title>Use consistent suffixes in driver functions</issue_title>
<issue_description>

_parquet not _geoparquet</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…nt suffixes

Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix driver functions to use consistent suffixes Rename pin_write_gdf_parquetpin_write_gdf_geoparquet Feb 23, 2026
…in_read_gdf_parquet

Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copilot AI changed the title Rename pin_write_gdf_parquetpin_write_gdf_geoparquet Use consistent _parquet suffix in GDF parquet driver functions Feb 23, 2026
@t-t-sonarqube
Copy link

t-t-sonarqube bot commented Feb 23, 2026

Passed Quality Gate passed

Issues

Measures

Project ID: tonkintaylor_geopins-python_93c10492-5d39-4c44-bcf3-bd14cb206319

View in SonarQube

@b-j-karl b-j-karl marked this pull request as ready for review February 25, 2026 02:09
@nathanjmcdougall nathanjmcdougall merged commit ee15d8b into develop Feb 25, 2026
15 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.

Use consistent suffixes in driver functions

3 participants