@taylorotwell going "will this code bring me joy" all over your PR.
-
-
-
Ten tweet jest niedostępny.
- Pokaż odpowiedzi
Nowa rozmowa -
-
-
I could've sold my feature and maintained it. But I were simply ignored. Dont play this game any more.
-
That’s not how it works though. There is no way to hold you responsible for maintaining that feature. Maybe you do, but if you don’t, then what? Then it falls on the maintainer.
Koniec rozmowy
Nowa rozmowa -
-
-
As someone with a few "sold" PRs, I think laravel maintainers are amongst the most welcoming maintainers in the whole open source community. Others may not even give any feedback, and some of them are not dying nor non profit projects at all.
-
Ten tweet jest niedostępny.
- Pokaż odpowiedzi
Nowa rozmowa -
-
-
But sometimes i really don't understand why turn down some PRs that fixes something, for example the PR that fixes guessing the extension of an UploadedFile
-
It’s not clear it fixed anything. It’s an opinion on how things should work and anything around file uploads needs ultra care around security. What they wanted to do could be handled in their own code. Just my two cents.
- Pokaż odpowiedzi
Nowa rozmowa -
-
-
The maintainer argument could be made for any code, not just PR. As long as the PR solves a problem (with no competing solutions), I don't see why it should be rejected (unless you have a specific reason). If you reject just for the sake of it, you are probably Linus Torvalds!
Dziękujemy. Twitter skorzysta z tych informacji, aby Twoja oś czasu bardziej Ci odpowiadała. CofnijCofnij
-
-
-
well said.
Dziękujemy. Twitter skorzysta z tych informacji, aby Twoja oś czasu bardziej Ci odpowiadała. CofnijCofnij
-
Wydaje się, że ładowanie zajmuje dużo czasu.
Twitter jest przeciążony lub wystąpił chwilowy problem. Spróbuj ponownie lub sprawdź status Twittera, aby uzyskać więcej informacji.
