This bounty is no longer available
Web3 DAO | Optimism Logo

Use unit (ms, s) when variable refers to time

Organization

Optimism

Deadline

N/A

Status

ENDED

INSTRUCTIONS

Let's heed @snario's suggestion and switch over the variables to include the unit. Great first issue for anyone who wants to contribute!

Something I've found useful in the past is to add the unit as a suffix to the variable name. e.g.,

  • MAX_BATCH_SUBMISSION_TIME_S
  • POLL_INTERVAL_MS

Originally posted by @snario in https://github.com/ethereum-optimism/batch-submitter/pull/7#r585682849