Update default role policy - #45
Conversation
| @@ -1,5 +1,5 @@ | |||
| name: aws-lambda | |||
| version: 2.0.2 | |||
| version: 3.0.0 | |||
There was a problem hiding this comment.
I think this should not be a major release, maybe a patch is a better fit?
| version: 3.0.0 | |
| version: 2.0.3 |
|
@MattiaPette I'm not sure, but this is how the express component did it, possibly because it's a breaking change. Are you sure I should change it? |
|
Any idea how long this PR could take? Can this be fixed locally whilst we wait? |
|
This currently leads to the normal "Deploy Lambda" getting started project on https://app.serverless.com/ being broken :/ Update: As a hack I published |
Yes I can confirm that, it's broken because of this. I tried tearing apart the npm package |
|
If you are using that example, is says |
|
@janpio - thanks! I'm a Serverless beginner and there's nothing quite like following the default flow and it throwing an error you don't understand. I changed the serverless.yml to your update and it deployed successfully. I am in your debt. If I've understood correctly, this is a breaking bug for all new lambda components? (including new, easily confused people like me). Is there some way we can increase the priority for the merge? |
eahefnawy
left a comment
There was a problem hiding this comment.
Thanks for this fix @alisalahio! 🙌
|
This is now published! Thanks all! |
closes https://github.com/serverless/components/issues/918#issuecomment-814418217