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

Clearer explanation for why the fork occurred, indicate the original … #137

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

mikebell90
Copy link

@mikebell90 mikebell90 commented Sep 10, 2024

…library continues, and link to discussion of options

Addresses #119

@tomix26
Copy link
Collaborator

tomix26 commented Sep 27, 2024

Thanks for updating the readme. I've made some minor adjustments to better highlight the purpose of this fork. I also removed the pros and cons section as I felt it wasn't entirely objective.

The following points are supported by my version:

  • multi-architecture support
  • works the same way on Mac, Windows, Linux
  • support for PostgreSQL 10+
  • easy version upgrades / changes

I do agree with the points about better manageability and extension support, though. So if you think a comparison of pros and cons would be beneficial, feel free to prepare a new pull request and we can discuss it further.

@tomix26 tomix26 merged commit 389fc61 into zonkyio:master Sep 27, 2024
6 checks passed
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.

2 participants