Skip to content

Create CONTRIBUTING.md #239

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Create CONTRIBUTING.md #239

wants to merge 6 commits into from

Conversation

starlightknown
Copy link
Contributor

ISS-177444

@starlightknown starlightknown requested a review from a team as a code owner May 27, 2025 06:35
Copy link
Contributor


Any contribution you make can be made to the code located in this repository. In order to contribute, you'll need to start off of a local version of this repository.

#### 1. Fork the repository
Copy link
Contributor

Choose a reason for hiding this comment

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

@nimit2801 and others have had troubles deploying changes from forks. Has it been confirmed that forking works?

Also, H4s are typically not a good idea. Headings are not required here--a numbered list is adequate.


### Create a new issue

If you spot a problem within a repository, [search if an issue already exists](https://docs.github.com/en/github/searching-for-information-on-github/searching-on-github/searching-issues-and-pull-requests#search-by-the-title-body-or-comments). If a related issue doesn't exist, you can open a new issue here!Please make sure any added issues are
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there supposed to be a link in here? This type of construction is not recommended. https://learn.microsoft.com/en-us/style-guide/urls-web-addresses#grammar-and-formatting


Thank you for investing your time in contributing to DevRev's documentation. Any contribution you make will be reviewed by our team. In this guide, you'll learn the different ways you can contribute.

### Create a new issue
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd recommend getting rid of this heading and the next one. Rather than highlighting the two possibilities it buries them.

> _For help, support, feature requests, and product questions - head to our [DevRev's Discord Community](https://discord.gg/c6JuJAT4Aa)🤖_

Thank you for investing your time in contributing to DevRev's documentation. Any contribution you make will be reviewed by our team. In this guide, you'll learn the different ways you can contribute.

Copy link
Contributor

Choose a reason for hiding this comment

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

It would be a good idea to include a TOC for the page since it's not rendered on a web site.

starlightknown and others added 4 commits May 28, 2025 14:07
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Co-authored-by: Ben Colborn <[email protected]>
Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

🌟 EkLine Reviewer

Hello! I’m here to help improve your docs. I’ve reviewed your pull request, and left in-line suggestions for quick fixes. For details, visit the Analytics Page.

For questions or feedback, please email [email protected].

Copy link
Contributor

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.

3 participants