I had a pull request against an open source project accepted after 10 years once, so I guess... that long.
More realistically, I guess the answer would be "be persuasive and produce overall improvements to the quality of life of the people who you're asking to review your work". You have 17 open Pull Requests, and some of your pull requests in the past have introduced bugs with changes which weren't adding user-value. You're asking others to do some hard thinking and prioritize reviewing your code over whatever else they're doing. That's a fair thing to ask for, but message like this can really hurt your case. When it comes down to it, this is a persuasion game - even for me and I'm the employer of some of the people in place to review code, I still need to make a persuasive case that it's worth their time to review my work! And find a good time, and ask nicely, and maybe ping them again rather than just assuming they'll come back and check among all the other pull requests to review mine. It's a chilling read to look at https://newsletter.goodtechthings.com/p/the-threat-to-open-source-comes-from and see how things like "your process is broken" can mess people up. I get your frustration but also, man this is not a good way to encourage any of the maintainers to give higher priority to your pull requests rather their own work on either their employer's priorities or things which are high priority in the project's significant-sized refactoring goals. Regards, Bron. On Thu, Feb 13, 2025, at 22:25, Дилян Палаузов wrote: > Hello, > > the Cyrus IMAP Development Process is described on two different places in > two different ways: > > At > https://www.cyrusimap.org/overview/cyrus_bylaws.html#iv-development-process > is stated that “The submitter of the code is responsible for ensuring that > the code gets reviewed by the core developers.” Submitters of changes have > absolutely no means to ensure that code gets reviewed, so they cannot be > responsible for this. > > The procedure at https://www.cyrusimap.org/imap/developer/process.html > suggests these steps: Create a pull request, wait for review (a quick note to > the mailing list can speed this along). > > In October 2024 I published > https://github.com/cyrusimap/cyrus-imapd/pull/5075 , which enhances the > CalDAV Admin HTML page ( http://server/dav/calendars/user/localpart@domain/ ) > by adding one more property (toggle to enable the server-side scheduling for > a calendar). This feature can be currently anyway applied by issuing > undocumented PROPPATCH requests. On 4 December I asked at > devel@cyrus.topicbox.com if that change can be integrated. > > So far nothing happened, which suggests that the suggested process: “Create a > pull request, then wait for review (a quick note to the mailing list can > speed this along)” is not working. > > A clearer criteria whether the procedures described at > https://www.cyrusimap.org/imap/developer/process.html do function, would be > to announce also how long one has to wait. Two years, more than three years? > > Greetings > Дилян -- Bron Gondwana, CEO, Fastmail Pty Ltd br...@fastmailteam.com ------------------------------------------ Cyrus: Devel Permalink: https://cyrus.topicbox.com/groups/devel/T4b8a7a5d201419b6-M4e19eb2a8dec9854e2f78118 Delivery options: https://cyrus.topicbox.com/groups/devel/subscription
