Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tracecompass-dev] Fixes for simple warnings in Eclipse Problems view

On 08/13/2016 04:46 PM, Ruediger Gad wrote:
Hi,

I just set up a Trace Compass build environment by following the, imho,
very good developer docs.
So far i did not do much, except setting up the Eclipse development
environment and running the unit tests on the latest code from git
That's a good start.
However, I noticed a number of warnings reported in the Eclipse Problems
view that could, i think, be easily fixed

Do you accept patches for such rather trivial fixes?
Yes, we will accept them.
If yes, what would be the appropriate way to do this?
All code changes or additions will go through a review process using Gerrit. We'll provide comments and suggestions for each patch if needed and when they are addressed we will merge the patches to repository.
Simply proceed following the developer docs and just push a patch or
discuss this first, e.g., on the mailinglist?
Yes, documentation found on the Trace Compass website should have all the information to get you started with providing patches through Gerrit. See reference [1] for a page on how to contribute to Trace Compass.

For bugfixes and bigger contributions where some discussions on design and implementation are required then it's good to have a bug report on bugzilla for reference and clarification. For trivial fixes like warnings you don't need to write a bug report.

The mailing list and IRC is a good way to contact the Trace Compass developers and community. Don't hesitate to use them.

[1] https://wiki.eclipse.org/Trace_Compass/Contributor_Guidelines



BR,

Ruediger


BR
Bernd




Back to the top