Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add 'gdal vector sql', as standalone or part of 'gdal vector pipeline' #11827

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Add 'gdal vector sql', as standalone or part of 'gdal vector pipeline' #11827

merged 1 commit into from
Feb 10, 2025

Conversation

rouault
Copy link
Member

@rouault rouault commented Feb 9, 2025

No description provided.

@rouault rouault added funded through GSP Work funded through the GDAL Sponsorship Program gdal_cli Anything related to the new 3.11 "gdal" CLI frontend labels Feb 9, 2025
@rouault rouault added this to the 3.11.0 milestone Feb 9, 2025
@rouault rouault force-pushed the gdal_vector_sql branch 3 times, most recently from ba55db5 to 90d9637 Compare February 9, 2025 23:51
@coveralls
Copy link
Collaborator

coveralls commented Feb 9, 2025 •
edited
Loading

Coverage Status

coverage: 70.066% (+0.003%) from 70.063%
when pulling 90d9637 on rouault:gdal_vector_sql
into 6332083 on OSGeo:master.

elpaso
elpaso approved these changes Feb 10, 2025
View reviewed changes
Copy link
Collaborator

@elpaso elpaso left a comment

Choose a reason for hiding this comment

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

Nice job!
I just left some minor comments.

apps/gdalalg_vector_sql.cpp Outdated Show resolved Hide resolved
apps/gdalalg_vector_sql.cpp Show resolved Hide resolved
apps/gdalalg_vector_sql.cpp Show resolved Hide resolved
apps/gdalalg_vector_sql.h Show resolved Hide resolved
doc/source/programs/gdal_vector_sql.rst Outdated Show resolved Hide resolved
doc/source/programs/gdal_vector_sql.rst Outdated Show resolved Hide resolved
gcore/gdalalgorithm.cpp Show resolved Hide resolved
@rouault rouault force-pushed the gdal_vector_sql branch 2 times, most recently from b2bc307 to d8d115b Compare February 10, 2025 14:22
@rouault rouault merged commit 5935820 into OSGeo:master Feb 10, 2025
37 checks passed
@dbaston dbaston mentioned this pull request Feb 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jratike80 jratike80 jratike80 left review comments

@elpaso elpaso elpaso approved these changes

Assignees
No one assigned
Labels
funded through GSP Work funded through the GDAL Sponsorship Program gdal_cli Anything related to the new 3.11 "gdal" CLI frontend
Projects
None yet
Milestone
3.11.0
Development

Successfully merging this pull request may close these issues.
4 participants
@rouault @coveralls @elpaso @jratike80