|
| 1 | +# Contributing |
| 2 | + |
| 3 | +`Testcontainers for Go` is open source, and we love to receive contributions from our community — you! |
| 4 | + |
| 5 | +There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests, or writing code for the core library or for a technology module. |
| 6 | + |
| 7 | +In any case, if you like the project, please star the project on [GitHub](https://github.com/testcontainers/testcontainers-go/stargazers) and help spread the word :) |
| 8 | +Also join our [Slack workspace](http://slack.testcontainers.org) to get help, share your ideas, and chat with the community. |
| 9 | + |
| 10 | +## Questions |
| 11 | + |
| 12 | +GitHub is reserved for bug reports and feature requests; it is not the place for general questions. |
| 13 | +If you have a question or an unconfirmed bug, please visit our [Slack workspace](https://testcontainers.slack.com/); |
| 14 | +feedback and ideas are always welcome. |
| 15 | + |
| 16 | +## Code contributions |
| 17 | + |
| 18 | +If you have a bug fix or new feature that you would like to contribute, please find or open an [issue](https://github.com/testcontainers/testcontainers-go/issues) first. |
| 19 | +It's important to talk about what you would like to do, as there may already be someone working on it, |
| 20 | +or there may be context to be aware of before implementing the change. |
| 21 | + |
| 22 | +Next would be to fork the repository and make your changes in a feature branch. **Please do not commit changes to the `main` branch**, |
| 23 | +otherwise we won't be able to contribute to your changes directly in the PR. |
| 24 | + |
| 25 | +### Submitting your changes |
| 26 | + |
| 27 | +Please just be sure to: |
| 28 | + |
| 29 | +- follow the style, naming and structure conventions of the rest of the project. |
| 30 | +- make commits atomic and easy to merge. |
| 31 | +- use [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) for the PR title. This will help us to understand the nature of the changes, and to generate the changelog after all the commits in the PR are squashed. |
| 32 | + - Please use the `feat!`, `chore!`, `fix!`... types for breaking changes, as these categories are considered as `breaking change` in the changelog. Please use the `!` to denote a breaking change. |
| 33 | + - Please use the `security` type for security fixes, as these categories are considered as `security` in the changelog. |
| 34 | + - Please use the `feat` type for new features, as these categories are considered as `feature` in the changelog. |
| 35 | + - Please use the `fix` type for bug fixes, as these categories are considered as `bug` in the changelog. |
| 36 | + - Please use the `docs` type for documentation updates, as these categories are considered as `documentation` in the changelog. |
| 37 | + - Please use the `chore` type for housekeeping commits, including `build`, `ci`, `style`, `refactor`, `test`, `perf` and so on, as these categories are considered as `chore` in the changelog. |
| 38 | + - Please use the `deps` type for dependency updates, as these categories are considered as `dependencies` in the changelog. |
| 39 | + |
| 40 | +!!!important |
| 41 | +There is a GitHub Actions workflow that will check if your PR title follows the conventional commits convention. If not, it contributes a failed check to your PR. |
| 42 | +To know more about the conventions, please refer to the [workflow file](https://github.com/testcontainers/testcontainers-go/blob/main/.github/workflows/conventions.yml). |
| 43 | + |
| 44 | +- use [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) for your commit messages, as it improves the readability of the commit history, and the review process. Please follow the above conventions for the PR title. |
| 45 | +- unless necessary, please try to **avoid pushing --force** to the published branch you submitted a PR from, as it makes it harder to review the changes from a given previous state. |
| 46 | +- apply format running `make lint-all`. It will run `golangci-lint` for the core and modules with the configuration set in the root directory of the project. Please be aware that the lint stage on CI could fail if this is not done. |
| 47 | + - For linting just the modules: `make -C modules lint-modules` |
| 48 | + - For linting just the examples: `make -C examples lint-examples` |
| 49 | + - For linting just the modulegen: `make -C modulegen lint` |
| 50 | +- verify all tests are passing. Build and test the project with `make test-all` to do this. |
| 51 | + _ For a given module or example, go to the module or example directory and run `make test`. |
| 52 | + _ If you find an `ld warning` message on MacOS, you can ignore it. It is indeed a warning: https://github.com/golang/go/issues/61229 |
| 53 | + |
| 54 | + > === Errors |
| 55 | + > ld: warning: '/private/var/folders/3y/8hbf585d4yl6f8j5yzqx6wz80000gn/T/go-link-2319589277/000018.o' has malformed LC_DYSYMTAB, expected 98 undefined symbols to start at index 1626, found 95 undefined symbols starting at index 1626 |
| 56 | +
|
| 57 | +- when updating the `go.mod` file, please run `make tidy-all` to ensure all modules are updated. |
| 58 | + |
| 59 | +## Documentation contributions |
| 60 | + |
| 61 | +The _Testcontainers for Go_ documentation is a static site built with [MkDocs](https://www.mkdocs.org/). |
| 62 | +We use the [Material for MkDocs](https://squidfunk.github.io/mkdocs-material/) theme, which offers a number of useful extensions to MkDocs. |
| 63 | + |
| 64 | +We publish our documentation using Netlify. |
| 65 | + |
| 66 | +### Adding code snippets |
| 67 | + |
| 68 | +To include code snippets in the documentation, we use the [codeinclude plugin](https://github.com/rnorth/mkdocs-codeinclude-plugin), which uses the following syntax: |
| 69 | + |
| 70 | +> <!--codeinclude--><br/> > [Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression<br/> > [Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression<br/> > <!--/codeinclude--><br/> |
| 71 | +
|
| 72 | +Where each title snippet in the same `codeinclude` block would represent a new tab |
| 73 | +in the snippet, and each `targeting_expression` would be: |
| 74 | + |
| 75 | +- `block:someString` or |
| 76 | +- `inside_block:someString` |
| 77 | + |
| 78 | +Please refer to the [codeinclude plugin documentation](https://github.com/rnorth/mkdocs-codeinclude-plugin) for more information. |
| 79 | + |
| 80 | +### Previewing rendered content |
| 81 | + |
| 82 | +From the root directory of the repository, you can use the following command to build and serve the documentation locally: |
| 83 | + |
| 84 | +```shell |
| 85 | +make serve-docs |
| 86 | +``` |
| 87 | + |
| 88 | +It will use a Docker container to install the required dependencies and start a local server at `http://localhost:8000`. |
| 89 | + |
| 90 | +Once finished, you can destroy the container with the following command: |
| 91 | + |
| 92 | +```shell |
| 93 | +make clean-docs |
| 94 | +``` |
| 95 | + |
| 96 | +### PR Preview deployments |
| 97 | + |
| 98 | +Note that documentation for pull requests will automatically be published by Netlify as 'deploy previews'. |
| 99 | +These deployment previews can be accessed via the `deploy/netlify` check that appears for each pull request. |
| 100 | + |
| 101 | +Please check the GitHub comment Netlify posts on the PR for the URL to the deployment preview. |
0 commit comments