8 ms·
> I try to assist inexperienced contributors and coach them to the finish line, because getting a PR accepted is an achievement to be proud of I really appreci
by andruby 1y ago
> I try to assist inexperienced contributors and coach them to the finish line, because getting a PR accepted is an achievement to be proud of
I really appreciate this point from mitchellh. Giving thoughtful constructive feedback to help a junior developer improve is a gift. Yet it would be a waste of time if the PR submitter is just going to pass it to an AI without learning from it.
- ants_everywhere 1y agoJunior developers are entering a workforce where they will never not be using AI
- eru 1y agoI don't think using AI at all is forbidden, he just doesn't want AI to do the whole PR?
- ants_everywhere 1y agoThe requirement is explicitly > If you are using *any kind of AI assistance* to contribute to Ghostty, it must be disclosed in the pull request. This is sufficiently confusing that someone is asking if this applies to tab completion. They commit actually says > trivial tab-completion doesn't need to be disclosed, so long as it is limited to single keywords or short phrases. So if you take this literally you're going to be disclosing every yasnippet expansion that completes boilerplate. The policy as written isn't sensible and I don't think it's entirely coming from a sensible place. Junior developers need to learn how to code with AI because that's what coding is now. Not that he has to help them. But it does read a bit weird to toot your horn about how important it is to be helpful until it comes to helping people understand how to navigate the current environment then it's not worth your time.
- 0x6c6f6c 1y agoDisclosing doesn't mean it will be declined, it's just a signal for reviewers.
- rerdavies 1y agoThe question is going to be: is it a USEFUL signal. I suspect not. And frankly, to be honest, as a senior developer who uses AI assistants routinely, I would consider it a serious disincentive to actually submit a PR in the first place to a repository that has such a policy. Submitting a good PR is hard work. Often an order of magnitude more work than the fix itself. If I think that a repository is going to not accept my PR just because I use a coding assistant, I'm going to be much less inclined to submit a PR at all.
- aleph_minus_one 1y ago> Junior developers need to learn how to code with AI because that's what coding is now. Rather: this is what coding is now in some Silicon Valley bubbles.
- makeitdouble 1y agoThe rules can be finely adjusted when it actually becomes problematic, they're not trying to pass a law through Congress.
- hagbarth 1y agoThey will still need to learn to recognise if the output from AI is good or not.
- tpoacher 1y agoYes, in the same way junior pilots are entering a workforce where they will never not be using an autopilot.
- aleph_minus_one 1y ago> Junior developers are entering a workforce where they will never not be using AI This remark seems very US-centric to me. In my observation, many people are much more skeptical concerning whether AI is actually useful beyond some gimmicky applications.
- MacsHeadroom 1y agoHalf the planet is more AI-positive than the US. Something like only 4% of Chinese people are concerned about AI while around half of Americans are concerned about AI.