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
Disable CloudWatch Log Group per function #7720
Disable CloudWatch Log Group per function #7720
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @AhmedFat7y ! It looks really good. Please see my remarks and let me know what you think
Additionally it'll be great to have it documented in: |
I added it here |
@medikoo Should I add these fixes as new commits or rebase and force push for cleaner history? |
Change disableLogs flag tests to use runServerless Ignore test fixtures .serverless dir Modify docs to include disableLogs flag
9abea1c
to
333af9d
Compare
333af9d
to
c7d0627
Compare
No there's no need. We will anyway make one squash commit out of it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AhmedFat7y Thank you! Looks great!
One side note: It appears that on your machine you don't have git setup with your email address. Note that your commits are not bound to your GitHub account (no avatar by them)
Hahahahahaha |
Closes: #7599