Skip to content

Update README.md#1825

Open
Nik-Kras wants to merge 1 commit intoabetlen:mainfrom
Nik-Kras:patch-2
Open

Update README.md#1825
Nik-Kras wants to merge 1 commit intoabetlen:mainfrom
Nik-Kras:patch-2

Conversation

@Nik-Kras
Copy link
Copy Markdown

@Nik-Kras Nik-Kras commented Nov 8, 2024

No description provided.

@ExtReMLapin
Copy link
Copy Markdown
Contributor

Average 'I can't code but I want to be in the contributors list' PR

@richdougherty
Copy link
Copy Markdown
Contributor

richdougherty commented Nov 24, 2024

Hi @Nik-Kras, I think it's nice to keep the text clean. My suggestion is that I would probably go lighter on the changes and just stick to obvious grammar errors (The model will will format), and not just minor style changes. This is because the changes add extra diffs to the code which can make history harder to trace. If you stick to fixes it will also make it easier for the maintainer to merge.

I'm reminded of this first PR on a project I worked on, which got a bit famous with a blog post: playframework/playframework#616. Hopefully that will encourage you.

@devashishraj
Copy link
Copy Markdown

readme > llm > PR

@jroper
Copy link
Copy Markdown

jroper commented Sep 19, 2025

@ExtReMLapin This kind of hostility to new contributors to a project is never appropriate.

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.

5 participants