Skip to content

feat: add timestamps to maven build logs#750

Closed
0xReLogic wants to merge 1 commit intoobjectionary:masterfrom
0xReLogic:feature/add-timestamps-maven-logs
Closed

feat: add timestamps to maven build logs#750
0xReLogic wants to merge 1 commit intoobjectionary:masterfrom
0xReLogic:feature/add-timestamps-maven-logs

Conversation

@0xReLogic
Copy link
Copy Markdown

@0xReLogic 0xReLogic commented Oct 27, 2025

This PR adds timestamp formatting to Maven build logs to make it easier to track build performance and debug timing issues.

Changes

  • Added -Dorg.slf4j.simpleLogger.showDateTime=true to enable timestamp display
  • Added -Dorg.slf4j.simpleLogger.dateTimeFormat=yyyy-MM-dd HH:mm:ss to set timestamp format

Example Output

Before:

[INFO] Scanning for projects...
[INFO] BUILD SUCCESS
[INFO] Total time:  2.039 s

After:

2025-10-27 04:40:41 [INFO] Scanning for projects...
2025-10-27 04:40:43 [INFO] BUILD SUCCESS
2025-10-27 04:40:43 [INFO] Total time:  2.039 s
Screenshot 2025-10-27 124118

Testing

  • All existing tests pass
  • Manually verified timestamp output with eoc --verbose parse command

Fixes #94

This commit adds timestamp formatting to Maven build logs to make it easier to track build performance and debug timing issues.

The timestamps are added using slf4j-simple logger properties:
- showDateTime=true enables timestamp display
- dateTimeFormat=yyyy-MM-dd HH:mm:ss sets the format

Example output:
2025-10-27 04:40:41 [INFO] Scanning for projects...
2025-10-27 04:40:43 [INFO] BUILD SUCCESS
2025-10-27 04:40:43 [INFO] Total time:  2.039 s

Fixes #94
@0xReLogic
Copy link
Copy Markdown
Author

Closing this PR as I found there's already an existing PR #617 that addresses the same issue with tests included. Sorry for the duplicate!

@0xReLogic 0xReLogic closed this Oct 27, 2025
@0crat
Copy link
Copy Markdown

0crat commented Oct 27, 2025

@0xReLogic Hey! Just a heads up - the branch name feature/add-timestamps-maven-logs doesn't follow our naming convention, so that's -6 points deducted according to our policy. It's not a good idea to use descriptive names like this - instead, just use the ticket number you're working on. For this one, something like 749 would be perfect! Keep it simple next time 👍

@0xReLogic
Copy link
Copy Markdown
Author

@0crat Are you okey? 😕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add timestamps for maven logs

2 participants