Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README update suggestion #976

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Haoyu-Gao
Copy link

Description

We have made some updates to the project’s README file, please take a look at the pull request and tell me about your thoughts on it.
About
This is part of a research project that aims to study whether a templated presentation of README file could help improve the comprehension of the users and reduce the practitioners’ maintenance efforts for README file. The template image is attached as follows.
template-section.pdf
This pull request restructures the README file to improve readability and navigation, making it easier for users to find essential information quickly. The changes focus on:

  • Reorganizing sections for logical flow
  • Enhancing headers for better section clarity
  • Streamlining content to highlight key information
    These updates aim to support new users and contributors by presenting a more accessible and informative README. A templated update also aims to remind the documentation maintainers of essential information to include when performing README updates.

Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are errors here, for example in the prerequisites section.
Please do a manual review before another PR.

@thomasht86
Copy link
Collaborator

Hi!
Thanks for your suggestion.
I think we can add some of the suggestions, but some also are not correct/helpful.
Did you use an LLM to generate it? If so, it would be nice to get a heads up, and would appreciate if you would do a human review before creating a PR.

-Thomas

@Haoyu-Gao
Copy link
Author

Hi Thomas,

Thank you for your prompt reply, and I am sorry that there are incorrect information within the updated version.

Yes, the workflow for us is applying the template on the previous version of your README to LLM (ChatGPT-4o), and manually checking the generated content for its consistency. However, due to my lack of knowledge in this project and sloppiness, some errors still exist in the new version.

May I ask what are the things you think would be good to add, while what are the incorrect/ not helpful content. I am happy to make further adjustment on that.

Best regards,
Haoyu

@Haoyu-Gao Haoyu-Gao requested a review from thomasht86 November 22, 2024 02:29
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