You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2024/02/10 13:23:33 UTC
(camel) branch main updated: (chores) doc: minor contributions clarifications
This is an automated email from the ASF dual-hosted git repository.
orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push:
new 1e397eee00f (chores) doc: minor contributions clarifications
1e397eee00f is described below
commit 1e397eee00f7c859d2cb0dff268cba8066b372ba
Author: Otavio R. Piske <an...@gmail.com>
AuthorDate: Sat Feb 10 11:27:32 2024 +0100
(chores) doc: minor contributions clarifications
- about build changes
- about missing assertions
Signed-off-by: Otavio R. Piske <an...@gmail.com>
---
docs/main/modules/contributing/pages/index.adoc | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/docs/main/modules/contributing/pages/index.adoc b/docs/main/modules/contributing/pages/index.adoc
index 6b5740b434d..fb6deb5c913 100644
--- a/docs/main/modules/contributing/pages/index.adoc
+++ b/docs/main/modules/contributing/pages/index.adoc
@@ -237,8 +237,11 @@ CAMEL-9999: Some message goes here
=== Making a good contribution
+* Less is more:
+** Avoid creating unnecessary Maven profiles (i.e.; to enable/disable tests)
+** Avoid creating custom surefire/failsafe configurations (use defaults as much as possible)
* Ensure that the unit tests include proper assertions.
-* Avoid simply outputting changes to the standard output/error or just logging.
+* Avoid simply outputting changes to the standard output/error or just logging (tests **must** have assertions).
* Please also avoid unnecessary changes, like reordering methods and fields, which will make your PR harder to review.
* When submitting a performance improvement, providing JMH test data as evidence or adding a JMH-based test on the https://github.com/apache/camel-performance-tests/[camel-performance-tests] repository is strongly recommended.
* Be responsive, assume good intent and respect the https://www.apache.org/foundation/policies/conduct.html[Code of Conduct]