Conversation
pascalre
commented
May 6, 2026
- Updated variable declarations from var to let or const in feed-validate.ts, publish.ts, receive.ts, reply.ts, request.ts, semp-connection.ts, chalkUtils.ts, checkparams.ts, config.ts, defaults.ts, init.ts, listfeeds.ts, logger.ts, parse.ts, prettify.ts, shorthash.ts, visualize.ts.
- This change improves code readability and maintains block scope for variables.
…ross multiple files - Updated variable declarations from var to let or const in feed-validate.ts, publish.ts, receive.ts, reply.ts, request.ts, semp-connection.ts, chalkUtils.ts, checkparams.ts, config.ts, defaults.ts, init.ts, listfeeds.ts, logger.ts, parse.ts, prettify.ts, shorthash.ts, visualize.ts. - This change improves code readability and maintains block scope for variables.
gvensan
left a comment
There was a problem hiding this comment.
Thanks for taking this up.
|
The src/ changes look good to me - I verified the whole Two things before we merge:
Minor: a few src/ files (config.ts, prettify.ts, shorthash.ts) have trailing |