-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add database client to PyPi #69
Comments
Help me understand this a bit more. You want to publish a copy of the Mesh spreadsheet client to PyPi? Is that for another project to use? IMO if it's going to get reused in multiple places it should probably have its own repo so you can do git submodules instead |
Yep, I'm using it for my analysis repo. Brian asked me to share with Marco, and I figured not making him clone multiple repos would be a nicer developer experience. |
Yeah, makes sense. I think we should still put it in it's own repo though,
feels weird to publish a subcomponent of this repo to PyPi
…On Sat, Feb 11, 2023, 23:08 Andy Baumgartner ***@***.***> wrote:
Yep, I'm using it for my analysis repo. Brian asked me to share with
Marco, and I figured not making him clone multiple repos would be nicer.
—
Reply to this email directly, view it on GitHub
<#69 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAX6K6LJKTB2L5FWPJHOVATWXBO25ANCNFSM6AAAAAAUY7G5TE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Could be, but I'd rather not have to update the PyPi every time I make an edit for Supportbot. The analysis script is just a extra thing. |
If you're going to upload it to PyPi, we should probably keep it up to
date? Otherwise just send Marco a zip
…On Sat, Feb 11, 2023, 23:12 Andy Baumgartner ***@***.***> wrote:
Could be, but I'd rather not have to update the PyPi every time I make an
edit for Supportbot. The analysis script is just a extra thing.
—
Reply to this email directly, view it on GitHub
<#69 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAX6K6IRET6AXFHQWCMLEE3WXBPI7ANCNFSM6AAAAAAUY7G5TE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
That's a lot more steps. I don't see any problem with it being out of date on PyPi. Zip seems much more messy. |
I can put it in it's own repo, but that seems unproductive. It will go away with the Mesh API anyways. |
I can keep the PyPi up to date if it makes you happy. I'd rather not waste time moving it to it's own repo though. There are lots of other issues I could be working on instead. |
Yeah don't worry about keeping it up-to-date I guess. I still think you should just send Marco a ZIP, but I guess uploading the ZIP to PyPi isn't that much more work |
There is actually an official package that does all the uploading. You run |
No description provided.
The text was updated successfully, but these errors were encountered: