Integrated: 8050: JMC Automated Analysis should improve the messages when ignoring to evaluate some rules.

Suchita Chaturvedi schaturvedi at openjdk.org
Thu May 18 13:43:00 UTC 2023


On Mon, 24 Apr 2023 00:37:35 GMT, Suchita Chaturvedi <schaturvedi at openjdk.org> wrote:

> One of our customers requested to see a detailed _Ignore Reason_ whenever we ignore any rule in JMC. Rather than just saying Ignored. As part of this PR, we have added a new field Ignored Reason which will be displayed for all the Ignored Rules. It will give a clear idea about the missing event availability requirement.
> 
> The Automated Analysis screen will look like this:
> ![image](https://user-images.githubusercontent.com/11155712/233875905-f70cf844-e9b3-405d-a9be-12a169abfaf1.png)
> 
> In the above screenshot, there are few rules which are Ignored and the corresponding reasons are mentioned.
> 
> Please review the PR and let me know your review comments.

This pull request has now been integrated.

Changeset: 3c705606
Author:    Suchita Chaturvedi <schaturvedi at openjdk.org>
URL:       https://git.openjdk.org/jmc/commit/3c7056066838e34e1a8ed1fd2071a652595197e1
Stats:     204 lines in 16 files changed: 178 ins; 2 del; 24 mod

8050: JMC Automated Analysis should improve the messages when ignoring to evaluate some rules.

Reviewed-by: hirt

-------------

PR: https://git.openjdk.org/jmc/pull/481


More information about the jmc-dev mailing list