FYI: notify: rename JsonUpdater to JsonNotifier

Robin Westberg rwestberg at openjdk.java.net
Wed Jun 10 08:43:38 UTC 2020


On Wed, 10 Jun 2020 08:32:02 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this small refactoring that renames `JsonUpdater` to `JsonNotifier`.
> 
> Testing:
> - `make test` passes on Linux x64
> 
> Thanks,
> Erik

Looks good! We could also consider deleting this notifier as there are no consumers of the output, but it doesn't hurt
anything either..

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

Marked as reviewed by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/647


More information about the skara-dev mailing list