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

Fix typo #199

Merged
merged 1 commit into from Oct 2, 2019
Merged

Fix typo #199

merged 1 commit into from Oct 2, 2019

Conversation

larshesel
Copy link
Contributor

I don't know if it has any adverse effects, but it showed up in the logs and I had to investigate if the undefned came from our application code or from the epgsql library - so I thought it was worth a PR.

@seriyps
Copy link
Member

seriyps commented Oct 1, 2019

Looks good, thanks! Failing build seems unrelated, but I would prefer to try to fix it before merging your PR.

@larshesel
Copy link
Contributor Author

Hi, is there anything I can do in that regard? The build that failed seems to be due to a wget command failing. Can you restart the build?

@seriyps
Copy link
Member

seriyps commented Oct 1, 2019

It't similar to gen-smtp/gen_smtp#179

@larshesel
Copy link
Contributor Author

ok - I'll try and add something like that.

@seriyps
Copy link
Member

seriyps commented Oct 1, 2019 via email

@larshesel
Copy link
Contributor Author

sure

@larshesel
Copy link
Contributor Author

Opened a fix for travis in #200 and rebased this PR on top.

@larshesel
Copy link
Contributor Author

Rebased this on top of devel.

@seriyps seriyps merged commit 5304480 into epgsql:devel Oct 2, 2019
@seriyps
Copy link
Member

seriyps commented Oct 2, 2019

Thank you!

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

2 participants