Skip to content
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

Всем видны pull requests #4

Open
NekR opened this issue Oct 1, 2013 · 4 comments
Open

Всем видны pull requests #4

NekR opened this issue Oct 1, 2013 · 4 comments

Comments

@NekR
Copy link

NekR commented Oct 1, 2013

Я и думаю другие пользователи видят пулл реквесты других пользователей.

Например кто то провёл оптимизации и отправил пул реквест, до него ещё несколько человек отправили пул реквесты с похожими оптимизациями, тогда человека который отправил позже, можно обвинить в плагиате/копировании.

Самые зловредные люди просто пройдутся по всем пул реквестам и скопируют все оптимизации. Если конкурс был на скорость, то об этом нужно было писать в правила.

@effrenus
Copy link

effrenus commented Oct 1, 2013

Форки все видят тоже.

@NekR
Copy link
Author

NekR commented Oct 1, 2013

Ну тем более, хотя форк я могу сам в приват спрятать раз такое дело. Например просто прочитав коммент к вашему пулл реквесту можно найти половину оптимизаций, потом пойти в соседний и добавить ещё тех, которых у вас нет.

А так хотелось поучавствовать в конкурсе.

@kbakba
Copy link
Contributor

kbakba commented Oct 1, 2013

Здравствуйте, уважаемые участники!

Мы понимаем ваши опасения, и мы подумали об этом перед началом конкурса.
Не все предложенные оптимизации положительно влияют на скорость отрисовки страницы. Так что взять все pull request'ы и объединить их в один не всегда будет полезным, да и сделать это не просто.
Мы будем следить за участниками, и если обнаружим неэтичное копирование чужих pull request'ов, то эти участники будут дисквалифицированы (все существующие pull request'ы отклонены, будущие не будут приниматься).

Все ваши pull request'ы обрабатывает робот, так что комментарии к pull-request'ам, разъясняющие суть ваших оптимизаций можно не оставлять.

@NekR
Copy link
Author

NekR commented Oct 1, 2013

Здравствуйте.

На мой взгляд всё равно получается странно. Например, я провёл некоторое количество оптимизаций, что то забыл, что то не досмотрел. А тут вижу, кто то уже послал пул реквест, просто тыкаю на него, даже не смотря изменения, по комментариям к нему понимаю, что я не досмотрел там то и там то. Иду себе, правлю, по своему, возможно даже лучше чем тот благодаря кому я заметил эту проблему. В конечно итого получается, что как бы я и сделал всё сам, но как бы и я подсмотрел у того человека, а ему будет обидно, если например я выиграю, а он нет.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants