-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
PEP 581, 588: Split GitHub migration rationale and plan into two separate PEPs #1013
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
Conversation
:robot_face: Mariatta was mentioned, but she's out of open source from March 18th to May 9th, 2019 . Be aware she might not get to this until after PyCon US. |
people who aren't already familiar with the code base. | ||
There is an open discussion about moving the source code of bpo to | ||
GitHub [#]_. If the source code of bpo does move to GitHub, it will | ||
become difficult to update patches from upstream. But as long as it |
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.
This isn't a huge problem. Roundup has recently added builtin REST interface. We can start to port our GitHub integration to use it. I don't think we are relaying on our fork's customizations too much. As always, I'm volunteering to do this work.
|
||
Issues with Roundup / bpo | ||
------------------------- | ||
|
||
- Less than five people maintain bpo. Some of them are core developers. | ||
|
||
- It is in Mercurial. Without any CI available, it puts heavy burden on the few | ||
existing maintainers in terms of reviewing, testing, and applying patches. | ||
- The upstream Roundup code is in Mercurial. Without any CI available, |
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.
Personally, I don't see how this is a problem here. pip install
supports installing from Mercurial repositories and its testing suite isn't that hard to run. Plus, as I said below, our Roundup instance can easily be moved to GitHub.
upstream. But as long as it is in Mercurial, it is difficult to maintain | ||
and onboard new contributors. | ||
- In its current state, the project is not equipped to accept lots of | ||
contributions from people who aren't already familiar with the 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 find the most complicated part of the installation process is installation of our Roundup fork. Once we get rid of our fork, it won't be harder than installing python.org.
I'd share my INSTALL
document here, but unfortunately I left it in my computer in my hometown. But you can take a look at the Docker image at https://github.com/python/docker-bpo. It can be simplified by using Docker Compose.
|
||
- The user interface needs an update and redesign. It will require UX/UI research | ||
to keep it up to date with current web standards, including accessibility. | ||
|
||
- Email address is exposed. There is no choice to mask it. | ||
- Users email addresses are exposed. There is no option to mask it. |
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.
Roundup can be modified to mask usernames and/or email addresses. See https://issues.roundup-tracker.org/issue2551034 for example: Usernames masked as [hidden]
there.
contributors and maintainers, while not having to jump from one interface to another. | ||
By using GitHub issues, where the CPython source code is currently | ||
hosted and where pull requests are taking place, we'll be providing | ||
consistent experience to contributors and maintainers, while not |
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.
We are not the only project that uses a different issue tracker while using GitHub to host its source code. Some projects (SQLAlchemy) even use GitLab for issue tracking.
Barry told me to post my email in python-committers here. I'll try to summarize it. I'm strongly against using GitHub Issues because of the following reasons:
|
@berkerpeksag can I request you tweak the tone of your comment? It's coming off as unnecessarily hostile. |
@brettcannon which part of my comment do you think is hostile? |
@berkerpeksag primarily your first bullet point. While you might think GitHub doesn't listen, I don't think that's true but you state it like it's a universal truth. And your last sentence in that point very much comes off as a dig as well. And just because they don't announce features before they launch doesn't mean they don't listen (I say the same things to my users at my work because I know if I over-promise and under-deliver I'm the one they will yell at, so it's easier to just not say anything until something is already released). |
@brettcannon I gave two examples from my own experience that they didn't listen (both issues can be classified as bug) in the second bullet point. I'd be happy to combine the first two bullet points if you think that would make my comment better.
True, but none of my reports were actually feature requests and I definitely don't expect constant communication from them. I don't think it's unreasonable to expect a response like "we are going to fix this in X months" a year after I reported it in a place where they were created to gather feedback from the community. |
Sorry you had terrible experience, but I think you're generalizing the situation. Personally I've had good experiences with GitHub. Whenever I had issues and questions, they answered within 24 hours. So I think saying "GitHub doesn't listen" doesn't sound fair, or maybe I'm really lucky.
I feel these issues are not related to GitHub-as-issue-tracker, and not the problems that PEP 581 trying to solve, so it seems like a distraction to current topic. I also don't feel the last one as a bug. If miss-islington actually did the work to backport and create the PR, it can earn the credit IMO. The others problems you mentioned here, we've tried to help solve by miss-islington's automerge. If core devs still chose not to use the automerge feature, then I don't know what to say to that...
You're definitely free to choose how you spend your free time. Roundup itself is not going away. It's not officially maintained by Python core developers, although several core developers like yourself maintain it. (thank you!!). I can understand if you're not interested in maintaining other workarounds like bedevere, miss-islington. However these projects are on GitHub and open source, and we already have quite a number of contributions to them from non core developers, so I don't feel there will be an issue.
Personally, I think that's the purpose of the APIs, and I actually think it is great that GitHub has a set of nice features, and we can still further customize the experience by building our own tools by utilizing their APIs. |
<https://discuss.python.org/t/proposal-create-bug-triage-team-on-github/992/5>`_ | ||
for us to create a "bug triage" team on GitHub to help with closing | ||
issues, notifying the appropriate parties, as well as applying labels | ||
to issues and pull requests. We can grant the "write" permission to |
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.
This should be updated to reflect that Github now has a triage level permission https://discuss.python.org/t/pep-581-using-github-issues/535/36?u=ammaraskar
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.
It's a private beta though.
No description provided.