You must log in or register to comment.
You made this?
Fork
I made this
Now everone expects you to be the maintainer. You get a lot of bug reports.
Then you ghost them and wait for the next sucker to fork your fork.
LOOK AT ME. I’M THE MAINTAINER NOW.
I’ve been there, but over the years I’ve gotten better at avoiding being in this situation.
If you are implementing something for yourself, and merging it back upstream is just a bonus, then by all means jump straight to implementing.
However, it’s emotionally draining to implement something and arrive at something you’re proud of only to have it ignored. So do that legwork upfront. File a feature request, open a discussion, join their dev chat - whatever it is, make sure what you want to do is valued and will be welcomed into the project before you start on it. They might even nudge you in a direction that you hadn’t considered before you started.
Be a responsible dev and communicate before you do the work.
The last 2 frames should be the same text
You should edit it and post a pull request.
You’re right. Oops
The repo owner is lost in the mountains and will accept your request after they get out of the hospital and find a new job and apartment in two years. :)
Only after a fork of the original project takes off and due to some new dependencies no longer supports the suggested feature.
Somebody please fork reprepro, there’s a super useful bugfix in one and a super useful feature in the other but I want both.
The bugfix is the zstd decompression-cancel race condition bug and the feature is multiple versions per package but they’re both super stale.
Maybe…
Maybe I can fork it…Fork the feature one, get the diff of the commit that patches the bug and apply the diff to your fork.
Now compile and test.
I don’t have a lot of experience with merging but then again, this is a great opportunity to learn.
This is why I always ask if the maintainers are open to a PR first.
Yeah…but even then they may not get to you.
Over the holidays, I had a good back and forth with the maintainer of a project that I started using. The documentation needed updating and created a PR.
Then I went almost 10 comment rounds on why it was necessary, why I wrote it the way I did, and all this bullshit.
I just left it saying “merge it or whatever. I’m moving on.”
It’s still open.
And this is why I don’t contribute. Or at least I’ll ask a question about whether or not something would be a desired feature and if I don’t get a clear yes or no by someone who can actually approve a PR, I. ain’t. coding. shit.
This is how its supposed to be done anyway.
You’re framing this as if it were something unusual. Unsolicited PRs are a no-go in my opinion. It’s just basic communication and collaboration to align with the maintainers whether a change is actually required or not.
Or worse, ban you from their GitHub repository without any reply or explanation
It is a bit sad that reviewing takes a long time. I have had the same thing for a project, someone on the team pings someone to do a review, 2 years later you get a review saying you should rebase because the PR is too old. I get why; it takes people and time to review. It is sad though.
I once made a PR changing
a = f(b)froma = "" if b == "" else f(b). About six months later still inpending review. I get that people have their own shit but damn.edit) Sorry. Clarified the code.
“closed by stalebot”
I did this, then it too so long that it broke. Still hasn’t been pulled.
If that’s happening to you that’s crazy. GitHub is way too noisy though. I get 30 notifications on that apps notification widget though for just bullshit I didn’t even know I signed up for or snyk or some other garbage.
Even better when someone makes the exact same PR and it gets merged a few days after being opened and yours left unreviewed.
