8000 [example] Group all examples under one project by sebsto · Pull Request #525 · swift-server/swift-aws-lambda-runtime · GitHub
[go: up one dir, main page]

Skip to content

[example] Group all examples under one project #525

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.

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

Closed
wants to merge 8 commits into from

Conversation

sebsto
Copy link
Contributor
@sebsto sebsto commented Jul 1, 2025

All the examples are now under one Swift project (one Package.swift file)

Motivation:

Building all the examples on my laptop took > 1h because each example downloaded and compiled the Lambda Runtime and its dependency (NIO)

Modifications:

I remove each individual example's Package.swift and consolidated all the targets into a single file.

I adjusted the README for each example (instruct user to use swift build --target xxx insead of swift build, and the same for the swift run and swift package archive commands

I adjusted the CI script

Result:

Examples build in 4 minutes on my laptop instead of ~60 min

@sebsto sebsto self-assigned this Jul 1, 2025
@sebsto sebsto added kind/enhancement Improvements to existing feature. semver/none No version bump required. area/testing Improvements to tests. labels Jul 1, 2025
@sebsto
8000
Copy link
Contributor Author
sebsto commented Jul 2, 2025

Closing this as this increases the build time during CI from ~8 min to ~20 min

@sebsto sebsto closed this Jul 2, 2025
@sebsto sebsto deleted the sebsto/examples branch July 4, 2025 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing Improvements to tests. kind/enhancement Improvements to existing feature. semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0