checkstyle / checkstyle/checkstyle

Add FileContents to AuditEvent

Open
#8,968 2 comments 0 reactions 0 assignees View on GitHub
discussion ongoing
Dominant language
Java
Stars
9.6k
Forks
4.2k
Avg merge
22h 23m
Merged PRs (30d)
232

Description

Currently, `FileContents` is created in each `AbstractFileSetCheck` descendant. This is redundant, since the file should not change and the result is always the same. This should be done once in the `Checker.processFiles` method. By doing this, we not only speed up the process, but also provide access to the file contents for `AuditListeners`.

_Originally posted by @pbludov in https://github.com/checkstyle/checkstyle/issues/8807#issuecomment-723415882_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.