From: "Tzvetomir Stoyanov (VMware)" <tz.stoyanov@gmail.com>
To: linux-trace-devel@vger.kernel.org
Subject: [PATCH] trace-cruncher: Enable Github development flow
Date: Wed, 15 Jun 2022 18:15:58 +0300 [thread overview]
Message-ID: <20220615151558.226641-1-tz.stoyanov@gmail.com> (raw)
Github development flow is more popular among python developers. The
CONTRIBUTING file is updated to enable both mailing list based (current
one) and PR based (github) development flows. The main motivation for
this change in to attract more contributors.
Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
---
CONTRIBUTING.md | 26 +++++++++++++++-----------
1 file changed, 15 insertions(+), 11 deletions(-)
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index 865fcc8..c22780b 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -5,21 +5,25 @@
The trace-cruncher project team welcomes contributions from the community. All contributions to this repository must be
signed. Your signature certifies that you wrote the patch or have the right to pass it on as an open-source patch.
-The development process of trace-cruncher is strongly coupled to the development of the Linux kernel tracing libraries [libtraceevent](https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git), [libtracefs](https://git.kernel.org/pub/scm/libs/libtrace/libtracefs.git/) and [KernelShark](https://git.kernel.org/pub/scm/utils/trace-cmd/kernel-shark.git/). Because of this reason we adopt and follow the development workflow established by those projects.
+### Submitting patches
-For contributions to development, please send patches to: linux-trace-devel@vger.kernel.org
+Patches can be submitted by either:
+ 1. Using the regular [Github Flow](https://docs.github.com/en/get-started/quickstart/github-flow):
+ - The main branch is **tracecruncher**.
+ - Any changes must be on a feature branch or on a fork.
+ - Tests must pass before merging, and the pull request must be [reviewed](https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests) and approved.
+ - Break the complex Pull Requests into small self-contained patches.
-[Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
+ 2. Using the [Linux kernel](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html) development workflow:
+ - Add prefix "trace-cruncher:" to the patch subject.
+ - Break the changes into small self-contained patches and group them in a patch set.
+ - Send patches to linux-trace-devel@vger.kernel.org
+ - [Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
### Code Style
The preferred coding style for the project is the [Linux kernel coding style](https://www.kernel.org/doc/html/v4.10/process/coding-style.html#linux-kernel-coding-style)
-### Formatting Commit Messages
-
-The project follows the conventions for [submitting patches](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html)
-as described by the Linux kernel.
-
### Tests
Make sure that all your changes are covered by the tests. Before submitting your patch, check if everything works at 100% by running the tests in **tracecruncher/tests**. Compile your changes and install trace-cruncher (`sudo make install`), to make sure that your code is used in the tests. As trace-cruncher interacts with the Linux kernel tracing infrastructure, the tests must be run with root privileges:
@@ -30,6 +34,6 @@ sudo python3 -m unittest discover .
```
## Reporting Bugs and Creating Issues
-For bug reports and issues, please file it [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---)
-
-When opening a new issue, try to roughly follow the commit message format conventions above.
+Bug reports and issues can be submitted by either:
+1. Create a [Github issue](https://docs.github.com/es/issues/tracking-your-work-with-issues/creating-an-issue) in [trace-cruncher](https://github.com/vmware/trace-cruncher).
+2. Report them in [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---), **Tools and utilities** category, **Trace-cmd and kernelshark** component.
--
2.35.1
next reply other threads:[~2022-06-15 15:16 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-15 15:15 Tzvetomir Stoyanov (VMware) [this message]
2022-06-15 15:45 ` [PATCH] trace-cruncher: Enable Github development flow John 'Warthog9' Hawley
2022-06-15 16:42 ` Yordan Karadzhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220615151558.226641-1-tz.stoyanov@gmail.com \
--to=tz.stoyanov@gmail.com \
--cc=linux-trace-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).