Vyxal - 2021-11-30

« first day (592 days earlier) ← previous day next day → last day (1183 days later) »

hyper-neutrino

00:18

💀

lyxal

01:16

y'all ever heard of lgtm.com?

it's actually quite good

rues

Nice

lyxal

it shows errors that we might miss

as well as security stuff

like it pointed out ctx was missing in some function calls

rues

Oh, I feel like flake8 should've caught that ಠ_ಠ

lyxal

user image

like that

rues

That looks really useful

lyxal

01:18

it is

but I don't think I'll be adding it as a workflow action though

because it's picking up on some things that aren't actually errors/have to be that way

rues

:( Then how do we use it? Is there a way to disable some of those errors?

lyxal

user image

see how it doesn't like the dynamic file creation in the flask app?

rues

Interesting, I wonder if there's a way for us to rewrite it so it accepts that?

lyxal

I'm just seeing if there's a way to share the dashboard with y'all

it's free btw

rues

I mean, we can all set up our own things if it's not shareable

lyxal

01:21

lgtm.com/projects/g/Vyxal/Vyxal/?mode=list

there you go

rues

pog

lyxal

it's openly accessible without an account

rues

We can look into Codefactor too in case that has less agressive error checking

lyxal

help.semmle.com/lgtm-enterprise/user/help/…

rues

Nice, suppression is pretty straightforward (py for the lazies like me)

lyxal

01:24

we should look into it a bit more before actually adding it as an application

rues

Just have to do # lgtm[py/errorname]

@lyxal Of course, but once we do, this'll be really helpful. Nice find

lyxal

CMQ: What do y'all think about adding lgtm as an automated code review for PRs and stuff?

!!/hyperping

Vyxal Bot

@AaronMiller @AaroneousMiller @Allxy @Ausername @Milk @PyGamer0 @Wansen @emanresuA @hyper-neutrino @math @pxeger @user ^

rues

!!/ping me

Vyxal Bot

@rues I have put you on the ping list.

lyxal

01:26

@rues I only found it because rak asked if APL was going to get LGTM support lol

rues

Ah

@lyxal LGTM :P

@lyxal Darn, some of these are actually pretty basic

lyxal

@rues I just pushed a commit onto the branch I'm working on to fix those

rues

01:46

Great

Vyxal Bot

cgccuser approved on PR #285 (Vyxal/Vyxal): "I could've sworn I approved this already."

rues

Yoo @lyxal ^ looks like a pretty minor and reasonable PR

lyxal

user image

That's why I haven't acted on it yet

Idk if it's finished

rues

02:01

Oh, missed that

Hmm, did we stop using VERSION at some point? If so, we could just accept the thing from main and be done with it

emanresu A

02:54

I didn'y know that :P gitcdn looks much better and uses the latest commit so :)

2 hours later…

PyGamer0

05:02

@lyxal yeah maybe probably

it shouts a lot, but i think we can ignore the shouts or something

2 hours later…

lyxal

06:56

@emanresuA I managed to get vectorised recursion working

I'll be opening a PR to add it to main soon

Vyxal Bot

07:12

Lyxal opened PR #301 (Vyxal/Vyxal) (Vyxal:recursion-vectorisation-and-fix-253 → Vyxal:main): Recursion vectorisation and fix 253

3 hours later…

Vyxal Bot

10:26

Lyxal approved on PR #285 (Vyxal/Vyxal)

chunkybanana approved on PR #301 (Vyxal/Vyxal)

1 hour later…

Vyxal Bot

11:34

pxeger approved on PR #301 (Vyxal/Vyxal): "eh, looks good enough to me"

pxeger closed issue #253 (Strings error on newlines with `D` flag, Vyxal/Vyxal)

pxeger merged PR #301 (Vyxal/Vyxal) (Vyxal:recursion-vectorisation-and-fix-253 → Vyxal:main): Recursion vectorisation and fix 253

pxeger merged PR #285 (Vyxal/Vyxal) (Vyxal:black_on_flask_app → Vyxal:main): run black on flask_app.py

pxeger deleted branch Vyxal/black_on_flask_app

pxeger deleted branch Vyxal/recursion-vectorisation-and-fix-253

pxeger opened PR #302 (Vyxal/Vyxal) (Vyxal:production → Vyxal:main): trick GitHub into thinking this branch is merged (even though it already is)

pxeger merged PR #302 (Vyxal/Vyxal) (Vyxal:production → Vyxal:main): trick GitHub into thinking this branch is merged (even though it already is)

pxeger

@VyxalBot such Pull Request. Many merge.

user image

Vyxal Bot

pxeger opened PR #303 (Vyxal/Vyxal) (Vyxal:main → Vyxal:production): Update production (30/11/21)

pxeger

11:50

@VyxalBot @lyxal @AaroneousMiller @emanresuA

Vyxal Bot

Lyxal approved on PR #303 (Vyxal/Vyxal): "LGTM"

3 hours later…

Aaroneous Miller

14:33

@user do you still need the function-names or more-generators branches for anything, or can they be deleted?

1 hour later…

Vyxal Bot

15:48

AMiller42 opened issue #304 in Vyxal/Vyxal: Stack isn't padded with zero

Aaroneous Miller

I don't know if it's an issue on my end or not, but I can't get the offline interpreter to work. I've tried with a file, with the e flag, in the REPL, with input, without input, and it always errors with TypeError: 'type' object is not subscriptable

Anyone else having issues?

Vyxal Bot

16:11

AMiller42 opened issue #305 in Vyxal/Vyxal: `°` doesn't do anything

AMiller42 opened issue #4 in Vyxal/Vylight: Comment highlighting never stops

Aaroneous Miller

16:40

@lyxal is there a reason that we have two different commands that both do 2 ** a?

2 hours later…

Aaroneous Miller

18:32

Only 2 days left to crack my Vyxal cop!

2 hours later…

Vyxal Bot

20:38

chunkybanana approved on PR #303 (Vyxal/Vyxal)

2 hours later…

lyxal

22:34

@VyxalBot status-not-implemented-yet

So the input isn't a string... that's useful. (* has different behaviour on str,str in 2.4 & 2.6) — emanresu A 2 hours ago

Dang it does?

I forgor 💀

lyxal

23:21

!!/run `abc``def`*

Vyxal Bot

@lyxal def

lyxal

W...

I honestly don't remember what * now does on two strings

emanresu A

Ring stranslate

!!/run `ol` `Hello, world!` *

Vyxal Bot

@emanresuA Heool, wlrod!

emanresu A

(although the argument order should be swapped)

Aaroneous Miller

23:58

The code boxes in the new site are doing a weird thing for me on safari on iOS:

user image

Can anybody else confirm?

« first day (592 days earlier) ← previous day next day → last day (1183 days later) »