Skip to content

[Backport 8.x] Move contributing instructions to CONTRIBUTING.md #4179

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

Merged
merged 1 commit into from
Mar 28, 2025
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
19 changes: 1 addition & 18 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,18 +1 @@
<!--

Hello there!

Thank you for opening a pull request!
Please make sure to follow the steps below when opening a pr:

- Sign the CLA https://www.elastic.co/contributor-agreement/
- Tag the relative issue (if any) and give a brief explanation on what your changes are doing
- If you did a spec change, remember to generate again the outputs, you can do it by running `make contrib`
- Add the appropriate backport labels. If you need to backport a breaking change (e.g. changing the structure of a type or changing the type/optionality of a field), please follow these rules:
- If the API is unusable without the change -> every supported version
- If the API is usable, but fix is on the response side -> every supported version
- If the API is usable, but fix is on the request side -> no backport, unless the API is _partially_ usable and the fix unlocks a missing feature that has no workaround

Happy coding!

-->
<!-- Hello there! Thank you for opening a pull request. See CONTRIBUTING.md for instructions. -->
60 changes: 60 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,60 @@
# Contributing to the Elasticsearch specification

Hello, and thank you for contributing!

There are many ways to contribute, including opening new issues, improving the docs or fixing typos. However, the instructions below focus on changes to the specification itself.

## Prepare the environment

For generating the JSON representation and running the validation code you need
to install and configure Node.js in your development environment.

You can install Node.js with [`nvm`](https://github.com/nvm-sh/nvm):

```sh
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.40.1/install.sh | bash
```

Once the installation is completed, install Node.js with `nvm`:

```sh
# this command will install the version configured in .nvmrc
nvm install
```

From here, you can install the dependencies with `make setup`:

```sh
make setup
```

## Make your change

Documents under the docs/ directory will help you to change the specification. The most relevant ones are:

* [How to add a new API](docs/add-new-api.md)
* [Specification structure](docs/specification-structure.md)
* [Modeling Guide](docs/modeling-guide.md)
* [Documenting the API specification](docs/doc-comments-guide.md)
* [Fixing a definition, the complete story](docs/validation-example.md)


## Send your pull request

To fix any style issues and generate the outputs (that we store in git), please run:

```sh
make contrib
```

Then, follow those steps:

- Sign the CLA https://www.elastic.co/contributor-agreement/
- Tag the relative issue (if any)
- Explain what your changes are doing
- Add the appropriate backport labels. If you need to backport a breaking change (e.g. changing the structure of a type or changing the type/optionality of a field), please follow these rules:
- If the API is unusable without the change -> every supported version
- If the API is usable, but fix is on the response side -> every supported version
- If the API is usable, but fix is on the request side -> no backport, unless the API is _partially_ usable and the fix unlocks a missing feature that has no workaround


4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,10 @@ $ make overlay-docs
# The generated output can be found in ./output/openapi/
```

## Contributing

See [CONTRIBUTING.md](./CONTRIBUTING.md)

## Make Targets

```
Expand Down