You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hahaha. Yeah, this is bad. The --topics config item is old and was never used until recently, so no-one scrutinised how ugly it is against the rest of the config items. I think --topic should be repeated argument (similar to almost all other repeated config items).
Problem
There's two ways to use topics flags:
--topics
- Takes a list of comma separated topics, CANNOT be repeated.--protected-topic
- Does NOT take a list, CAN be repeate.Makes no sense.
Impact
Almost made me vomit.
Expected behavior
Not inducing nauseousness. Probably just make
--topics
into--topic
and allow it to be repeated.Additional context
nwaku/apps/wakunode2/config.nim
Lines 210 to 213 in b277ce1
nwaku/apps/wakunode2/config.nim
Lines 40 to 43 in b277ce1
The text was updated successfully, but these errors were encountered: