Skip to content

feat: Add swift wasm CI targets to DataLoader#26

Merged
NeedleInAJayStack merged 4 commits intoGraphQLSwift:mainfrom
PassiveLogic:feat/add-swift-wasm-to-ci
Feb 13, 2026
Merged

feat: Add swift wasm CI targets to DataLoader#26
NeedleInAJayStack merged 4 commits intoGraphQLSwift:mainfrom
PassiveLogic:feat/add-swift-wasm-to-ci

Conversation

@scottmarchant
Copy link
Contributor

@scottmarchant scottmarchant commented Jun 23, 2025

Change summary

Added swift wasm as a CI target, to help prevent future breakages to swift wasm builds in DataLoader

Details

  • Added ci stage that checks wasm compilation.
  • Temporarily pointing to a branch of swift-nio to verify wasm compilation, as swift-nio/NIOCore is currently broken. The swift-nio version will be bumped before merging this.
  • Bump swift version from 5.8 to 5.10.0, which is the new minimum for the latest swift-nio dependency.
  • Add wasi 0.1 build status badge to README.md.

Notes

  • Because the build status badge uses an unmerged workflow (.github/workflows/wasm.yaml), it can't be tested until this PR is merged. Some follow-on work may be needed after merging to ensure proper badge functionality.
  • This PR is part of a larger effort by PassiveLogic to add wasm support to many popular repositories.

PR Dependencies

The following PR's must be merged before this PR can be merged:

Testing done

Verified new github workflow builds, once required upstream dependencies are patched in swift-nio

See https://github.com/PassiveLogic/DataLoader/actions/runs/15833601501/job/44631947127?pr=1

Latest test results as of 2026 here:

PassiveLogic#2

Note that there are some lint errors from files not modified in this branch. Likely those lint errors are already in the latest main.

Remaining work

  • Update this PR after swift-nio wasm build is merged and released
  • After merging, verify badge works as expected. As noted, this can't be tested until after merging. Submit patch for new badge if needed.

@scottmarchant scottmarchant changed the title feat: Add swift wasm to ci to DataLoader feat: Add swift wasm CI targets to DataLoader Jun 23, 2025
@@ -1,4 +1,4 @@
// swift-tools-version:5.8
// swift-tools-version:5.10.0
Copy link
Contributor Author

@scottmarchant scottmarchant Jun 23, 2025

Choose a reason for hiding this comment

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

Note, this might be a breaking change for some users of this repository, depending on what swift version they are using. It shouldn't be very hard for anyone to move from Swift 5.8 to Swift 5.10, though.

@scottmarchant scottmarchant marked this pull request as ready for review June 24, 2025 20:34
@scottmarchant
Copy link
Contributor Author

The error for swift 5.8 should pass now that the ci change is merged.

error: 'dataloader': package 'dataloader' is using Swift tools version 5.10.0 but the installed version is 5.8.1

@scottmarchant scottmarchant force-pushed the feat/add-swift-wasm-to-ci branch from 35fb67c to ea25a54 Compare February 12, 2026 00:37
@scottmarchant scottmarchant force-pushed the feat/add-swift-wasm-to-ci branch from ea25a54 to ed8b795 Compare February 12, 2026 00:40
[![](https://img.shields.io/badge/License-MIT-blue.svg?style=flat)](https://tldrlegal.com/license/mit-license)
[![](https://img.shields.io/endpoint?url=https%3A%2F%2Fswiftpackageindex.com%2Fapi%2Fpackages%2FGraphQLSwift%2FDataLoader%2Fbadge%3Ftype%3Dswift-versions)](https://swiftpackageindex.com/GraphQLSwift/DataLoader)
[![](https://img.shields.io/endpoint?url=https%3A%2F%2Fswiftpackageindex.com%2Fapi%2Fpackages%2FGraphQLSwift%2FDataLoader%2Fbadge%3Ftype%3Dplatforms)](https://swiftpackageindex.com/GraphQLSwift/DataLoader)
[![WASI 0.1](https://github.com/GraphQLSwift/DataLoader/actions/workflows/wasm.yml/badge.svg?branch=main)](https://github.com/GraphQLSwift/DataLoader/actions/workflows/wasm.yml)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a somewhat theoretical change. Not sure on the best way to test this until it is merged.

@scottmarchant
Copy link
Contributor Author

@NeedleInAJayStack I've rebased this on the latest main. The lint error appears to be on files already merged in main, as those files aren't modified in this PR. Please let me know if you would like me to resolve those in this PR, or in a separate one.

@NeedleInAJayStack
Copy link
Member

Awesome!

The lint error appears to be on files already merged in main, as those files aren't modified in this PR. Please let me know if you would like me to resolve those in this PR, or in a separate one.

I'm okay with resolving those in this MR.

Copy link
Member

@NeedleInAJayStack NeedleInAJayStack left a comment

Choose a reason for hiding this comment

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

Thanks Scott!

@scottmarchant
Copy link
Contributor Author

@NeedleInAJayStack Lint has been addressed now.

@NeedleInAJayStack NeedleInAJayStack merged commit dbe2cef into GraphQLSwift:main Feb 13, 2026
10 checks passed
@scottmarchant scottmarchant deleted the feat/add-swift-wasm-to-ci branch February 13, 2026 16:59
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.

2 participants