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

Spit traceback to stdout when error loading app #78

Merged
merged 2 commits into from
Dec 18, 2018
Merged

Spit traceback to stdout when error loading app #78

merged 2 commits into from
Dec 18, 2018

Conversation

nx-jackal
Copy link
Contributor

When loaded app. having error such as missing import or syntax error, then sls logs -t -f function_name doesn't give any clue at all.

When loaded app. having error such as missing import or syntax error, then `sls logs -t -f function_name` doesn't give any clue at all.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 99.494% when pulling 53d751c on gs-jackal:patch-1 into 9372c05 on logandk:master.

@nx-jackal
Copy link
Contributor Author

@logandk I'm unsure how to fix the coverage test error, any help?

@logandk logandk merged commit 53d751c into logandk:master Dec 18, 2018
@logandk
Copy link
Owner

logandk commented Dec 18, 2018

@gs-jackal Thanks for the PR! 👍 I added a test to fix the coverage error

@nx-jackal
Copy link
Contributor Author

Cool, would be nice if you can push a release for current master branch? Thanks!

@nx-jackal nx-jackal deleted the patch-1 branch December 18, 2018 22:11
@logandk
Copy link
Owner

logandk commented Dec 18, 2018

Sure, it's out now - wanted to get a few other things into this release

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

3 participants