-
Notifications
You must be signed in to change notification settings - Fork 9
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
Check for blank environment variables #75
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was pairing with @wildmaples to add this to our CI, and it was failing with:
It turned out we were setting
SMART_TODO_SLACK_TOKEN
, but it was evaluating to an empty string 🙊 . I think it's a simple adjustmen to check for it to be non-empty.The text was updated successfully, but these errors were encountered: