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

Log invalid lines #283

Merged
merged 1 commit into from
Sep 14, 2021
Merged

Log invalid lines #283

merged 1 commit into from
Sep 14, 2021

Conversation

ungaratto93
Copy link
Contributor

Hi, I had a issue with my .env that I mistyped something and couldn't figure out what was wrong.
So I added a log when the regex doesn't match.

@sergeyklay sergeyklay deleted the branch joke2k:develop September 4, 2021 12:00
@sergeyklay sergeyklay closed this Sep 4, 2021
@sergeyklay sergeyklay reopened this Sep 4, 2021
@coveralls
Copy link

coveralls commented Sep 4, 2021

Coverage Status

Coverage decreased (-0.06%) to 89.391% when pulling 32e473f on ungaratto93:'log-invalid-lines' into dff27c0 on joke2k:develop.

@sergeyklay
Copy link
Collaborator

PR was closed by mistake. Reopened it.

@sergeyklay sergeyklay added the enhancement New feature or request label Sep 7, 2021
@sergeyklay sergeyklay self-requested a review September 14, 2021 23:10
@sergeyklay sergeyklay self-assigned this Sep 14, 2021
@sergeyklay sergeyklay merged commit 192fea9 into joke2k:develop Sep 14, 2021
@sergeyklay
Copy link
Collaborator

Merged! Thank you for the patch, and for helping us make django-environ better. And I am sorry about the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants