Skip to content

Conversation

@mustiikhalil
Copy link
Collaborator

@mustiikhalil mustiikhalil commented Jun 22, 2025

Adding support for windows requires the code generations
to add a compiler statement to completely ignore GRPC code
generation on windows

Cleanup the project to use the main Package.swift to run tests
instead of having it separate and includes the imports for GRPC
within it.

@github-actions github-actions bot added c++ CI Continuous Integration grpc swift python labels Jun 22, 2025
@mustiikhalil mustiikhalil force-pushed the add-swift-windows-support branch 4 times, most recently from 2caabc6 to e103c3e Compare June 27, 2025 22:59
@mustiikhalil mustiikhalil self-assigned this Jun 29, 2025
Adding support for windows requires the code generations
to add a compiler statement to completely ignore GRPC code
generation on windows

Cleanup the project to use the main Package.swift to run tests
instead of having it separate and includes the imports for GRPC
within it.

Adds windows swift ci
@mustiikhalil mustiikhalil force-pushed the add-swift-windows-support branch from e103c3e to 887cdbd Compare June 29, 2025 18:55
@aardappel aardappel merged commit b8db3a9 into google:master Jun 30, 2025
50 checks passed
@aardappel
Copy link
Collaborator

Thanks!

@mustiikhalil mustiikhalil deleted the add-swift-windows-support branch June 30, 2025 12:46
dongjoon-hyun added a commit to apache/spark-connect-swift that referenced this pull request Oct 28, 2025
### What changes were proposed in this pull request?

This PR aims to upgrade `FlatBuffers` to `v25.9.23`

### Why are the changes needed?

To bring the latest bug fixes and improvements like `Windows` support.
- https://github.com/google/flatbuffers/releases/tag/v25.9.23 (2025-09-23)
  - google/flatbuffers#8484
  - google/flatbuffers#8577
  - google/flatbuffers#8622
  - google/flatbuffers#8637
  - google/flatbuffers#8643
  - google/flatbuffers#8650
  - google/flatbuffers#8649
  - google/flatbuffers#8702

### Does this PR introduce _any_ user-facing change?

No. There is no behavior change.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #254 from dongjoon-hyun/SPARK-54045.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants