Skip to content

Conversation

xrchz
Copy link
Contributor

@xrchz xrchz commented Sep 17, 2022

Issues Addressed

Closes #3101

Proposed Changes

Add global flag to suppress timestamps in the terminal logger.

## Issues Addressed

Closes sigp#3101

## Proposed Changes

Add global flag to suppress timestamps in the terminal logger.
@michaelsproul michaelsproul added the ready-for-review The code is ready for review label Sep 17, 2022
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

Happy to merge this once the dust for v3.1.1 has settled. I'll mark it as blocked until then.

I've also opened a related issue about testing flags like this, but no pressure to take that on: #3588

@michaelsproul michaelsproul added blocked ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review blocked labels Sep 19, 2022
@michaelsproul
Copy link
Member

We decided not to release v3.1.1 due to a bug (cf. #3600), so I'm unblocking this for inclusion in v3.1.2

@michaelsproul michaelsproul added the v3.1.2 Release after v3.1.0 (formerly v3.1.1) label Sep 23, 2022
@paulhauner
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Sep 23, 2022
## Issues Addressed

Closes #3101

## Proposed Changes

Add global flag to suppress timestamps in the terminal logger.
@bors bors bot changed the title Add disable-log-timestamp flag (#3101) [Merged by Bors] - Add disable-log-timestamp flag (#3101) Sep 23, 2022
@bors bors bot closed this Sep 23, 2022
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Issues Addressed

Closes sigp#3101

## Proposed Changes

Add global flag to suppress timestamps in the terminal logger.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to merge. v3.1.2 Release after v3.1.0 (formerly v3.1.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants