search

aergoio / aergo

aergo blockchain kernel
http://docs.aergo.io
MIT License
214 stars 45 forks source link

brick: fix redefined flag #214

Closed kroggen closed 1 year ago

kroggen commented 1 year ago

Fix error "redefined flag: v" on brick

kroggen commented 1 year ago

@hayarobi

I guess this started happening when the master and develop branches were merged

The v flag is already being used in the log module, so it gives the error "flag redefined: v" when we run brick

I just renamed the flag to b on brick

Do you have any other idea?

hayarobi commented 1 year ago

How about using capital V? Anyway, the other parts look ok.

kroggen commented 1 year ago

Good idea! I changed it to V

As you can see, both the full and short workflows were triggered with this new commit. Only the full one should run. I made a fix to this in another PR