#(Darwin) why the hell is this valid?

36 messages · Page 1 of 1 (latest)

austere ibex
#

I just found huge resource leak caused by this nonsense

accidentally spotted ~20 runlaters of block scanning running every 15s

#changelog message can I nuke this?

hardy swiftBOT
#

(Darwin) why the hell is this valid?

hardy swiftBOT
#

Hi I'm AutoThreadBot! Don't mind me, I'll just be adding the helper team to this thread so they can see it. A human will get to you soon.

austere ibex
#

😡 any random crap would be true

#

It's a rhetorical question

#

i just hate it

olive oyster
#

If you want that any positive number returns true, you can check for its truthiness.

#

Not sure why a broken tag should return true?

austere ibex
#

it should return FALSE

#

but it's not

austere ibex
olive oyster
#

Oh I see

austere ibex
#

Idk, I should close this thread I think
It's not a bug, it's just how if works right now

azure aspen
#

thaats weird cause for me it indeed is returning false, actually any random shit is returning false

austere ibex
#

did you tried it?

azure aspen
#

ye

austere ibex
#
test2:
  type: task
  debug: false
  script:
  - if 234324534:
    - narrate done
azure aspen
#

wait im testing on outdated server, will test on another one

austere ibex
#

(line 27) if flase
If command passed, running block.

azure aspen
#

oh right its borky on newer version

austere ibex
#

🤯

austere ibex
#

Somebody of the devs should confirm is this a bug or not

#

If your build is older than this update #changelog message, than this isn't a bug

gilded geode
#

this isn't a bug, the post you linked is why

#

your tag evaluated to flase, which is truthy

azure aspen
#

well but could be breaking change since it has now different behaviour

gilded geode
#

only a breaking change if your if statements were not already evaluating to true/false, which would be spamming errors

#

true/false used to be required, now it's truthy/falsey, and reverting would break now-valid scripts

#

if anything, it could be a config option defaulting to truthy behavior. but i don't see that as a huge need

ashen tuskBOT
#
Thread Closing Reminder

Has your issue been resolved, or your question been answered?
If so, please use the </resolved:1028673926114594866> command to close your thread.
Or </invalid:1028673926898909185> if it's not possible to resolve.

If not yet resolved, please reply below to tell us what you still need.

(Note that if there is no reply for a few days, this thread will eventually close itself.)

#

@austere ibex

gilded geode
#

(also scripts written with truthy ifs would not be shareable if it was optional, which seems awkward to handle)

austere ibex
#

😔