Conversation
|
Nice idea! I can see how this could be useful, but I believe it would be to the benefit of the users' freedom if we could deploy the option as a standalone, much just like the current Also, with the current overriding design, in order for the ISO date to be actually displayed both the Please feel free to share your thoughts on this approach : ) |
| red, | ||
| underline, | ||
| yellow | ||
| } = chalk; |
There was a problem hiding this comment.
Unneeded changes, like this one, should also be removed for the PR to be merged
| "displayScope": true, | ||
| "displayBadge": true, | ||
| "displayDate": false, | ||
| "displayISODate": false, |
| - Type: `Boolean` | ||
| - Default: `false` | ||
|
|
||
| Override `displayDate` format and show the current local date in ISO 8601 `YYYY-MM-DDTHH:mm:ss.TDZ` format. |
There was a problem hiding this comment.
Display the current local date in ISO 8601 `YYYY-MM-DDTHH:mm:ss.TDZ` format.|
I cen help out with these too if u need! Happy to conribute! |
This PR add a little functionality to override currently data format used from local to ISO as optional, so turning the flag
displayISODateto true makes the timestamp to be in ISO 8601 format.