-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support for automatic uploading to pypi #9
base: master
Are you sure you want to change the base?
Conversation
@denesb do you have pypi account ? so I can add you as an owner for the package ? |
I don't. I can create one if it is as simple as registering to a
website. Send me a link if it is so.
…On Tue, 2019-08-06 at 12:19 -0700, Israel Fruchter wrote:
@denesb do you have pypi account ? so I can add you as an owner for
the package ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
[
{
***@***.***": "http://schema.org",
***@***.***": "EmailMessage",
"potentialAction": {
***@***.***": "ViewAction",
"target": "
#9?email_source=notifications\u0026email_token=AAKTFWO3DWPFPIFBBIRY5X3QDHFDBA5CNFSM4IJZOLXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WF35I#issuecomment-518807029
",
"url": "
#9?email_source=notifications\u0026email_token=AAKTFWO3DWPFPIFBBIRY5X3QDHFDBA5CNFSM4IJZOLXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WF35I#issuecomment-518807029
",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
***@***.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]
|
https://pypi.org/account/register/
On Wed, Aug 7, 2019 at 10:06 AM Botond Dénes <[email protected]>
wrote:
… I don't. I can create one if it is as simple as registering to a
website. Send me a link if it is so.
On Tue, 2019-08-06 at 12:19 -0700, Israel Fruchter wrote:
> @denesb do you have pypi account ? so I can add you as an owner for
> the package ?
>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
> [
> {
> ***@***.***": "http://schema.org",
> ***@***.***": "EmailMessage",
> "potentialAction": {
> ***@***.***": "ViewAction",
> "target": "
>
#9?email_source=notifications\u0026email_token=AAKTFWO3DWPFPIFBBIRY5X3QDHFDBA5CNFSM4IJZOLXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WF35I#issuecomment-518807029
> ",
> "url": "
>
#9?email_source=notifications\u0026email_token=AAKTFWO3DWPFPIFBBIRY5X3QDHFDBA5CNFSM4IJZOLXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WF35I#issuecomment-518807029
> ",
> "name": "View Pull Request"
> },
> "description": "View this Pull Request on GitHub",
> "publisher": {
> ***@***.***": "Organization",
> "name": "GitHub",
> "url": "https://github.com"
> }
> }
> ]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9?email_source=notifications&email_token=AACTH42PN4KOSZ56JI3JXA3QDJX5XA5CNFSM4IJZOLXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3XNRCQ#issuecomment-518969482>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AACTH47EUUEWK2YPIRGOCWLQDJX5XANCNFSM4IJZOLXA>
.
|
Done. Username: bdenes, Email: dns.botond at gmail dot com. |
README.md
Outdated
pip install -r requirement-dev.txt | ||
pre-commit install | ||
|
||
# now each commit you do would fixed by black |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was me trying to explain yelp's pre-commit, with small success 😭
I'll remove this line
description='Scylla SStable Tools', | ||
packages=['sstable_tools'], | ||
scripts=['sstable-compressioninfo.py', 'sstable-index.py', 'sstable-statistics.py', 'sstable-summary.py'], | ||
name="sstable_tools", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these changes done by some automatic formatter?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes,
I was using black
https://github.com/psf/black
it was recently adopted by the Python Foundation
.pre-commit-config.yaml
Outdated
# hooks: | ||
# - id: pylint | ||
# name: pylint27 | ||
# language_version: python2.7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove commented code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to also add pylint, but lint is a bit more work, we could do that later, I'll remove those
There seems to be another PR (#8) that also adds a licence. |
I'll close #8 ( I was using github tool, that automatically suggest a PR... it's nice but not enough) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wish the formatting changes were in a separate commit, but I won't insist.
@denesb when we can configure travis ci, release a version is just tagging on master:
|
need to push a tag into master and a version would be uploaded to pypi
need to enable travis ci for it to work