Skip to content
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

fix failing CI: upgrade @types packages, apollo-server-lambda #2738

Merged
merged 3 commits into from Jun 4, 2021

Conversation

thedavidprice
Copy link
Contributor

@thedavidprice thedavidprice commented Jun 4, 2021

CI is failing E2E CLI build command due to type issue related to the /api package and @types/aws-lamdba and apollo-server-lambda.

  • upgrade packages
  • add missing type to packages/api/src/functions/fixtures/apiGatewayProxyEvent.fixture.ts

@cypress
Copy link

cypress bot commented Jun 4, 2021



Test summary

17 0 0 0Flakiness 2


Run details

Project RedwoodJS Framework
Status Passed
Commit 67edc07 ℹ️
Started Jun 4, 2021 9:04 PM
Ended Jun 4, 2021 9:16 PM
Duration 12:05 💡
OS Linux Ubuntu - 20.04
Browser Chrome 91

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@thedavidprice thedavidprice changed the title [chore] rebuild yarn.lock fix failing CI: upgrade @types packages, apollo-server-lambda Jun 4, 2021
@@ -22,6 +22,7 @@ export const mockedAPIGatewayProxyEvent: APIGatewayProxyEvent = {
apiKey: null,
apiKeyId: null,
caller: null,
clientCert: null,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dac09 It seems the @types/aws-lambda package required this and was failing during build.

If CI passes, I'm going to merge. But please review this and update as needed in future PR.

@thedavidprice thedavidprice requested a review from dac09 June 4, 2021 21:04
@thedavidprice
Copy link
Contributor Author

🚀

@thedavidprice thedavidprice merged commit cda70e7 into main Jun 4, 2021
@thedavidprice thedavidprice deleted the dsp-update-yarn.lock-v0.33 branch June 4, 2021 21:17
@github-actions github-actions bot added this to the next-release milestone Jun 4, 2021
@thedavidprice thedavidprice modified the milestones: next-release, v0.33.2 Jun 7, 2021
thedavidprice added a commit that referenced this pull request Jun 7, 2021
* rebuild yarn.lock

* upgrade types packages

* update api/.../fixtures type
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.

None yet

1 participant