-
Notifications
You must be signed in to change notification settings - Fork 37
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
is this project alive ?? #22
Comments
I guess paying to free some of the @fschulze time could solve the problem for now. |
It would, or someone takes over maintenance. |
I can't volunteer to totally take over maintenance but I'd be willing to help. I'm trying to come up to speed on the codebase to be able to submit a PR for #24 and #25. I'd be willing to help review PRs, etc if that were helpful. Would it make sense to move the repo to an organization to facilitate bringing on help? |
@RagingRoosevelt Thanks for the offer! For now any PR which can be readily merged would already be of much help. A good PR either has a test for the bug or feature, or a good explanation on the what and why if a test doesn't make sense for some reason. A changelog entry if it isn't something minor or internal. For this project I don't require documentation, but it would always be nice to improve that. Also helping with issues/PRs by commenting when there is information missing or how to make a PR mergable. If necessary it is easy to add you as a collaborator on this repo without an organization. Oh and if you know how, it would be great to move to GitHub Actions for tests, otherwise I will do it whenever I get to it. Just for reference: I did it recently for https://github.com/devpi/devpi/, but the setup is quite a bit more complex than what we need here, but the required bits and pieces are in there. |
Finally added that PR I promised! Still l need to do the other. |
@fschulze I replaced malfunctioning Travis CI with GitHub Actions in #28. They pass in my fork https://github.com/abitrolly/schemadisplay but |
In #29 Python 3 tests fail, but work fine on Python 2.7. Looking into how to fix them. |
Test are fixed in #29. Should be good to go now. |
I merged and fixed #30 to make the release process easier. |
@fschulze thanks! I've just I started to walk down the slippery road of vendoring dependencies, and now I can relax and chill a bit. :D It is safe to get prereleases to PyPI -
|
Vendored and released cmdline wrapper as https://pypi.org/project/sqlite2png/ |
I took over the project and did a new releases:
|
@Zlopez thanks a lot. That was a needed step to not lose the progress on DB diagramming. |
Am a bit worries with PR's and issues...
see #21
The text was updated successfully, but these errors were encountered: