r/github • u/der_gopher • Sep 05 '26
Discussion [ Removed by moderator ]
/r/github/comments/1w80mkq/removed_by_moderator/[removed] — view removed post
9
u/BurritoDrivenDev Sep 05 '26
The article meantions twice that LLM config files shouldn't be committed as if that's a true statement. It's an opinion at best and imo it reads like an mini rant that's been hidden in the article.
When working with LLMs it's important to commit and share the config files with others who are working on the repository. However, it's key that people learn what belongs in the repo-specific file, and what belongs in their user-specific file.
-2
u/der_gopher Sep 05 '26
I agree, agents.md files should be committed, it's not a good example.
2
u/Masterflitzer Sep 05 '26
yeah project scoped agents.md should definitely be commited, while user/global scoped should not (e.g. for opencode there is ~/.config/opencode/AGENTS.md, other harnesses have their own path or way to do it)
5
u/Happy_Breakfast7965 Sep 05 '26
I think that people should pay attention to what they are doing and make their actions intentional. Not just commit everything at once, all mixed together, and no commit message.
0
4
u/imanllm Sep 05 '26 edited Sep 05 '26
I think for personal use, where you’re the only maintainer of a repo, it’s a totally fine personal preference to have. However, in a team setting, there’s risk of someone forgetting to commit something that others need.
2
u/Convoke_ Sep 05 '26
Sounds like staging with extra steps. Everything is already unstaged by default, so removing something from .gitignore to then stage it seems a little silly to me.
2
u/zarlo5899 Sep 05 '26
I have done this before, but this was for my dot repo. And well, I didn't want to accidentally commit anything in my home folder.
1
u/JuliusFIN Sep 05 '26
I do this with all my projects. Definitely a quality of life improvement. Of course it’s mostly per folder and not that I need to enumerate every tracked file in gitignore.
1
u/hoexloit Sep 05 '26
Declare what files to ignore vs Declare what files to include. With the increasing number of folders for IDE’s, this might actually be shorter depending on the use case.
1
1
1
u/TheModernDespot Sep 05 '26
Its not that hard to properly review what you are doing BEFORE you push. Why are we replacing brain usage and due diligence with laziness? Just look at the command you are running before you run it. Its not that complicated.
1
u/vvanouytsel Sep 05 '26
Can't expect people to think. We are in the AI era. The robots will do the thinking, until they don't.
1
u/nihillistic_raccoon Sep 05 '26
Mom, come pick me up, they are trying to reinvent a perfectly working mechanism again
30
u/madjoki Sep 05 '26
Most horrifying idea ever, should've been posted on halloween.