Hoping this PR can make it into NB26, freeze in less than a month if I'm counting correctly. The PR is

   In Surround with try-catch hint add/default to System.Logger.
   https://github.com/apache/netbeans/pull/8253

In addition to the System.Logger option, logger related hints are fixed for System.Logger awareness. And there's a couple minor new features. AFAICT, the PR works as advertised.

To smooth the process, the review could be done in two parts.

1. Look at the PR's OP, and follow on post which has a few questions,
   to check the overall approach and features.
2. Regular code review.

After (1) the PR can be adjusted hoping to minimize the changes during (2).

In addition to the PR, there's the discussion that kicked things off

   consider System.Logger in related hints/code-generation
   https://github.com/apache/netbeans/issues/8240

Items in the discussion's OP that are included in the PR are annotated with "*included in #8253*". Refer to this if you want details on what's included in the PR.

Reply via email to