gun icon indicating copy to clipboard operation
gun copied to clipboard

LEX improvement

Open orimay opened this issue 3 years ago • 3 comments

Must be ES5-compatible. Passes 'match text' test ;)

orimay avatar Feb 05 '22 11:02 orimay

@Orimay 🔥 it looks right, the LEX tests are passing. Deleting my lines of code is the best gold a JSgolfer like me can receive :D :D pure outgamed bliss, I bow to you, this is Objective Truth, Scientific Beauty, and makes my heart sing.

.certify( keeps timing out, usually timeout errors are a problem with the VMs the test runners are on, but .certify( heavily uses LEX so we still may have more digging. You've proven this can work tho. Even if its not a silly thing, it'll make sense to update certify to comply with your simpler/cleaner approach. Do they pass on your machine, or think we need to do another call?

amark avatar Feb 19 '22 07:02 amark

Which test would that be?

orimay avatar Feb 19 '22 18:02 orimay

@Orimay https://github.com/amark/gun/blob/master/test/sea/sea.js#L566-L701 .

I hope you're talents & brain are in a healthy place!

amark avatar Feb 27 '22 12:02 amark