search

logdna / logdna-cloudwatch

AWS Lambda for logging from CloudWatch into LogDNA
MIT License
12 stars 19 forks source link

Fix positional args. #8

Closed lextoumbourou closed 5 years ago

lextoumbourou commented 5 years ago

Hi gang,

This code is currently broken due to this error:

Syntax error in module 'logdna_cloudwatch': non-default argument follows default argument (logdna_cloudwatch.py, line 64)

which was introduced in this PR.

This PR should fix it, but it might be worth adding either linting or some simple unit tests to find issues like this.

smusali commented 5 years ago

Thanks, @lextoumbourou, I just wanted to start fixing this - you did it before me.