Skip to content

Remove "unused" files in client/python#1678

Merged
flyrain merged 1 commit intoapache:mainfrom
jbonofre:python-cleanup
Jun 7, 2025
Merged

Remove "unused" files in client/python#1678
flyrain merged 1 commit intoapache:mainfrom
jbonofre:python-cleanup

Conversation

@jbonofre
Copy link
Member

A couple of files in client/python seem "historical" and should be cleaned up.

adutra
adutra previously approved these changes May 26, 2025
@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board May 26, 2025
dimas-b
dimas-b previously approved these changes May 26, 2025
@eric-maynard
Copy link
Contributor

Can you also clean up the corresponding entries in this file?

@jbonofre
Copy link
Member Author

@eric-maynard good catch. Let me do it. Thanks !

MonkeyCanCode
MonkeyCanCode previously approved these changes May 26, 2025
HonahX
HonahX previously approved these changes Jun 2, 2025
Copy link
Contributor

@adnanhemani adnanhemani left a comment

Choose a reason for hiding this comment

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

LGTM

@jbonofre jbonofre dismissed stale reviews from MonkeyCanCode, dimas-b, and adutra via 3d85b63 June 6, 2025 22:53
@jbonofre jbonofre requested review from adutra and dimas-b June 6, 2025 22:55
@jbonofre
Copy link
Member Author

jbonofre commented Jun 6, 2025

@eric-maynard I did a more "complete" cleanup. Do you mind to take a new look ? Thanks !

@flyrain
Copy link
Contributor

flyrain commented Jun 6, 2025

Should we also remove .gitlab-ci.yml in file .openapi-generator-ignore?

@jbonofre
Copy link
Member Author

jbonofre commented Jun 6, 2025

@flyrain good catch, done 😄

@flyrain flyrain merged commit b3a4299 into apache:main Jun 7, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Jun 7, 2025
snazy added a commit to snazy/polaris that referenced this pull request Jun 13, 2025
* main: Update dependency io.netty:netty-codec-http2 to v4.2.2.Final (apache#1819)

* main: Update dependency com.google.cloud:google-cloud-storage-bom to v2.53.0 (apache#1811)

* main: Pin actions/setup-java action to c5195ef (apache#1809)

* main: Update dependency boto3 to v1.38.31 (apache#1803)

* Downgrade open api generator to 7.11 (apache#1823)

* main: Update dependency io.smallrye.config:smallrye-config-core to v3.13.1 (apache#1826)

* JDBC: Use PreparedStatement (apache#1802)

* Remove "unused" files in client/python (apache#1678)

* main: Update dependency org.junit:junit-bom to v5.13.1 (apache#1832)

* main: Update dependency boto3 to v1.38.32 (apache#1828)

* main: Update dependency io.opentelemetry:opentelemetry-bom to v1.51.0 (apache#1829)

* Spark: Add license and notice to spark client jar and push jar to maven (apache#1830)

* Update versions in LICENSE and NOTICE (apache#1833)

* Core: Fix  maxCode generation for ConnectionTypeEnum (apache#1827)

* main: Update dependency com.github.ben-manes.caffeine:caffeine to v3.2.1 (apache#1836)

* main: Update dependency software.amazon.awssdk:bom to v2.31.59 (apache#1835)

* Fix the issue where the Polaris Server exposes backend metadata in the error response body when the database is not bootstrapped. (apache#1837)

* Fix LICENSE and NOTICE in Spark plugin (apache#1834)

* main: Update dependency io.micrometer:micrometer-bom to v1.15.1 (apache#1842)

* main: Update dependency boto3 to v1.38.33 (apache#1841)

* NoSQL: bump dependencies

* Last merged commit 9934fc4

---------

Co-authored-by: Mend Renovate <[email protected]>
Co-authored-by: Honah (Jonas) J. <[email protected]>
Co-authored-by: Prashant Singh <[email protected]>
Co-authored-by: JB Onofré <[email protected]>
Co-authored-by: gh-yzou <[email protected]>
Co-authored-by: Pooja Nilangekar <[email protected]>
Co-authored-by: Bo Wang <[email protected]>
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.

8 participants