Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 13 additions & 16 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
# Contributing to `fetch`` <!-- omit from toc -->
# Contributing to `fetch` <!-- omit from toc -->

First off, thank you for taking the time to contribute to `fetch`! :+1: :tada:
`fetch` is released under the Apache 2.0 license. If you would like to
contribute something or want to hack on the code, this document should help you
get started. You can find some hints for starting development in `fetch`'s
First off, thank you for taking the time to contribute to fetch! :+1: :tada:
fetch is released under the Apache 2.0 license. If you would like to contribute
something or want to hack on the code, this document should help you get
started. You can find some hints for starting development in fetch's
[README](https://github.com/StacklokLabs/fetch/blob/main/README.md).

## Table of contents <!-- omit from toc -->
Expand All @@ -14,7 +14,6 @@ get started. You can find some hints for starting development in `fetch`'s
- [Using GitHub Issues](#using-github-issues)
- [Not sure how to start contributing?](#not-sure-how-to-start-contributing)
- [Pull request process](#pull-request-process)
- [Contributing to docs](#contributing-to-docs)
- [Commit message guidelines](#commit-message-guidelines)

## Code of conduct
Expand All @@ -37,8 +36,8 @@ security vulnerabilities using GitHub issues; instead, please follow this
### Using GitHub Issues

We use GitHub issues to track bugs and enhancements. If you have a general usage
question, please ask in
[fetch's discussion forum](https://discord.gg/stacklok).
question, please ask in the #mcp-servers channel of the
[Stacklok Discord server](https://discord.gg/stacklok).

If you are reporting a bug, please help to speed up problem diagnosis by
providing as much information as possible. Ideally, that would include a small
Expand All @@ -51,7 +50,10 @@ PRs to resolve existing issues are greatly appreciated and issues labeled as
are a great place to start!

### Pull request process
-All commits must include a Signed-off-by trailer at the end of each commit message to indicate that the contributor agrees to the Developer Certificate of Origin. For additional details, check out the [DCO instructions](dco.md).

-All commits must include a Signed-off-by trailer at the end of each commit
message to indicate that the contributor agrees to the Developer Certificate of
Origin. For additional details, check out the [DCO instructions](dco.md).

- Create an issue outlining the fix or feature.
- Fork the fetch repository to your own GitHub account and clone it locally.
Expand All @@ -61,15 +63,10 @@ are a great place to start!
- Open a PR by ensuring the title and its description reflect the content of the
PR.
- Ensure that CI passes, if it fails, fix the failures.
- Every pull request requires a review from the core fetch team before
merging.
- Every pull request requires a review from the core fetch team before merging.
- Once approved, all of your commits will be squashed into a single commit with
your PR title.

### Contributing to docs

TBD

### Commit message guidelines

We follow the commit formatting recommendations found on
Expand All @@ -80,4 +77,4 @@ We follow the commit formatting recommendations found on
1. Capitalize the subject line
1. Do not end the subject line with a period
1. Use the imperative mood in the subject line
1. Use the body to explain what and why vs. how
1. Use the body to explain what and why vs. how
Loading