kentonv changed the topic of #sandstorm to: Welcome to #sandstorm: home of all things sandstorm.io. Say hi! | Have a question but no one is here? Try asking in the discussion group: https://groups.google.com/group/sandstorm-dev
dustyweb has quit [Remote host closed the connection]
frigginglorious has joined #sandstorm
_whitelogger has joined #sandstorm
<isd>
Probably.
<isd>
Does this mean we're in for a whole month of this?
<JacobWeisz[m]>
I flagged those two PRs as invalid, which is a label Hacktoberfest recognizes as a maintainer indicating a spam PR.
<isd>
I reported them to GitHub; hadn't realized this thing existed at the time.
<isd>
Fortunately we're not quite as popular as the html spec; hopefully the workload won't get too bad
<isd>
But this is dumb.
<JacobWeisz[m]>
They are apparently working on fixing the nightmare they built according to some folks on Twitter.
<JacobWeisz[m]>
I think the problem with this and the Keybase crypto giveaway and stuff all comes down to overconfidence in automation. Security and moderation actually don't scale well.
<JacobWeisz[m]>
DO could fix this easily by having people submit requests to be considered for a shirt and having an employee review the PRs they submit, but of course, that's a lot of man-hours.
frigginglorious has quit [Ping timeout: 260 seconds]
<isd>
I think requiring the PRs to actually get merged would do a lot to discourage spam. But also, what's the point of all this? I think there are some baseline assumptions that need revisiting.
<simpson>
Pretty sure it started and stopped with "t-shirt cannons are fun"
wings has quit [*.net *.split]
wings has joined #sandstorm
wings has quit [*.net *.split]
wings has joined #sandstorm
wings has quit [Ping timeout: 244 seconds]
<JacobWeisz[m]>
Their response to not requiring merges is that some big projects don't use GitHub to merge, so for those, closed PRs may represent actual accepted contributions.
<TimMc>
Wow, gross. I pinged someone I know at DO and another at GH.
<TimMc>
JacobWeisz[m]: ...which would be so easy to fix with requiring that the label "accepted" be added in those cases, but they obviously want to put all the burden on maintainers rather than "contributors".
<isd>
You could also look at the commits that land, rather than the pr itself.
<isd>
Also, big projects are going to be the ones who suffer most due to the spam, which brings us back to "this whole thing is stupid and they should just not do it."
<TimMc>
From the Hacktoberfest discord, it sounds like if you mark something as invalid, it will get picked up within 2 hours by DO.
<TimMc>
So you could mark all new PRs with "invalid" along with a friendly "ignore the label" note, then unmark later.
<TimMc>
"Since we likely can’t change the behavior of these participants entirely, we are creating a shortcut for these participants to get their shirt and move on, without having to spam repositories on GitHub."
<isd>
fwiw the xorg page on EVoC doesn't cite spam/low quality submissions as a problem with GSoC. Though I have heard of that being a problem before.
<simpson>
EVoC avoids it by hand-selecting students and having a long and involved vetting process, as opposed to GSoC publishing a Google Docs form for anybody to fill out. But it's not perfect.