Skip to content

Conversation

@sja26
Copy link
Contributor

@sja26 sja26 commented Aug 11, 2022

Reduce compiler warnings and errors from sample code.

Motivation:

As a first time visitor, I noticed that the sample code doesn't run without error. I would like to help fix it for future first time visitors so they aren't put off or give up.

Modifications:

Modified sample code in the README, more details can be found in the commit log.

Result:

1 warning and 1 error fixed.

sja26 added 2 commits August 11, 2022 12:35
'MyLambda' was identified as an executable target given the presence of a 'main.swift' file. Starting with tools version 5.4.0 executable targets should be declared as 'executableTarget()'
The swift-aws-lambda-events repo has not yet pushed tag 0.1.0 so the build fails, see output from swift build:

Computing version for https://github.com/swift-server/swift-aws-lambda-events.git
error: Dependencies could not be resolved because no versions of 'swift-aws-lambda-events' match the requirement 0.1.0..<1.0.0 and root depends on 'swift-aws-lambda-events' 0.1.0..<1.0.0.

This commit adds the workaround mentioned in the issue for the missing tag awslabs/swift-aws-lambda-events#3
@swift-server-bot
Copy link

Can one of the admins verify this patch?

14 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@tomerd
Copy link
Contributor

tomerd commented Aug 11, 2022

thanks for this @sja26, the docs have "moved too fast" to reflect the upcoming changes in 1.0

@tomerd
Copy link
Contributor

tomerd commented Aug 11, 2022

@swift-server-bot test this please

@tomerd tomerd enabled auto-merge (squash) August 11, 2022 17:11
@tomerd tomerd merged commit 568897c into awslabs:main Aug 11, 2022
@sja26 sja26 deleted the update-readme branch August 11, 2022 19:33
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.

3 participants