News for hackers. Stuff that matters.
Re:New GitHub Profile Badges Beta by david_allison on Thursday June 09, 2022 @12:15PM (#31683086)
One of the achievements is 'YOLO': 'merged without a review'.

We really shouldn't be encouraging this.

Re:New GitHub Profile Badges Beta by joshmanders on Thursday June 09, 2022 @12:38PM (#31683397)
It's not exactly as bad as it sounds, there's been a few times I have merged a PR from someone where I reviewed it, I just never went through the "Review this PR" -> "Approve PR" workflow that adds the meta data "joshmanders has approved this PR"
Re:New GitHub Profile Badges Beta by crazysim on Thursday June 09, 2022 @12:36PM (#31683373)
It's like one of those anti-achievements. Like the infamous "The exact opposite of your job" achievement.
Re:New GitHub Profile Badges Beta by lloydatkinson on Thursday June 09, 2022 @06:27PM (#31687762)
I don’t generally do reviews when I merge a PR into a branch of my own projects
Re:New GitHub Profile Badges Beta by datavirtue on Thursday June 09, 2022 @06:58PM (#31688030)
Gets hired because he has 4k YOLO badges. Agile.
Re:New GitHub Profile Badges Beta by O__________O on Thursday June 09, 2022 @12:26PM (#31683232)
More than likely, this is a red flag, not an achievement; or at least one would hope it is.
Re:New GitHub Profile Badges Beta by drakonka on Thursday June 09, 2022 @12:56PM (#31683662)
I don't know how it could serve as any meaningful flag. It's normal to merge to main without any review in your own solo hobby projects, and I kind of doubt this badge goes so far as to exclude those kinds of repos.
Re:New GitHub Profile Badges Beta by Beltalowda on Thursday June 09, 2022 @03:52PM (#31685937)
I almost never "review" PRs people send on my repos through the GitHub UI. I "review" them in the sense that I look at them, and comment when needed of course.

This "achievement" is not indicative of anything at all.

Re:New GitHub Profile Badges Beta by bastardoperator on Thursday June 09, 2022 @12:40PM (#31683419)
Only if you can't imagine working on a project by yourself...
Re:New GitHub Profile Badges Beta by throwaway3neu94 on Thursday June 09, 2022 @12:58PM (#31683701)
Depends on the criteria. Do automated reviews count?

I set up build+test via GH actions and have GitHub run them on PR's. And make all changes through PR's.

This not only prevents "forgot to run tests" type accidents during regular development, it also helps me when I come back to a hobby project months later, and it lets me confidently do small changes (think version bumps) directly through GitHub's editor, which saves a bunch of time.

And when, against the odds, I actually do get a contribution, the infra is already there and it really cuts down my turnaround time to get it merged, bump the version and deploy the release.

Of course if a single direct push yields you the badge then I guess pretty much every solo dev gets it sooner or later. But that indirectly turns it back into a good badge, right? "Independent enough to have a life outside a standardized corporate process"?

Re:New GitHub Profile Badges Beta by bastardoperator on Thursday June 09, 2022 @01:37PM (#31684274)
Actions can still provide the feedback even without a PR if you trigger on push, you just don't have a gate like PR for contribution. I do similar to what you're describing and would recommend a feedback loop for build/test 100% of the time.

I just don't think merging without review equates is an absolute negative depending on the situation, in this case no reviewers exist outside of myself and from my perspective this is how a lot projects start, someone just goes for it and starts committing! Yolo...

Re:New GitHub Profile Badges Beta by ep103 on Thursday June 09, 2022 @05:15PM (#31686996)
Where is this list of github achievements? The only place I see this mentioned is here
Re:New GitHub Profile Badges Beta by david_allison on Thursday June 09, 2022 @07:11PM (#31688135)
Re:New GitHub Profile Badges Beta by mshockwave on Thursday June 09, 2022 @12:21PM (#31683161)
yeah this achievement is really immature, maybe same as the one who came up with this in GH :-P