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

added to the README file #254

Closed
wants to merge 1 commit into from
Closed

Conversation

priya2004k
Copy link

What kind of change does this PR introduce?

This PR changes the README file by adding a table of contents, description, purpose, and contribution guidelines section to the file.

eg: Bug fix, feature, docs update, ...

Why was this change needed?

The change was needed to help make the ReadMe file better and to abide by good documentation guidelines.

Please link to related issues when possible.

Other information:

eg: Did you discuss this change with anybody before working on it (not required, but can be a good idea for bigger changes). Any plans for the future, etc?

Copy link

vercel bot commented Sep 24, 2024

@priya2004k is attempting to deploy a commit to the Listinai Team on Vercel.

A member of the Team first needs to authorize it.

@jamesread
Copy link
Collaborator

The contributing section is typically in contribututing.md, I also requested a PR on this, see #242.

@jamesread
Copy link
Collaborator

Hey @priya2004k , thanks for this contribution, but on further review, it kind of messes the formatting of README.md too much, and @nevo-david put a lot of effort into that. We're not going to merge this PR this time.

Next time, consider raising in issue first to dicuss what changes you think might be necessary before making the changes.

@jamesread jamesread closed this Oct 4, 2024
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