EngineHub / EngineHub/CommandHelper
Logging is not consistent
- Dominant language
- Java
- Stars
- 128
- Forks
- 70
- PR merge metrics
- No merged PRs in 30d
Description
CMDHELPER-3163 - Reported by PseudoKnight
We need to do a pass over the logging in CH to make it more consistent and work on various platforms correctly. (some platforms show [CommandHelper] twice in some locations, other times not at all) This means we need to use the plugin logger if available, and make it accessible from a singular location like Static. I propose Static.getLogger(). I'm not sure how CHLog would fit into this, though, as that seems to be more for warnings/errors. Maybe it should be adapted instead? We need to figure out what we want before fixing this.
Contributor guide
Research direction
Start by reading the logging paths around Static and CHLog, then trace where the plugin logger is available and where [CommandHelper] is added on each platform. Clarify whether CHLog should be adapted or replaced before changing callers. Done means logging has one consistent access point and produces consistent prefixes across supported platforms.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100