From 892cb088aa1b80ebdf7ace5b11947848b04a658d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Manuel=20de=20la=20Pe=C3=B1a?= Date: Fri, 22 Nov 2024 16:13:37 +0100 Subject: [PATCH] docs: better contribution guidelines (#2893) * docs: better contribution guidelines * fix: wrong copy paste * chore: add whitespace * docs: explicitly mention the branch is published * docs: put lint stage first * docs: mention ld warning --- Makefile | 7 ++ docs/contributing.md | 105 +++++++++++++++++++++++++---- docs/contributing_docs.md | 46 ------------- mkdocs.yml | 4 +- modulegen/internal/mkdocs/types.go | 2 +- 5 files changed, 100 insertions(+), 64 deletions(-) delete mode 100644 docs/contributing_docs.md diff --git a/Makefile b/Makefile index 7c8c5e36b3..9c5a968ee7 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,12 @@ include ./commons-test.mk +.PHONY: lint-all +lint-all: + $(MAKE) lint + $(MAKE) -C modulegen lint + $(MAKE) -C examples lint-examples + $(MAKE) -C modules lint-modules + .PHONY: test-all test-all: tools test-tools test-unit diff --git a/docs/contributing.md b/docs/contributing.md index 822c113487..13364b6c83 100644 --- a/docs/contributing.md +++ b/docs/contributing.md @@ -1,16 +1,93 @@ # Contributing -* Star the project on [Github](https://github.com/testcontainers/testcontainers-go) and help spread the word :) -* Join our [Slack workspace](http://slack.testcontainers.org) -* [Post an issue](https://github.com/testcontainers/testcontainers-go/issues) if you find any bugs -* Contribute improvements or fixes using a [Pull Request](https://github.com/testcontainers/testcontainers-go/pulls). If you're going to contribute, thank you! Please just be sure to: - * discuss with the authors on an issue ticket prior to doing anything big. - * follow the style, naming and structure conventions of the rest of the project. - * make commits atomic and easy to merge. - * when updating documentation, please see [our guidance for documentation contributions](contributing_docs.md). - * when updating the `go.mod` file, please run `make tidy-all` to ensure all modules are updated. It will run `golangci-lint` with the configuration set in the root directory of the project. Please be aware that the lint stage could fail if this is not done. - * apply format running `make lint` - * For examples: `make -C examples lint` - * For modules: `make -C modules lint` - * verify all tests are passing. Build and test the project with `make test-all` to do this. - * For a given module or example, go to the module or example directory and run `make test`. +`Testcontainers for Go` is open source, and we love to receive contributions from our community — you! + +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. + +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 :) +Also join our [Slack workspace](http://slack.testcontainers.org) to get help, share your ideas, and chat with the community. + +## Questions + +GitHub is reserved for bug reports and feature requests; it is not the place for general questions. +If you have a question or an unconfirmed bug, please visit our [Slack workspace](https://testcontainers.slack.com/); +feedback and ideas are always welcome. + +## Code contributions + +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. +It's important to talk about what you would like to do, as there may already be someone working on it, +or there may be context to be aware of before implementing the change. + +Next would be to fork the repository and make your changes in a feature branch. **Please do not commit changes to the `main` branch**, +otherwise we won't be able to contribute to your changes directly in the PR. + +### Submitting your changes + +Please just be sure to: + +* follow the style, naming and structure conventions of the rest of the project. +* make commits atomic and easy to merge. +* 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. +* 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. +* 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. +* 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. + * For linting just the modules: `make -C modules lint-modules` + * For linting just the examples: `make -C examples lint-examples` + * For linting just the modulegen: `make -C modulegen lint` +* verify all tests are passing. Build and test the project with `make test-all` to do this. + * For a given module or example, go to the module or example directory and run `make test`. + * If you find an `ld warning` message on MacOS, you can ignore it. It is a indeed a warning: https://github.com/golang/go/issues/61229 +> === Errors +> 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 + +* when updating the `go.mod` file, please run `make tidy-all` to ensure all modules are updated. + +## Documentation contributions + +The _Testcontainers for Go_ documentation is a static site built with [MkDocs](https://www.mkdocs.org/). +We use the [Material for MkDocs](https://squidfunk.github.io/mkdocs-material/) theme, which offers a number of useful extensions to MkDocs. + +We publish our documentation using Netlify. + +### Adding code snippets + +To include code snippets in the documentation, we use the [codeinclude plugin](https://github.com/rnorth/mkdocs-codeinclude-plugin), which uses the following syntax: + +> <!--codeinclude-->
+> [Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression
+> [Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression
+> <!--/codeinclude-->
+ +Where each title snippet in the same `codeinclude` block would represent a new tab +in the snippet, and each `targeting_expression` would be: + +- `block:someString` or +- `inside_block:someString` + +Please refer to the [codeinclude plugin documentation](https://github.com/rnorth/mkdocs-codeinclude-plugin) for more information. + +### Previewing rendered content + +#### Using Python locally + +From the root directory of the repository, you can use the following command to build and serve the documentation locally: + +```shell +make serve-docs +``` + +It will use a Python's virtual environment to install the required dependencies and start a local server at `http://localhost:8000`. + +Once finished, you can destroy the virtual environment with the following command: + +```shell +make clean-docs +``` + +#### PR Preview deployments + +Note that documentation for pull requests will automatically be published by Netlify as 'deploy previews'. +These deployment previews can be accessed via the `deploy/netlify` check that appears for each pull request. + +Please check the Github comment Netlify posts on the PR for the URL to the deployment preview. diff --git a/docs/contributing_docs.md b/docs/contributing_docs.md deleted file mode 100644 index bfa2ba6a69..0000000000 --- a/docs/contributing_docs.md +++ /dev/null @@ -1,46 +0,0 @@ -# Contributing to documentation - -The _Testcontainers for Go_ documentation is a static site built with [MkDocs](https://www.mkdocs.org/). -We use the [Material for MkDocs](https://squidfunk.github.io/mkdocs-material/) theme, which offers a number of useful extensions to MkDocs. - -We publish our documentation using Netlify. - -## Adding code snippets - -To include code snippets in the documentation, we use the [codeinclude plugin](https://github.com/rnorth/mkdocs-codeinclude-plugin), which uses the following syntax: - -<!--codeinclude-->
-[Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression
-[Human readable title for snippet](./relative_path_to_example_code.go) targeting_expression
-<!--/codeinclude-->
- -Where each title snippet in the same `codeinclude` block would represent a new tab -in the snippet, and each `targeting_expression` would be: - -- `block:someString` or -- `inside_block:someString` - -Please refer to the [codeinclude plugin documentation](https://github.com/rnorth/mkdocs-codeinclude-plugin) for more information. - -## Previewing rendered content - -### Using Python locally - -From the root directory of the repository, you can use the following command to build and serve the documentation locally: - -```shell -make serve-docs -``` - -It will use a Python's virtual environment to install the required dependencies and start a local server at `http://localhost:8000`. - -Once finished, you can destroy the virtual environment with the following command: - -```shell -make clean-docs -``` - -### PR Preview deployments - -Note that documentation for pull requests will automatically be published by Netlify as 'deploy previews'. -These deployment previews can be accessed via the `deploy/netlify` check that appears for each pull request. diff --git a/mkdocs.yml b/mkdocs.yml index 2d80a5b420..f8decf74ed 100644 --- a/mkdocs.yml +++ b/mkdocs.yml @@ -134,9 +134,7 @@ nav: - system_requirements/using_colima.md - system_requirements/using_podman.md - system_requirements/rancher.md - - Contributing: - - contributing.md - - contributing_docs.md + - Contributing: contributing.md - Getting help: getting_help.md edit_uri: edit/main/docs/ extra: diff --git a/modulegen/internal/mkdocs/types.go b/modulegen/internal/mkdocs/types.go index d6145c3fd6..a1131d5b12 100644 --- a/modulegen/internal/mkdocs/types.go +++ b/modulegen/internal/mkdocs/types.go @@ -35,7 +35,7 @@ type Config struct { Examples []string `yaml:"Examples,omitempty"` Modules []string `yaml:"Modules,omitempty"` SystemRequirements []interface{} `yaml:"System Requirements,omitempty"` - Contributing []string `yaml:"Contributing,omitempty"` + Contributing string `yaml:"Contributing,omitempty"` GettingHelp string `yaml:"Getting help,omitempty"` } `yaml:"nav"` EditURI string `yaml:"edit_uri"`