Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

📚 Fix Postgres connection string examples #362

Merged
merged 2 commits into from
May 31, 2023
Merged

📚 Fix Postgres connection string examples #362

merged 2 commits into from
May 31, 2023

Conversation

spoukke
Copy link
Contributor

@spoukke spoukke commented May 31, 2023

No description provided.

@spoukke spoukke requested a review from busser as a code owner May 31, 2023 07:51
busser
busser requested changes May 31, 2023
View reviewed changes
Copy link
Owner

@busser busser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think this PR should also fix the connection string in the jsonpath filter section :)

@spoukke
Copy link
Contributor Author

spoukke commented May 31, 2023

Thanks! I think this PR should also fix the connection string in the jsonpath filter section :)

Done 👍

@spoukke spoukke requested a review from busser May 31, 2023 12:48
@busser
Copy link
Owner

busser commented May 31, 2023

I'm going to force merge this branch despite the failing workflow, since the breakage is unrelated to the proposed changes.

I have opened issue #364 to track the workflow problem.

@busser busser changed the title docs: fix postgres connection string in readme 📚 Fix Postgres connection string example May 31, 2023
@busser busser merged commit 4b77d25 into busser:main May 31, 2023
@busser busser changed the title 📚 Fix Postgres connection string example 📚 Fix Postgres connection string examples May 31, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@busser busser busser approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@spoukke @busser