Skip to content

Conversation

@gab-dev-7
Copy link
Contributor

Just so you can see that, yes i actually ran it locally and all builds/tests passed (apart from some dataraces, but those were there already; 35 Warnings), without adding anything else.

I created another PR just bc I've already put my time into this. This time it worked editing the filed with 'sed -i'. I didn't use AI, but I won't try to convince you.

I actually went and asked gemini now why that could happen and it told me about CRLF vc LF, but hey, idk. Have a good one :)

Image Image Image

@Morcules
Copy link
Owner

Thank you!!
I appreciate your help.

If you really didn’t use AI, I’m sorry. For the past week I’ve been receiving only AI PRs and it’s kinda frustrating.
It looked like an AI, because of changing ptr location, etc.

Are you using a formatter or some program that might cause this issue?

@Morcules Morcules marked this pull request as ready for review January 16, 2026 16:02
@Morcules Morcules merged commit 113100c into Morcules:main Jan 16, 2026
@Morcules
Copy link
Owner

Screenshot 2026-01-16 at 17 05 23 This looks very AI generated. Either way have a nice day 🤍

@gab-dev-7
Copy link
Contributor Author

Hey, I can imagine that receiving AI PRs are really annoying.

I use nvim to edit all my files and for c i have the "clang-format" plugin. I attached a video where it shows how the code in your image behaves when i try to format the whole file.

If you also use nvim (or know how to solve this anyway) could you tell me what formatter you use or how can i prevent this from happening?

I press <leader>cf every 2 secs so i didnt even realized what had happened until you pointed it out ;)

2026-01-17.14-06-25.mp4

@Morcules
Copy link
Owner

Hello. I don’t use any formatter. I’m also using neovim. If you want, you can remove the keybind and just use the formatter via a command so it doesn’t cause any issues when contributing to other projects.

Sorry for accusing you earlier, and thank you again for contributing. I really appreciate it 😃

@gab-dev-7
Copy link
Contributor Author

good to know. will do!

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

Successfully merging this pull request may close these issues.

2 participants