#dev-contrib | Python | Page 109

gritty wind May 3, 2021, 12:56 AM

#

How did you set PG up?

short snow May 3, 2021, 1:01 AM

#

gritty wind How did you set PG up?

CREATE USER pysite WITH SUPERUSER PASSWORD 'pysite';
CREATE DATABASE pysite WITH OWNER pysite;
CREATE DATABASE metricity WITH OWNER pysite;

short snow May 3, 2021, 2:39 AM

#

i specified no password for it, but then it shows fatal, password authentication failed

short snow May 3, 2021, 3:08 AM

#

@cold island , do i show whether the message is truncated or not?

#

umm like this

cold island May 3, 2021, 1:57 PM

#

short snow umm like this

You could do something like

shortened_text = textwrap.shorten(text, 300)
if not shortened_text:
    shortened_text = text[:50].rstrip()

shortened_text = "\n".join(line for line in shortened_text.split("\n")[:4])

if len(shortened_text) < len(text):
    shortened_text += " ..."

#

but with those magic numbers in appropriate constants

cold island May 3, 2021, 3:06 PM

#

Btw nothing in the function you showed is async

fallen patrol May 3, 2021, 4:48 PM

#

cold island Btw nothing in the function you showed is async

Why's that problematic?

sleek steppe May 3, 2021, 4:48 PM

#

Why would you have an async function if you aren't gonna await anything

fallen patrol May 3, 2021, 4:50 PM

#

Why not

fervent sage May 3, 2021, 4:50 PM

#

because its confusing to have it async when it doesnt need to be, and confusing code is bad code

brazen charm May 3, 2021, 4:52 PM

#

You'd make the caller think that a context switch can occur while the above function will only run synchronously

fallen patrol May 3, 2021, 4:53 PM

#

Ah

#

Isn't that what await asyncio.sleep(0) does? It lets other things in the queue run.

sleek steppe May 3, 2021, 4:56 PM

#

Any time it yields, which is what asyncio.sleep(0) does

fallen patrol May 3, 2021, 5:02 PM

#

Ah

fallen patrol May 3, 2021, 6:29 PM

#

if @stable mountain is down and a member needs to be unmuted or muted so it happens manually, what occurs?

#

how does the bot resync the database with the actual state of the server when it comes back online?

tawdry vapor May 3, 2021, 6:30 PM

#

It doesn't

#

That also has to be done manually

fallen patrol May 3, 2021, 6:31 PM

#

rats

sleek steppe May 3, 2021, 6:33 PM

#

I thought it does?

tawdry vapor May 3, 2021, 6:34 PM

#

Well, it's one-way.

#

If the role is added manually, then there is no sync to the DB.

#

If the DB is edited manually, then the role will get added automatically.

sleek steppe May 3, 2021, 6:35 PM

#

Oh I think I read it wrong. Were they asking if it was applied manually would it remove it automatically?

fallen patrol May 3, 2021, 6:35 PM

#

~~i'm right here~~

fallen patrol May 3, 2021, 6:36 PM

#

sleek steppe Oh I think I read it wrong. Were they asking if it was applied manually would it...

if it was applied manually would the database realize that it was applied manually and create an infraction event

#

answer is no

tawdry vapor May 3, 2021, 6:37 PM

#

Also, if an infraction expires while the bot is offline, then it will never remove the role from the user.

sleek steppe May 3, 2021, 6:38 PM

#

I thought it just comes late, like reminders, or is that a different case

tawdry vapor May 3, 2021, 6:40 PM

#

Maybe. There's an active field separate from the expiration date. I don't know if that gets set automatically or manually.

#

If it's manual then yes, it would actually avoid the issue I mentioned.

#

By manual i mean that there has to be an API request to set it to false rather than the DB automatically making it false when expire date is reached.

cold island May 3, 2021, 6:45 PM

#

We've encountered expired yet active infractions before

sleek steppe May 3, 2021, 7:03 PM

#

tawdry vapor Maybe. There's an `active` field separate from the expiration date. I don't know...

Yeah, I muted an alt for 30 seconds, stopped the bot for 5 minutes, started it again and it got umnuted

patent pivot May 3, 2021, 10:47 PM

#

lol toxic, how did you accidentally sign one commit

tawdry vapor May 3, 2021, 10:49 PM

#

Joe, regarding your comment, that is cause the return type is inaccurate otherwise

#

send() returns a message but commands are typed as returning None typically.

#

So either the return type has to change or the return has to be on a separate line

patent pivot May 3, 2021, 10:50 PM

#

Hmmm, yeah, that's what Xith told me

#

It makes sense now thinking about it, but stylistically it kills me a bit

tawdry vapor May 3, 2021, 10:50 PM

#

I've grown to live with it

sleek steppe May 3, 2021, 10:55 PM

#

patent pivot lol toxic, how did you accidentally sign one commit

Wasn't an accident, I wanted to try signing, but I put the wrong email so I just stopped signing it

patent pivot May 3, 2021, 10:55 PM

#

ahhhha that makes sense hahaha

vocal wolf May 3, 2021, 11:00 PM

#

@ashen isle Greetings. I know it's only been like 4 days since I last poked you, but I forgot to suggest that if you're not able to finish the tag PRs, we can have someone take over.

#

@whole forge Greetings. What's the status of this? https://github.com/python-discord/bot/pull/1430

thorny obsidian May 3, 2021, 11:03 PM

#

I wonder if I could set-up a basic test bot that allows for the embed command just to help build out tags so you don't need to spin up the entirety of a bot

#

and we can just have it in a test server to use the embed cmd

vocal wolf May 3, 2021, 11:07 PM

#

bot#1441 needs review.

dusky shoreBOT May 3, 2021, 11:07 PM

#

GitHub

PROpen Unescape html escape characters in reddit text and titles

vocal wolf May 3, 2021, 11:07 PM

#

small.

sleek steppe May 3, 2021, 11:08 PM

#

Oh yeah, I was gonna ask if that was even going to get merged because of bot#1542

dusky shoreBOT May 3, 2021, 11:08 PM

#

GitHub

PROpen Annihilate reddit cog

vocal wolf May 3, 2021, 11:09 PM

#

oh right that's a thing

vocal wolf May 3, 2021, 11:10 PM

#

dusky shore

@eternal owl this needs completion.

ashen isle May 3, 2021, 11:11 PM

#

vocal wolf <@!182894631921516545> Greetings. I know it's only been like 4 days since I last...

that would probably be a good idea. there's probably like half a change left and I don't think I can any time soon

vocal wolf May 3, 2021, 11:11 PM

#

Alright. thanks for update/quick reply.

#

I'll take over, then. I haven't contributed in a while.

tawdry vapor May 3, 2021, 11:12 PM

#

thorny obsidian I wonder if I could set-up a basic test bot that allows for the embed command ju...

There's a website that can visualise embeds

thorny obsidian May 3, 2021, 11:17 PM

#

which website?

vocal prairie May 3, 2021, 11:24 PM

#

thorny obsidian which website?

https://cog-creators.github.io/discord-embed-sandbox/

#

That one can do it. Although it isn't the best.

tawdry vapor May 3, 2021, 11:25 PM

#

https://leovoel.github.io/embed-visualizer/

thorny obsidian May 3, 2021, 11:26 PM

#

huh, super neat. Thanks!

fallen patrol May 3, 2021, 11:29 PM

#

patent pivot lol toxic, how did you accidentally sign one commit

How do sign commits?

patent pivot May 3, 2021, 11:29 PM

#

fallen patrol How do sign commits?

https://docs.github.com/en/github/authenticating-to-github/signing-commits

Signing commits - GitHub Docs

You can sign commits locally using GPG or S/MIME.

fallen patrol May 3, 2021, 11:30 PM

#

thanks

#

i've made like

#

uh

#

15-30 commits to a repo this week

patent pivot May 3, 2021, 11:31 PM

#

dope

fallen patrol May 3, 2021, 11:31 PM

#

#

👀

#

linux

patent pivot May 3, 2021, 11:32 PM

#

gpg-agent works on linux

fallen patrol May 3, 2021, 11:33 PM

#

ah

#

[bleep]

patent pivot May 3, 2021, 11:35 PM

#

thinkingcrew

fallen patrol May 3, 2021, 11:37 PM

#

#

what name do we enter 🤔

#

github name, or alias name

#

sleek steppe May 3, 2021, 11:38 PM

#

Real name /s

patent pivot May 3, 2021, 11:38 PM

#

you can use a github name

fallen patrol May 3, 2021, 11:38 PM

#

username?

patent pivot May 3, 2021, 11:38 PM

#

yes

#

anything

#

it doesn't really matter, though some orgs may require you to sign with a key that has an actual identity on it

fallen patrol May 3, 2021, 11:39 PM

#

patent pivot anything

😂

fallen patrol May 3, 2021, 11:52 PM

#

patent pivot gpg-agent works on linux

….

#

https://linux.die.net/man/1/gpg-agent

gpg-agent(1): Secret key management for GnuPG - Linux man page

gpg-agent is a daemon to manage secret (private) keys independently from any protocol. It is used as a backend for gpg and gpgsm as well as for a couple of other utilities.

patent pivot May 3, 2021, 11:52 PM

#

huh?

fallen patrol May 3, 2021, 11:52 PM

#

#

its an out of date manual 🤔

patent pivot May 3, 2021, 11:54 PM

#

I don't use gpg-agent anymore ¯_(ツ)_/¯

#

I just use the macOS keychain

fallen patrol May 3, 2021, 11:54 PM

#

ah

#

i use linux

fallen patrol May 4, 2021, 12:06 AM

#

patent pivot I don't use gpg-agent anymore ¯\_(ツ)_/¯

good

#

its confusing and i am so confused now

patent pivot May 4, 2021, 12:06 AM

#

lol

vocal prairie May 4, 2021, 12:06 AM

#

Why not just, not sign the commits?

fallen patrol May 4, 2021, 12:06 AM

#

the only thing i've managed to do is

patent pivot May 4, 2021, 12:06 AM

#

yeah no that's correct

#

you just haven't added the key to your github account

fallen patrol May 4, 2021, 12:07 AM

#

vocal prairie Why not just, not sign the commits?

because signing is good and I've made a bunch of commits to a OOS project today

fallen patrol May 4, 2021, 12:07 AM

#

patent pivot yeah no that's correct

yeah no that was retroactive

#

i turned on a setting when trying to add a key and now github is like that

patent pivot May 4, 2021, 12:08 AM

#

ah, right

vocal prairie May 4, 2021, 12:08 AM

#

Oh wait, I didn't recognize you because of your pfp.

fallen patrol May 4, 2021, 12:08 AM

#

patent pivot you just haven't added the key to your github account

those commits weren't signed

patent pivot May 4, 2021, 12:08 AM

#

lol

#

gotcha

fallen patrol May 4, 2021, 12:08 AM

#

vocal prairie Oh wait, I didn't recognize you because of your pfp.

lol i took a screenshot of my background today and changed it to that lmfao

#

#

bing image otd

#

anyways

#

joe is there an alternative to gpg-agent

patent pivot May 4, 2021, 12:11 AM

#

not really

#

i mean probably

#

but gpg-agent is by far the most frequently used one

#

we're veering off topic from pydis discussions here now though

vocal prairie May 4, 2021, 12:14 AM

#

So for site#366, I don't really know how you would go about implementing this, especially with allowing fuzzy hits and searching both markdown and html pages. Does anyone have any thoughts?

dusky shoreBOT May 4, 2021, 12:14 AM

#

GitHub

IssueOpen Dewikification: Search function

patent pivot May 4, 2021, 12:14 AM

#

hmmmmm

#

wonder if something like https://www.algolia.com/ is worth looking into here

#

I'm not sure really

vocal prairie May 4, 2021, 12:15 AM

#

Algolia is great from my limited knowledge, especially with their new autocomplete.

fallen patrol May 4, 2021, 12:15 AM

#

patent pivot wonder if something like https://www.algolia.com/ is worth looking into here

i opened that thinking it was gpg related 🤦 🥴

vocal prairie May 4, 2021, 12:15 AM

#

But someone mentioned about how we prefer to do things internally here last time I brought this up.

patent pivot May 4, 2021, 12:16 AM

#

No, let's move GPG discussion out of here.

patent pivot May 4, 2021, 12:16 AM

#

vocal prairie But someone mentioned about how we prefer to do things internally here last time...

I think that is used a bit when talking about lancebot (I remember spearheading that a bit here), but yeah, I get the point

#

hmmmm

fallen patrol May 4, 2021, 12:16 AM

#

vocal prairie But someone mentioned about how we prefer to do things internally here last time...

kutiekat

#

glorified api wrapper iirc

thorny obsidian May 4, 2021, 12:17 AM

#

I was fine for trying to home roll our own if we wanted, but I'm also fine with using something else.

vocal prairie May 4, 2021, 12:17 AM

#

How in the world do all the mods appear right when they're mentioned 👀

fallen patrol May 4, 2021, 12:18 AM

#

patent pivot I think that is used a bit when talking about lancebot (I remember spearheading ...

damn i apparently have a great memory to get the one phrase correct

patent pivot May 4, 2021, 12:18 AM

#

It's two separate things really, Lancebot is an educational resource and I know that when I've commented on it then we've been primarily talking about Lancebot

thorny obsidian May 4, 2021, 12:18 AM

#

This was my comment: #dev-contrib message

patent pivot May 4, 2021, 12:18 AM

#

I think that if there is a tool that can achieve what we want to in site better than we can we can look into that, but yeah, it is as a bit of a last ditch attempt if we can't implement our own

#

Searching is indeed hard, especially since we've got a bit of a split of content between markdown/dynamic and so on

vocal prairie May 4, 2021, 12:20 AM

#

There's also the problem about being able to do things as people type

patent pivot May 4, 2021, 12:20 AM

#

yeah

vocal prairie May 4, 2021, 12:23 AM

#

This kind of fuzzy search would be great. But it's a very hard thing to implement.

fallen patrol May 4, 2021, 12:23 AM

#

vocal prairie This kind of fuzzy search would be great. But it's a very hard thing to implemen...

is it?

#

!pypi fuzzywuzzy

stable mountainBOT May 4, 2021, 12:24 AM

#

fuzzywuzzy v0.18.0

Fuzzy string matching in python

patent pivot May 4, 2021, 12:24 AM

#

Yes, it is

fallen patrol May 4, 2021, 12:24 AM

#

i personally use that

#

which uses difflib within itself

#

or python-Levenshtein

vocal prairie May 4, 2021, 12:24 AM

#

fallen patrol is it?

You're forgetting that we have to do it while people are typing, and while referencing markdown and html files

patent pivot May 4, 2021, 12:24 AM

#

How do you propose we search the content for pages that we haven't yet rendered? For example, the resources page

#

the fuzzy algorithms are fine once you've got the content and a way to provide results

#

we don't have that in some situations

vocal prairie May 4, 2021, 12:25 AM

#

vocal prairie This kind of fuzzy search would be great. But it's a very hard thing to implemen...

This is from some page that algolia tweeted about today

patent pivot May 4, 2021, 12:25 AM

#

yeah

#

I know Cloudflare use Algolia for their docs

vocal prairie May 4, 2021, 12:26 AM

#

I just found a wonderful example of good search… only to realize they used algolia

patent pivot May 4, 2021, 12:26 AM

#

pahahahaha

vocal prairie May 4, 2021, 12:28 AM

#

The only thing I can think of is to do something that indexes website content before it renders. Or at least, before search renders.

#

But I have no idea what that could be

cold moon May 4, 2021, 5:16 AM

#

I think using Algolia is good idea for searching.

short snow May 4, 2021, 5:18 AM

#

isn't algolia paid? like $2 per month

cold moon May 4, 2021, 5:20 AM

#

This have free plan

#

But max is 1k searches per month

short snow May 4, 2021, 5:39 AM

#

ah, won't that be a problem then?

short snow May 4, 2021, 5:40 AM

#

cold island You could do something like ```py shortened_text = textwrap.shorten(text, 300) ...

right, this is shorter and smarter, are we limiting to 3 or 4 lines?

#

and yeah i will change it to a non async function

cold island May 4, 2021, 5:51 AM

#

short snow right, this is shorter and smarter, are we limiting to `3` or `4` lines?

This won't work, sorry. You need to check for the placeholder and not an empty string

#

And an empty placeholder makes the function behave differently

short snow May 4, 2021, 5:51 AM

#

we can pass the placeholder as an empty string

#

default is [...]

#

ah, what do you mean by behanve differently?

cold island May 4, 2021, 5:52 AM

#

It will slice the first word instead of discarding it fully

#

But only the first word. The rest of the sentence is treated the same

#

¯\_(ツ)_/¯

short snow May 4, 2021, 6:03 AM

#

hmm, then the current solution should be good

cold island May 4, 2021, 6:04 AM

#

Well, it would be better to first slice the message

#

And then count newlines

short snow May 4, 2021, 6:07 AM

#

reduce it to 300 characters and then count?

cold island May 4, 2021, 6:08 AM

#

Yeah, as then you have less things to count

#

Well, to either 300 or 50

#

Also just making the placeholder three dots is enough IMO, the embed should be as short as possible

short snow May 4, 2021, 6:12 AM

#

right, got that

#

would it be better to use fields for the content?

#

since it is multilined

cold island May 4, 2021, 6:13 AM

#

Hmm why fields? It's the same piece of text

short snow May 4, 2021, 6:15 AM

#

this will happen without fields:

**Content:** This is
not a single
line :)

with fields:

#Content
This is
not a single
line :)

cold island May 4, 2021, 6:39 AM

#

Mmm maybe. I feel like it will make it too vertical

short snow May 4, 2021, 6:44 AM

#

uhh, it just addition of ~1 line

#

i will try both and show the diff here, once free

#

bump lemon_grimace

cold island May 4, 2021, 6:51 AM

#

short snow uhh, it just addition of ~1 line

So content will be a field, but the rest won't be in fields?

short snow May 4, 2021, 6:51 AM

#

yep

#

rest aren't needed to be a field

#

they can be a inline field tho

cold island May 4, 2021, 6:58 AM

#

I don't think inline fields will render well on mobile

green oriole May 4, 2021, 7:02 AM

#

I believe they are fixed now

#

Can you int e one?

brazen charm May 4, 2021, 8:45 AM

#

Any input on bot#1545 ?

dusky shoreBOT May 4, 2021, 8:45 AM

#

GitHub

IssueOpen Create tag groups

whole forge May 4, 2021, 10:15 AM

#

vocal wolf <@!151347084602245120> Greetings. What's the status of this? <https://github.com...

it should work, but IIRC there was no consensus on whether this was something people actually want

green oriole May 4, 2021, 10:19 AM

#

I don't have any strong opinion one way or the other

#

I don't like the use of the x- prefix though

short snow May 4, 2021, 10:19 AM

#

short snow ```bash CREATE USER pysite WITH SUPERUSER PASSWORD 'pysite'; CREATE DATABASE pys...

ok i tried twice but am always getting this error, is there a way around this?

green oriole May 4, 2021, 10:20 AM

#

What's the exact error?

short snow May 4, 2021, 10:20 AM

#

here

green oriole May 4, 2021, 10:20 AM

#

Interesting

#

When do you get that?

#

As soon as you start psql?

short snow May 4, 2021, 10:24 AM

#

yeah

#

after doing sudo service postgresql start

#

Loading .env environment variables...
Starting in debug mode.
Starting server.
Starting in debug mode.
Waiting for PostgreSQL database.
Database is ready.
Applying migrations.
``` before the error

#

i did it again now, but still the error remains

vocal prairie May 4, 2021, 11:32 AM

#

cold moon But max is 1k searches per month

10,000

#

According to their pricing page

#

But still, that's like a request every four minutes

#

Which isn't great

whole forge May 4, 2021, 11:48 AM

#

green oriole I don't like the use of the `x-` prefix though

the x prefix is needed tho to make sure compose doesn't try to read those blocks as actual compose blocks like services or networks

green oriole May 4, 2021, 11:49 AM

#

I know, why not pydis-logging or so?

whole forge May 4, 2021, 11:53 AM

#

green oriole I know, why not `pydis-logging` or so?

it would have to be x-pydis-logging

https://docs.docker.com/compose/compose-file/compose-file-v3/#extension-fields
Those special fields can be of any format as long as they are located at the root of your Compose file and their name start with the x- character sequence.

green oriole May 4, 2021, 11:55 AM

#

Oh okay

#

Fair enough then

whole forge May 4, 2021, 11:56 AM

#

so should i un-draft-ify that PR?

#

bot#1430

dusky shoreBOT May 4, 2021, 11:58 AM

#

GitHub

PRDraft Add logging and restart config to docker compose

whole forge May 4, 2021, 11:58 AM

#

oh neat it does have a draft icon

short snow May 4, 2021, 12:08 PM

#

short snow ```txt Loading .env environment variables... Starting in debug mode. Starting se...

hmm, do you know akarys?

green oriole May 4, 2021, 12:25 PM

#

Not sure, there is probably a root password set somehow

green oriole May 4, 2021, 12:26 PM

#

whole forge so should i un-draft-ify that PR?

Yeah, sure. Although for the future it would be best if you open an issue first.

short snow May 4, 2021, 12:28 PM

#

green oriole Not sure, there is probably a root password set somehow

how do i remove the root password?

green oriole May 4, 2021, 12:28 PM

#

You probably shouldn't

#

But I don't know, it depends on how we installed it

whole forge May 4, 2021, 12:29 PM

#

green oriole Yeah, sure. Although for the future it would be best if you open an issue first.

in the past i have been told that if i want to do something myself anyways, theres no need for a issue

green oriole May 4, 2021, 12:30 PM

#

Well, we need to coordinate contributions first, or we can end up in spending a lot of time working on a PR (both for the contributors and the reviewers) and end up scraping them.

whole forge May 4, 2021, 12:31 PM

#

in this case, i had the modified docker compose anyways since thats what im using personally, so i made a draft PR to be able to coordinate whether this is something we want

green oriole May 4, 2021, 12:33 PM

#

Well we may want a totally different configuration or solution, that's why issues are important. Besides we more often check issues rather than PRs for early discussions.

vocal prairie May 4, 2021, 12:47 PM

#

Can sir-lancebot#634 be looked at to remove the Easter .topics since Easter is over?

dusky shoreBOT May 4, 2021, 12:47 PM

#

GitHub

IssueOpen over-used easter topics in .topic command of sir-lancebot

green oriole May 4, 2021, 12:48 PM

#

Will ask others ok_handbutflipped

short snow May 4, 2021, 12:48 PM

#

green oriole But I don't know, it depends on how we installed it

should i reinstall it

sleek steppe May 4, 2021, 12:48 PM

#

whole forge oh neat it does have a draft icon

sir-lancebot#665 😎

dusky shoreBOT May 4, 2021, 12:48 PM

#

GitHub

PRMerged Use PRDraft emoji when the pr is a draft pr for the .issue|.pr command

short snow May 4, 2021, 12:55 PM

#

whats the correct way to install psql?

#

ubuntu

cold moon May 4, 2021, 12:56 PM

#

sudo apt install postgresql-client

vocal prairie May 4, 2021, 12:56 PM

#

https://www.postgresql.org/download/linux/ubuntu/ Here's the guide if you want

short snow May 4, 2021, 1:08 PM

#

weeeee it worked now

#

hmm, the site now works on 0.0.0.0:8000, but all the links direct to pythondiscord.local:8000

#

to get

fallen patrol May 4, 2021, 1:18 PM

#

short snow hmm, the site now works on `0.0.0.0:8000`, but all the links direct to `pythondi...

:blink:

green oriole May 4, 2021, 1:19 PM

#

short snow hmm, the site now works on `0.0.0.0:8000`, but all the links direct to `pythondi...

Have you setup your host files?

sleek steppe May 4, 2021, 1:24 PM

#

haha lol (spring cleanup PR if someone wants to review)

short snow May 4, 2021, 1:25 PM

#

green oriole Have you setup your host files?

lemme add that, just a hosts in the root directory right?

green oriole May 4, 2021, 1:26 PM

#

no no, in /etc/hosts

short snow May 4, 2021, 1:30 PM

#

yeah 🤦‍♂️, worked thanks

fallen patrol May 4, 2021, 1:42 PM

#

👀

#

Superstarify & catify.

#

What would happen

gritty wind May 4, 2021, 1:44 PM

#

Wdym?

#

Nothing

#

It'll just catify the starified name

#

not really a problem

short snow May 4, 2021, 1:45 PM

#

if a user is under a active superstarify, and someone/anyone changes their name, it would be changed back to the superstarified named

#

not sure, lemme have a look at the source, one sec

#

!src superstarify

stable mountainBOT May 4, 2021, 1:45 PM

#

Command: superstarify

Temporarily force a random superstar name (like Taylor Swift) to be the user's nickname.

Source Code

Go to GitHub

short snow May 4, 2021, 1:46 PM

#

https://github.com/python-discord/bot/blob/main/bot/exts/moderation/infraction/superstarify.py#L63-L82

#

yep

#

python didn't respond?

vocal prairie May 4, 2021, 1:58 PM

#

Is quackstack stable enough for site#465?

dusky shoreBOT May 4, 2021, 1:58 PM

#

GitHub

IssueOpen Build-Your-Own Ducky!

short snow May 4, 2021, 2:58 PM

#

vocal prairie Is quackstack stable enough for site#465?

yes!

dry folio May 4, 2021, 3:00 PM

#

@clever wraith @wide elm @vocal prairie CAT DEVS UNITE

#

WE HAVE A NEW PROJECT

#

PROJECT: ADD MATH AND SCIENCE CATEGORY TO SIR LANCEBOT'S .quiz command

wide elm May 4, 2021, 3:00 PM

#

hullo

vocal prairie May 4, 2021, 3:00 PM

#

Chill with the caps Object

dry folio May 4, 2021, 3:00 PM

#

vocal prairie Chill with the caps Object

lol

vocal prairie May 4, 2021, 3:00 PM

#

And y'all can go ahead, I'm busy

dry folio May 4, 2021, 3:00 PM

#

vocal prairie And y'all can go ahead, I'm busy

aw, fine

wide elm May 4, 2021, 3:00 PM

#

aw, alright

clever wraith May 4, 2021, 3:01 PM

#

dry folio <@456226577798135808> <@!416345453530054657> <@!788119126450765844> CAT DEVS UNI...

Aight

dry folio May 4, 2021, 3:01 PM

#

clever wraith Aight

first we'll need to open an issue, ima go do that

wide elm May 4, 2021, 3:01 PM

#

k

vocal prairie May 4, 2021, 3:01 PM

#

short snow yes!

Okie. I asked a week or two ago, and it was a no, so I was wondering

clever wraith May 4, 2021, 3:01 PM

#

also, can y'all stop TYPING ALL CAPS IT HURTS MY FKING EYES

You were saying @Object#9891 ?

fervent sage May 4, 2021, 3:01 PM

#

Am a cat too pensivewobble

dry folio May 4, 2021, 3:01 PM

#

clever wraith > also, can y'all stop TYPING ALL CAPS IT HURTS MY FKING EYES You were saying @...

:)

wide elm May 4, 2021, 3:01 PM

#

fervent sage Am a cat too <a:pensivewobble:749519576060002337>

haha

clever wraith May 4, 2021, 3:02 PM

#

Why the fuck are mentions not working

dry folio May 4, 2021, 3:02 PM

#

fervent sage Am a cat too <a:pensivewobble:749519576060002337>

join us in our project then

vocal prairie May 4, 2021, 3:02 PM

#

vocal prairie Is quackstack stable enough for site#465?

It is, right vco?

fervent sage May 4, 2021, 3:02 PM

#

Yep

vocal prairie May 4, 2021, 3:02 PM

#

Noice

sleek steppe May 4, 2021, 3:02 PM

#

Anyone wants to review sir-lancebot#718 so I don't have to resolve a bunch of merge conflicts since it's a big diff

dusky shoreBOT May 4, 2021, 3:02 PM

#

GitHub

PROpen Spring cleanup

mellow hare May 4, 2021, 3:03 PM

#

dry folio <@456226577798135808> <@!416345453530054657> <@!788119126450765844> CAT DEVS UNI...

Please don't spam ping people just because they have cats in their nickname

clever wraith May 4, 2021, 3:04 PM

#

@sleek steppe nice

fervent sage May 4, 2021, 3:04 PM

#

Lol

wide elm May 4, 2021, 3:04 PM

#

lmao

lyric raven May 4, 2021, 3:04 PM

#

xD

mellow hare May 4, 2021, 3:04 PM

#

It's troublesome and unnecessary

sleek steppe May 4, 2021, 3:04 PM

#

clever wraith <@506618674921340958> nice

https://discordapp.com/channels/267624335836053506/635950537262759947/839130289300701254

dry folio May 4, 2021, 3:04 PM

#

mellow hare Please don't spam ping people just because they have cats in their nickname

no, we're a group

wide elm May 4, 2021, 3:04 PM

#

we discussed beforehand

fervent sage May 4, 2021, 3:04 PM

#

I'm surprised it isn't more tbh

dry folio May 4, 2021, 3:04 PM

#

mellow hare Please don't spam ping people just because they have cats in their nickname

we decided to work on this

mellow hare May 4, 2021, 3:04 PM

#

.... seriously?

wide elm May 4, 2021, 3:04 PM

#

yeah

dry folio May 4, 2021, 3:04 PM

#

mellow hare Please don't spam ping people just because they have cats in their nickname

check #ot0-psvm’s-eternal-disapproval for details

mellow hare May 4, 2021, 3:04 PM

#

Uh.... huh

clever wraith May 4, 2021, 3:04 PM

#

sleek steppe https://discordapp.com/channels/267624335836053506/635950537262759947/8391302893...

What

wide elm May 4, 2021, 3:04 PM

#

lmao

dry folio May 4, 2021, 3:04 PM

#

https://github.com/python-discord/sir-lancebot/issues/719

GitHub

Add "Math" and "Science" category in .quiz · Issue #719 · python-di...

Description Since the .quiz command currently only has the "General" category which is pretty much only geography and history questions, we propose to add "Math"...

wide elm May 4, 2021, 3:05 PM

#

eyy

fervent sage May 4, 2021, 3:05 PM

#

I feel you hemlock lol

clever wraith May 4, 2021, 3:05 PM

#

Did we just pull one over on hemlock

dry folio May 4, 2021, 3:05 PM

#

mellow hare .... seriously?

yes, this is our official website
https://catdevs.tk

Cat Dev Group

The site for the Cat Dev Group

fervent sage May 4, 2021, 3:05 PM

#

Omg

wide elm May 4, 2021, 3:05 PM

#

haha

clever wraith May 4, 2021, 3:05 PM

#

https://catdevs.tk object get it right jesus

Cat Dev Group

The site for the Cat Dev Group

dry folio May 4, 2021, 3:05 PM

#

wait it's offline

fervent sage May 4, 2021, 3:05 PM

#

dead

dry folio May 4, 2021, 3:05 PM

#

clever wraith https://catdevs.tk object get it right jesus

lmao sorry

clever wraith May 4, 2021, 3:06 PM

#

Lol

vocal prairie May 4, 2021, 3:06 PM

#

In alphabetical order, does a space go before or after letters?

wide elm May 4, 2021, 3:06 PM

#

hm

clever wraith May 4, 2021, 3:06 PM

#

I have no ide

#

Idea

wide elm May 4, 2021, 3:06 PM

#

clever wraith I have no ide

I have an IDE

dry folio May 4, 2021, 3:06 PM

#

vocal prairie In alphabetical order, does a space go before or after letters?

check ascii

thorny obsidian May 4, 2021, 3:06 PM

#

Hi cat devs! Since this will be your second PR, please do try to follow our contributing guidelines and especially do the following:

Y'all better lint everything before you even commit
Have good commit messages please
Make sure you're on the same page and test things before you PR

wide elm May 4, 2021, 3:06 PM

#

Ok

placid ermine May 4, 2021, 3:06 PM

#

dry folio wait it's offline

bruh

dry folio May 4, 2021, 3:06 PM

#

thorny obsidian Hi cat devs! Since this will be your second PR, please do try to follow our cont...

yes, we understand now and everyone has black

fervent sage May 4, 2021, 3:06 PM

#

Uh in ASCII, before

clever wraith May 4, 2021, 3:06 PM

#

Linting *cough* object *cough*

wide elm May 4, 2021, 3:07 PM

#

haha

thorny obsidian May 4, 2021, 3:07 PM

#

Thanks~ Don't rely on black entirely. Our linting doesn't necessarily follow black.

dry folio May 4, 2021, 3:07 PM

#

clever wraith Linting \*cough\* object \*cough\*

lol, i installed black

vocal prairie May 4, 2021, 3:07 PM

#

vocal prairie In alphabetical order, does a space go before or after letters?

I think spaces go before

wide elm May 4, 2021, 3:07 PM

#

thorny obsidian Thanks~ Don't rely on black entirely. Our linting doesn't necessarily follow bla...

Hm, k

sleek steppe May 4, 2021, 3:07 PM

#

Don't use black for the most part

dry folio May 4, 2021, 3:07 PM

#

thorny obsidian Thanks~ Don't rely on black entirely. Our linting doesn't necessarily follow bla...

oh, so is there a better one?

placid ermine May 4, 2021, 3:07 PM

#

just do pipenv run lint

wide elm May 4, 2021, 3:07 PM

#

Aigt

thorny obsidian May 4, 2021, 3:07 PM

#

what stinky said. If you run the lint, it'll let you know what the errors are. There is benefit to manually fixing the errors.

fervent sage May 4, 2021, 3:07 PM

#

Flake8 makes my code look worse pensivewobble

wide elm May 4, 2021, 3:07 PM

#

haha

clever wraith May 4, 2021, 3:07 PM

#

Anyways I gotta run

placid ermine May 4, 2021, 3:07 PM

#

pro

#

orz

dry folio May 4, 2021, 3:08 PM

#

thorny obsidian what stinky said. If you run the lint, it'll let you know what the errors are. T...

ohhh ok, so do i have to install lint?

clever wraith May 4, 2021, 3:08 PM

#

I'll do the math shit later

fervent sage May 4, 2021, 3:08 PM

#

No you just run the command

placid ermine May 4, 2021, 3:08 PM

#

dry folio ohhh ok, so do i have to install `lint`?

no

gritty wind May 4, 2021, 3:08 PM

#

Lint is a pipenv command, not a package

dry folio May 4, 2021, 3:08 PM

#

gritty wind Lint is a pipenv command, not a package

so i install pipenv. yes?

clever wraith May 4, 2021, 3:08 PM

#

Lmao

gritty wind May 4, 2021, 3:08 PM

#

How are you working on these projects

vocal prairie May 4, 2021, 3:08 PM

#

Also, do you pay attention to the word "the" when doing alphabetical order?

thorny obsidian May 4, 2021, 3:09 PM

#

yup! pipenv is great. I recommend going through our setup guide and using pipenv

sleek steppe May 4, 2021, 3:09 PM

#

@fervent sage https://github.com/python-discord/sir-lancebot/pull/718/files#r625864163 🤔

placid ermine May 4, 2021, 3:09 PM

#

gritty wind How are you working on these projects

anarchy

clever wraith May 4, 2021, 3:09 PM

#

gritty wind How are you working on these projects

As a team

#

Ish

placid ermine May 4, 2021, 3:09 PM

#

nah its just anarchy

gritty wind May 4, 2021, 3:09 PM

#

Let me reword that question

clever wraith May 4, 2021, 3:09 PM

#

Lol

gritty wind May 4, 2021, 3:09 PM

#

How are you working on this project without pipenv

clever wraith May 4, 2021, 3:09 PM

#

We aren't

dry folio May 4, 2021, 3:09 PM

#

placid ermine anarchy

no, that's anaguild game one which died

wide elm May 4, 2021, 3:09 PM

#

clever wraith As a team

haha

clever wraith May 4, 2021, 3:09 PM

#

Or at least I'm not

wide elm May 4, 2021, 3:09 PM

#

we haven't started yet

fervent sage May 4, 2021, 3:09 PM

#

sleek steppe <@!297045071457681409> <https://github.com/python-discord/sir-lancebot/pull/718/...

Lol tf

dry folio May 4, 2021, 3:10 PM

#

gritty wind How are you working on this project without pipenv

i think i have it i just don't know it exists lol

clever wraith May 4, 2021, 3:10 PM

#

*cough * object again *cough*

wide elm May 4, 2021, 3:10 PM

#

lol

clever wraith May 4, 2021, 3:10 PM

#

God my throat is sore today I should take a cough drop

vocal prairie May 4, 2021, 3:10 PM

#

vocal prairie Also, do you pay attention to the word "the" when doing alphabetical order?

So would "The Weeknd" go in T or W?

dry folio May 4, 2021, 3:10 PM

#

clever wraith \*cough \* object again \*cough\*

ok SORRY i don't have enough open-source dev experience

#

i'm learning :(

clever wraith May 4, 2021, 3:10 PM

#

Lol it's not that just read the contributing guide

gritty wind May 4, 2021, 3:10 PM

#

Like Kat said, the getting started guide would be a good place to start

clever wraith May 4, 2021, 3:10 PM

#

vocal prairie So would "The Weeknd" go in T or W?

Is say T

dry folio May 4, 2021, 3:11 PM

#

thorny obsidian yup! pipenv is great. I recommend going through our setup guide and using pipenv

alright!

vocal prairie May 4, 2021, 3:11 PM

#

Okay

fervent sage May 4, 2021, 3:11 PM

#

Yeah T sounds good for that

dry folio May 4, 2021, 3:11 PM

#

thorny obsidian Hi cat devs! Since this will be your second PR, please do try to follow our cont...

but we need the issue to be approved first lol

clever wraith May 4, 2021, 3:11 PM

#

Otherwise it would be "Weeknd, The" becuase alphabet is weird

wide elm May 4, 2021, 3:11 PM

#

lmao

dry folio May 4, 2021, 3:12 PM

#

shall we start working on the questions? we need 60 in total

#

30 for math, 30 for science

wide elm May 4, 2021, 3:12 PM

#

I can't atm, but I can start in a bit

dry folio May 4, 2021, 3:12 PM

#

k

#

i'll ping y'all in the server

fallen patrol May 4, 2021, 3:12 PM

#

dry folio oh, so is there a better one?

Doing it manually based on what Flake8 says :hehe

clever wraith May 4, 2021, 3:13 PM

#

wide elm I can't atm, but I can start in a bit

Same

sleek steppe May 4, 2021, 3:14 PM

#

For https://github.com/python-discord/sir-lancebot/pull/718#issuecomment-832018486 😔

green oriole May 4, 2021, 3:20 PM

#

I’m not sure if this is really needed tbh

vocal prairie May 4, 2021, 3:26 PM

#

This link isn't working https://quackstack.pythondiscord.com/manduck

#

It's sending errors like: {"detail":"Invalid option provided: lightsaber.png not found."}

clever wraith May 4, 2021, 3:27 PM

#

Yeah @fervent sage quackstack manducks are having a tough time

green oriole May 4, 2021, 3:27 PM

#

nice

fervent sage May 4, 2021, 3:28 PM

#

I'll take a peek now

clever wraith May 4, 2021, 3:28 PM

#

Also bills on normal ducks are broken

#

fervent sage May 4, 2021, 3:28 PM

#

known issue, will be fixed soonTM

clever wraith May 4, 2021, 3:28 PM

#

Kop

#

Oop

#

Wrong photo

stable mountainBOT May 4, 2021, 3:29 PM

#

Hey @clever wraith!

It looks like you tried to attach file type(s) that we do not allow (). We currently allow the following file types: .gif, .jpg, .jpeg, .mov, .mp4, .mpg, .png, .mp3, .wav, .ogg, .webm, .webp, .flac, .m4a.

Feel free to ask in #community-meta if you think this is a mistake.

placid ermine May 4, 2021, 3:29 PM

#

clever wraith Wrong photo

sure

clever wraith May 4, 2021, 3:29 PM

#

Wh

#

Okay it doesn't like that image

#

There

sleek steppe May 4, 2021, 3:30 PM

#

green oriole I’m not sure if this is really needed tbh

@brazen charm what do you think?

fallen patrol May 4, 2021, 3:37 PM

#

fervent sage known issue, will be fixed <:soonTM:788808419062710332>

69 commits is not an issue tho

sleek steppe May 4, 2021, 3:44 PM

#

sleek steppe <@!184351770636582913> what do you think?

I feel like this is gonna make a billion merge conflicts for my PR or other PRs I I'll leave it

fallen patrol May 4, 2021, 3:45 PM

#

Just wait for the other prs to happen and then change it

sleek steppe May 4, 2021, 3:46 PM

#

That means even more conflicts

fallen patrol May 4, 2021, 3:46 PM

#

I mean

#

New pr*

brazen charm May 4, 2021, 3:49 PM

#

sleek steppe I feel like this is gonna make a billion merge conflicts for my PR or other PRs ...

The changes are simple so those should be mostly resolved automatically, and most of the PRs are for new features. I don't see a reason not to do it as it'll be the last normal opportunity if everything gets reviewed properly in the future

slim widget May 4, 2021, 3:50 PM

#

vocal prairie So would "The Weeknd" go in T or W?

Not sure what the context is, but I would tend to just do both.

#

If it's like, some kind of index on a website.

#

So that the user doesn't have to go hunting for it.

vocal prairie May 4, 2021, 3:51 PM

#

I was listing things alphabetically for the superstarify command

#

We were adding new people

#

So the order doesn't really matter

#

I was just curious

slim widget May 4, 2021, 3:51 PM

#

Ohh, right.

sleek steppe May 4, 2021, 3:59 PM

#

brazen charm The changes are simple so those should be mostly resolved automatically, and mos...

ah ok

short snow May 4, 2021, 4:04 PM

#

vocal prairie Okie. I asked a week or two ago, and it was a no, so I was wondering

Ah alright, just need to make the listing to show man ducks and it would be done, @fervent sage had a few more ideas tho.

But yeah it is perfect and stable, a good ducky!

fervent sage May 4, 2021, 4:04 PM

#

short snow Ah alright, just need to make the listing to show man ducks and it would be done...

stable™️ but not quite™️

#

https://quackstack.pydis.com/manduck

short snow May 4, 2021, 4:08 PM

#

fervent sage known issue, will be fixed <:soonTM:788808419062710332>

What’s the issue?

fervent sage May 4, 2021, 4:09 PM

#

it returns a 400 on every request for random ones

clever wraith May 4, 2021, 4:09 PM

#

fervent sage https://quackstack.pydis.com/manduck

Aad

#

Sad

short snow May 4, 2021, 4:09 PM

#

Hmmm

clever wraith May 4, 2021, 4:09 PM

#

It can't find the assets

short snow May 4, 2021, 4:12 PM

#

I will have a look at it if I get time today

#

!remind 12hrs do it if not done yet

stable mountainBOT May 4, 2021, 4:19 PM

#

Bad argument

12hrs is not a valid duration string.

short snow May 4, 2021, 4:19 PM

#

!remind 12h do it if not done yet

stable mountainBOT May 4, 2021, 4:19 PM

#

Sure thing!

Your reminder will arrive in 12 hours!

vale ibex May 4, 2021, 4:44 PM

#

Currently testing a bot PR but getting this error django.db.utils.OperationalError: FATAL: database "metricity" does not exist

#

I'm using the docker setup, are there migrations that need to be ran, or have I missed something?

gritty wind May 4, 2021, 4:44 PM

#

docker setup for the bot or metricity?

vale ibex May 4, 2021, 4:44 PM

#

bot

#

haven't setup anything special for metricity

#

Do I need to?

gritty wind May 4, 2021, 4:45 PM

#

Alright, just create the DB manually

#

What are you working on?

vale ibex May 4, 2021, 4:45 PM

#

Akarys's PR

#

https://github.com/python-discord/bot/pull/1549

GitHub

Nomination archive automation by Akarys42 · Pull Request #1549 · py...

This Pull Request adds automation to the #nomination-archive channel.
It adds a reaction handler to the talentpool cog restricted to the #nomination-voting channel :

🎫: unpin the message
incident ...

gritty wind May 4, 2021, 4:46 PM

#

I see

#

Easiest if you set up metricity, but you can also fake it

vale ibex May 4, 2021, 4:47 PM

#

should I just run this?

#

https://github.com/python-discord/site/blob/main/postgres/init.sql

GitHub

python-discord/site

pythondiscord.com - A Django and Bulma web application. - python-discord/site

#

the site is running, just with no metricity db

gritty wind May 4, 2021, 4:48 PM

#

I think that should work?

vale ibex May 4, 2021, 4:49 PM

#

yea that's done it

#

I wonder why it didn't get run on startup

sleek steppe May 4, 2021, 5:00 PM

#

[spring-cleanup a6cc40f] chore: Prefer double quotes over single quotes
 60 files changed, 430 insertions(+), 430 deletions(-)

vale ibex May 4, 2021, 5:02 PM

#

lol

sleek steppe May 4, 2021, 5:04 PM

#

big diff

fallen patrol May 4, 2021, 5:04 PM

#

Did you check that it didn't replace any of these didn't

#

@sleek steppe ^

sleek steppe May 4, 2021, 5:05 PM

#

I went through them one by one

fallen patrol May 4, 2021, 5:05 PM

#

Oh damn lol

sleek steppe May 4, 2021, 5:05 PM

#

I probably messed up somewhere though

fallen patrol May 4, 2021, 5:05 PM

#

~~Yeah probably~~

dry folio May 4, 2021, 5:07 PM

#

https://docs.google.com/document/d/1qIbapQUMLD40s5yzrqXZ7n0-6p1z3azpz1O_rPOWTv4 the cat devs made the list of math questions, would a reviewer mind looking if this is good or if we need to remove some? thanks a lot!!

Google Docs

slq math

generate linear systems of equations, two unknowns (dynamic) 2. what is the highest power of a biquadratic polynomial - 4 3. what is the formula for surface area of a sphere - 4pi2^2, 4πr^2 4. which theorem states that hypotenuse2 = base2 + height2 - Pythagorean 5. basic modular arithmetic add...

fervent sage May 4, 2021, 5:07 PM

#

ill take a look

dry folio May 4, 2021, 5:07 PM

#

fervent sage ill take a look

thank you!

dry folio May 4, 2021, 5:07 PM

#

fervent sage ill take a look

https://github.com/python-discord/sir-lancebot/issues/719 this is the issue btw

GitHub

Add "Math" and "Science" category in .quiz · Issue #719 · python-di...

Description Since the .quiz command currently only has the "General" category which is pretty much only geography and history questions, we propose to add "Math"...

patent pivot May 4, 2021, 5:11 PM

#

if anyone notices events dropped from #dev-log please give me a ping

#

we've had a couple of timeouts with the middleman worker so want to ensure it was just blips

vocal prairie May 4, 2021, 5:31 PM

#

Could we add scroll-behavior: smooth; to the site's css? That's the thing where when you scroll using a section id, it's doesn't jump there, it instead scrolls.

#

It's a little easier on the eyes if you have smooth scrolling

gritty wind May 4, 2021, 5:51 PM

#

Feel free to PR it

#

just remember to poly-fill

vocal prairie May 4, 2021, 5:51 PM

#

poly-fill?

gritty wind May 4, 2021, 5:52 PM

#

Scroll-behavior is not standard yet, and missing from some large browsers

#

namely safari and I think firefox

#

Polyfilling is just adding missing features

vocal prairie May 4, 2021, 5:52 PM

#

How do you do that?

gritty wind May 4, 2021, 5:53 PM

#

You're working on the main site?

vocal prairie May 4, 2021, 5:53 PM

#

The site repo? Yeah

#

A branch made off the main branch

gritty wind May 4, 2021, 5:54 PM

#

Not really sure how to do it in django

vocal prairie May 4, 2021, 5:55 PM

#

base.css is the right file to put it in, right?

gritty wind May 4, 2021, 5:55 PM

#

Poly filling isn't done in CSS. The property is added to the CSS, but the polyfilling is JS

vocal prairie May 4, 2021, 5:55 PM

#

Since that goes into base.html which is extended into most other files.

vocal prairie May 4, 2021, 5:55 PM

#

gritty wind Poly filling isn't done in CSS. The property is added to the CSS, but the polyfi...

I'm referring to the scroll behavior, not polyfill

gritty wind May 4, 2021, 5:56 PM

#

You need to poly fill to add this feature

vocal prairie May 4, 2021, 5:56 PM

#

Oh

#

So you don't put it in css?

gritty wind May 4, 2021, 5:56 PM

#

gritty wind Poly filling isn't done in CSS. The property is added to the CSS, but the polyfi...

^

vocal prairie May 4, 2021, 5:56 PM

#

Got it

gritty wind May 4, 2021, 5:57 PM

#

There are JS packages that do it for you because it's a lot of annoying code

#

But basically you can implement yourself by moving the page down one bit at a time

#

Of course that's a lot of implementation for very little gain

vocal prairie May 4, 2021, 5:58 PM

#

gritty wind There are JS packages that do it for you because it's a lot of annoying code

But we can't use those, right?

gritty wind May 4, 2021, 5:58 PM

#

We can't

vocal prairie May 4, 2021, 5:58 PM

#

Thought so

#

I have no idea how to polyfill. So I probably just won't pr it.

gritty wind May 4, 2021, 6:02 PM

#

It can still be added, it'll just... not work on some browsers

#

I checked, and it wasn't firefox, but IE that didn't support it

#

So that's Safari and IE mainly

#

But I don't really care about IE, or any of the smaller browsers

#

https://caniuse.com/css-scroll-behavior

vocal prairie May 4, 2021, 6:04 PM

#

IE is old now

gritty wind May 4, 2021, 6:05 PM

#

You'd be surprised how prevalent IE still is

vocal prairie May 4, 2021, 6:05 PM

#

Really?

gritty wind May 4, 2021, 6:05 PM

#

A lot of legacy systems just never upgrade

#

Hospitals, banks, anything government, etc

vocal prairie May 4, 2021, 6:06 PM

#

So should I try to figure out the polyfilling, pr without the polyfill, or just not pr?

vocal prairie May 4, 2021, 6:06 PM

#

gritty wind Hospitals, banks, anything government, etc

Interesting. I would have thought they would upgrade

gritty wind May 4, 2021, 6:06 PM

#

It's a pretty minor feature, so I don't care either way. Poly filling would be ideal, but I don't know how to do it in django

olive cloak May 4, 2021, 6:06 PM

#

not sure if its the right place to ask but on the python github and noticed something put this under their issue, does this mean no one else can work on it?

#

are can anyone work on it still?

gritty wind May 4, 2021, 6:07 PM

#

One of those boxes should have a checkmark beside it

olive cloak May 4, 2021, 6:07 PM

#

gritty wind May 4, 2021, 6:07 PM

#

Well, they messed that up

olive cloak May 4, 2021, 6:07 PM

#

so only they can work on it with the cat dev team

gritty wind May 4, 2021, 6:07 PM

#

But they want to add it

olive cloak May 4, 2021, 6:07 PM

#

ah right

gritty wind May 4, 2021, 6:07 PM

#

You can talk to them if you wish

vocal prairie May 4, 2021, 6:07 PM

#

olive cloak so only they can work on it with the `cat dev team`

They would definitely let you help.

olive cloak May 4, 2021, 6:07 PM

#

gritty wind You can talk to them if you wish

you can talk on github???

gritty wind May 4, 2021, 6:08 PM

#

Yeah? You can leave comments

olive cloak May 4, 2021, 6:08 PM

#

ohh right yeah

gritty wind May 4, 2021, 6:08 PM

#

Here too

vocal prairie May 4, 2021, 6:08 PM

#

olive cloak ohh right yeah

I can link you to the server they were discussing in if you wish. It's open

olive cloak May 4, 2021, 6:08 PM

#

what is the cat dev team also?

gritty wind May 4, 2021, 6:08 PM

#

A group of people that have cats in their name

#

you better not post that invite here, or you'll make the mods very happy

olive cloak May 4, 2021, 6:09 PM

#

vocal prairie I can link you to the server they were discussing in if you wish. It's open

they were talking about it in #ot0-psvm’s-eternal-disapproval right?, user "wait... its all objects?" and "Aboo"

patent pivot May 4, 2021, 6:09 PM

#

lol

gritty wind May 4, 2021, 6:09 PM

#

That's them

vocal prairie May 4, 2021, 6:09 PM

#

gritty wind you better not post that invite here, or you'll make the mods very happy

DMs, always

patent pivot May 4, 2021, 6:09 PM

#

@dry folio @wide elm hi

vocal prairie May 4, 2021, 6:09 PM

#

olive cloak they were talking about it in <#291284109232308226> right?, user "wait... its al...

mhm

wide elm May 4, 2021, 6:09 PM

#

hullo

olive cloak May 4, 2021, 6:09 PM

#

hi 👋

#

i commented another idea on that thread as well

wide elm May 4, 2021, 6:10 PM

#

Cool

wide elm May 4, 2021, 6:10 PM

#

gritty wind you better not post that invite here, or you'll make the mods very happy

lmao

dry folio May 4, 2021, 6:11 PM

#

gritty wind Well, they messed that up

i messed that up. sorry, not too familiar with what should i do if we're a group and not just an individual

patent pivot May 4, 2021, 6:12 PM

#

messing up wasn't working together, it was just a markdown mess up 🙂

dry folio May 4, 2021, 6:12 PM

#

patent pivot <@!769221607888912487> <@!416345453530054657> hi

hello joe!

patent pivot May 4, 2021, 6:12 PM

#

[ x] doesn't render as a checkmark, but [x] does iirc

dry folio May 4, 2021, 6:12 PM

#

patent pivot messing up wasn't working together, it was just a markdown mess up 🙂

oh ok lol

dry folio May 4, 2021, 6:13 PM

#

olive cloak i commented another idea on that thread as well

yes, would you like to be in our cat dev team?

patent pivot May 4, 2021, 6:13 PM

#

lol

olive cloak May 4, 2021, 6:14 PM

#

dry folio yes, would you like to be in our cat dev team?

yeah sure, not too familiar with Github tho

gritty wind May 4, 2021, 6:14 PM

#

patent pivot messing up wasn't working together, it was just a markdown mess up 🙂

I didn't even know what issue this was lol

dry folio May 4, 2021, 6:14 PM

#

olive cloak yeah sure, not too familiar with Github tho

i sent you it in dms

dry folio May 4, 2021, 6:14 PM

#

gritty wind I didn't even know what issue this was lol

https://github.com/python-discord/sir-lancebot/issues/719 this one :D

GitHub

Add "Math" and "Science" category in .quiz · Issue #719 · python-di...

Description Since the .quiz command currently only has the "General" category which is pretty much only geography and history questions, we propose to add "Math"...

wide elm May 4, 2021, 6:15 PM

#

lol

sleek steppe May 4, 2021, 6:36 PM

#

This code confuses me, https://github.com/python-discord/bot/blob/main/bot/exts/info/reddit.py#L117, (~~even though I'm testing it on sir-lancebot~~). If I get a 200 (because I do), is it supposed to fail?

stable mountainBOT May 4, 2021, 6:36 PM

#

bot/exts/info/reddit.py line 117

if response.status == 204 and response.content_type == "application/json":```

vale ibex May 4, 2021, 6:52 PM

#

https://tools.ietf.org/html/rfc7009 I'm guessing

#

OAuth token revoking should return a 204

tawdry vapor May 4, 2021, 6:52 PM

#

Weird. Cause here it does actually say 200 https://tools.ietf.org/html/rfc7009#section-2.2

#

If I understand correctly, it's saying it'd return a 200 for an invalid token too? WTF why

vale ibex May 4, 2021, 6:53 PM

#

wut

#

https://github.com/reddit-archive/reddit/wiki/OAuth2#manually-revoking-a-token

#

Note: Per RFC 7009, this request will return a success (204) response even if the passed in token was never valid.

#

is this just wrong?

tawdry vapor May 4, 2021, 6:54 PM

#

Well we would be getting that warning in prod then, wouldn't we?

vale ibex May 4, 2021, 6:54 PM

#

looking

#

nothing in the last 7days

#

or last 30

#

shrugR

#

I wonder if we generated a permanent?

#

it's temp in code though..

tawdry vapor May 4, 2021, 6:56 PM

#

Maybe relevant https://www.reddit.com/r/redditdev/comments/kvzaot/oauth2_api_changes_upcoming/

r/redditdev - OAuth2 API Changes Upcoming

60 votes and 52 comments so far on Reddit

#

Other libraries seem to expect a 204 currently https://github.com/draw-dev/DRAW/blob/master/lib/src/auth.dart#L116-L121

stable mountainBOT May 4, 2021, 6:57 PM

#

lib/src/auth.dart lines 116 to 121

  if (response.statusCode != 204) {
    // We should always get a 204 response for this call.
    final parsed = json.decode(response.body);
    _throwAuthenticationError(parsed);
  }
}```

gritty wind May 4, 2021, 6:57 PM

#

Not returning useful information for token revokation is pretty standard

#

I guess to prevent people from using revocation endpoints to guess tokens or something?

tawdry vapor May 4, 2021, 6:58 PM

#

That's not the problem

#

The problem is that the spec says it should be 200 but reddit says 204, and then someone is getting a 200 anyway

gritty wind May 4, 2021, 6:58 PM

#

Uhh...

#

generate a token and test?

#

I can give it a shot

#

Once I figure out how to de-uglify reddit's docs

tawdry vapor May 4, 2021, 7:05 PM

#

PRAW also expects 204 https://github.com/praw-dev/prawcore/blob/master/prawcore/auth.py#L83-L96

stable mountainBOT May 4, 2021, 7:05 PM

#

prawcore/auth.py lines 83 to 96

def revoke_token(self, token, token_type=None):
    """Ask Reddit to revoke the provided token.

    :param token: The access or refresh token to revoke.
    :param token_type: (Optional) When provided, hint to Reddit what the token type
        is for a possible efficiency gain. The value can be either ``access_token``
        or ``refresh_token``.

    """
    data = {"token": token}
    if token_type is not None:
        data["token_type_hint"] = token_type
    url = self._requestor.reddit_url + const.REVOKE_TOKEN_PATH
    self._post(url, success_status=codes["no_content"], **data)```

vale ibex May 4, 2021, 7:24 PM

#

.ext unload latex

dusky shoreBOT May 4, 2021, 7:24 PM

#

:ok_hand: Extension successfully unloaded: bot.exts.evergreen.latex.

gritty wind May 4, 2021, 7:55 PM

#

@tawdry vapor @vale ibex finally had a chance to test it

#

Can confirm it's a 200

#

The guide is in fact, incorrect

#

They reference the standard, so I'm guessing it's a miscommunication or a typo

#

Though

#

If we're worried about it changing

#

It's easy enough to test it on startup

sleek steppe May 4, 2021, 8:03 PM

#

Would response.status in (200, 204) be fine?

#

https://discordapp.com/channels/267624335836053506/635950537262759947/839208977447649281 token removals are documented to return a 204 response on success yet it is returning a 200

tawdry vapor May 4, 2021, 8:11 PM

#

Popular libraries only check for 204 and I didn't see any users reporting an issue about a wrong status code

#

It might have to do with the recent changes I linked to.

#

Might be worth making a reddit post there to ask, since the devs seem to watch that sub.

thorny obsidian May 4, 2021, 10:13 PM

#

Hi, selfish request if anyone is up for (what I hope is) not too hard work: https://github.com/python-discord/bot/issues/1563

GitHub

When a duration is specified for !ban, it should become a tempban ·...

Proposal When a duration is specified for !ban like: !ban <id> 3D You are now banned. It will become a temporary ban for the specified duration. !ban with no provided duration: !ban &...

clever wraith May 4, 2021, 10:29 PM

#

vale ibex .ext unload latex

Lmao we should just pin this at this point

short snow May 5, 2021, 12:31 AM

#

thorny obsidian Hi, selfish request if anyone is up for (what I hope is) not too hard work: http...

i will do it

short snow May 5, 2021, 12:50 AM

#

anyone free, need to test this feature?

vocal prairie May 5, 2021, 1:04 AM

#

short snow anyone free, need to test this feature?

I'm always up for being banned

sleek steppe May 5, 2021, 1:21 AM

#

I'm not sure about this change, tempban and ban would do the exact same thing except duration is optional for ban

vocal prairie May 5, 2021, 1:21 AM

#

It's the same situation as mute automatically becoming tempmute

sleek steppe May 5, 2021, 1:22 AM

#

Mute is an alias for the tempmute command

vocal prairie May 5, 2021, 1:22 AM

#

I didn't know that.

#

Oh

#

So why not do the same thing for ban, and make it an alias for tempban?

#

Or is there a reason it's not?

thorny obsidian May 5, 2021, 1:23 AM

#

The mods really want this change. It'll be a very nice QoL thing for us.

sleek steppe May 5, 2021, 1:23 AM

#

vocal prairie So why not do the same thing for ban, and make it an alias for tempban?

ban can be permanent, mute can't

vocal prairie May 5, 2021, 1:23 AM

#

sleek steppe ban can be permanent, mute can't

Oh yeah

#

But it's the same situation

#

Actually, no it's not

#

Ignore me

short snow May 5, 2021, 1:24 AM

#

a ban with no expiration is a permanent ban

#

so that could be done

vocal prairie May 5, 2021, 1:24 AM

#

What's the default length for !tempban?

sleek steppe May 5, 2021, 1:25 AM

#

no default

vocal prairie May 5, 2021, 1:25 AM

#

So then, why not make ban an alias? And when no time is provided, it defaults to permanent.

short snow May 5, 2021, 1:25 AM

#

no status embeds sent in #dev-log pithink

#

yeah

vocal prairie May 5, 2021, 1:25 AM

#

short snow no status embeds sent in <#622895325144940554> <:pithink:652247559909277706>

Wdym status?

short snow May 5, 2021, 1:26 AM

#

#dev-log message

#

two runs

vocal prairie May 5, 2021, 1:26 AM

#

If some embeds aren't showing, joe said to ping him

#

Just make sure it's an actual error

sleek steppe May 5, 2021, 1:27 AM

#

It's not an error, there were two runs

short snow May 5, 2021, 1:27 AM

#

not a problem with that, dunno but i see 3 checks on this pr, while there were only 2 commits

vocal prairie May 5, 2021, 1:27 AM

#

vocal prairie So then, why not make ban an alias? And when no time is provided, it defaults to...

Does this make sense, or would this be bad in some way?

short snow May 5, 2021, 1:28 AM

#

would be good, since basically ban, with this pr does that

vocal prairie May 5, 2021, 1:28 AM

#

@thorny obsidian Is that something that would do what you wanted?

short snow May 5, 2021, 1:29 AM

#

maybe ask zig, he is the moderation lead 🤷‍♂️

thorny obsidian May 5, 2021, 1:38 AM

#

I think that's fine. Zig's a codeowner for the mod cogs iirc so he'll be requested to have a review.

short snow May 5, 2021, 2:21 AM

#

vocal prairie It's sending errors like: `{"detail":"Invalid option provided: lightsaber.png no...

quackstack#40 will fix this
cc: @clever wraith

dusky shoreBOT May 5, 2021, 2:21 AM

#

GitHub

PROpen (hotfix): Correct order of ProceduralDucky namedtuple

clever wraith May 5, 2021, 2:21 AM

#

Oh yhanks!

#

thanks

#

and you guys already know bills are kinda broken atm right

vocal prairie May 5, 2021, 2:23 AM

#

Yeah, they know

#

They layering was off

clever wraith May 5, 2021, 2:23 AM

#

ah

short snow May 5, 2021, 2:27 AM

#

there was a fix already done for that, lemme ahve a look

#

yep its fixed now

#

fallen patrol May 5, 2021, 4:01 AM

#

clever wraith Lmao we should just pin this at this point

Off topic name 😳

fallen patrol May 5, 2021, 4:02 AM

#

thorny obsidian The mods *really* want this change. It'll be a very nice QoL thing for us.

Mutes could be permanent, no?

fallen patrol May 5, 2021, 4:02 AM

#

sleek steppe ban can be permanent, mute can't

Oh wrong reply, meant this fro the above msg.

fallen patrol May 5, 2021, 4:04 AM

#

short snow

What the duck.

sleek steppe May 5, 2021, 4:05 AM

#

fallen patrol Mutes could be permanent, no?

Defaulted to one hour, so no

#

Which is why mute is just an alias to tempmute

fallen patrol May 5, 2021, 4:05 AM

#

Ah

placid ermine May 5, 2021, 4:11 AM

#

fallen patrol Off topic name 😳

!otn a latext unload 😳

stable mountainBOT May 5, 2021, 4:19 AM

#

@short snow

It has arrived!

Here's your reminder: do it if not done yet.
[Jump back to when you created the reminder](#dev-contrib message)

short snow May 5, 2021, 4:29 AM

#

umm, i got three pings from here, where is the third one?

fallen patrol May 5, 2021, 4:52 AM

#

short snow umm, i got three pings from here, where is the third one?

~~a serious typo that the edited tag would make look real~~

fallen patrol May 5, 2021, 4:52 AM

#

fallen patrol What the duck.

Autocorrect legit changed the d to an f in this message

short snow May 5, 2021, 6:29 AM

#

lol

cold island May 5, 2021, 7:45 AM

#

vocal prairie So then, why not make ban an alias? And when no time is provided, it defaults to...

Tempban should error if no duration is provided (e.g forgot). Tempban dealing a non-temp ban doesn't sound like great ux.

The issue was that sometimes we might accidentally type ban instead of tempban, so not accidentally permanently banning someone when a duration is specified is a nice qol feature.

stable mountainBOT May 5, 2021, 8:55 AM

#

Identity vs. Equality

Should I be using is or ==?

To check if two things are equal, use the equality operator (==).

x = 5
if x == 5:
    print("x equals 5")
if x == 3:
    print("x equals 3")
# Prints 'x equals 5'

To check if two things are actually the same thing in memory, use the identity comparison operator (is).

x = [1, 2, 3]
y = [1, 2, 3]
if x is [1, 2, 3]:
    print("x is y")
z = x
if x is z:
    print("x is z")
# Prints 'x is z'

vocal wolf May 5, 2021, 8:55 AM

#

@fervent sage noice noice

patent pivot May 5, 2021, 8:55 AM

#

that is such a horrid branch name LOL

vocal wolf May 5, 2021, 8:55 AM

#

ikr

patent pivot May 5, 2021, 8:55 AM

#

pain

fervent sage May 5, 2021, 8:55 AM

#

patent pivot that is such a horrid branch name LOL

i was checking it out

#

and i made 4 typos

patent pivot May 5, 2021, 8:55 AM

#

lmfao

fervent sage May 5, 2021, 8:56 AM

#

identity-vs-equality would have been nicer kek

#

holup

#

!e

x = [1, 2, 3]
y = [1, 2, 3]
if x is [1, 2, 3]:
    print("x is y")
z = x
if x is z:
    print("x is z")
# Prints 'x is z'

stable mountainBOT May 5, 2021, 8:58 AM

#

@fervent sage :white_check_mark: Your eval job has completed with return code 0.

x is z

vocal wolf May 5, 2021, 8:58 AM

#

HMMM

fervent sage May 5, 2021, 8:58 AM

#

ok im just going insane, noted

vocal wolf May 5, 2021, 8:59 AM

#

yeah looks good lol

fervent sage May 5, 2021, 8:59 AM

#

if x is [1, 2, 3]:
    print("x is y")
```i think this bit actually confuses me more

#

cause thats checking is x is the [] not y, which are ofc identical anyway but it seems odd

vocal wolf May 5, 2021, 8:59 AM

#

Also using the integer 420 would be less noisy than [1, 2, 3] and keep the same behavior.
@green oriole where 420? /s

fervent sage May 5, 2021, 8:59 AM

#

^ im more than happy to put in 420 but idk where kek

green oriole May 5, 2021, 9:00 AM

#

Instead of the list

#

Like if variable is 420

#

Oh lol, that was sacarstic

vocal wolf May 5, 2021, 9:00 AM

#

if variable is 1, yes of course ~~this joke is terrible, I'm sorry~~

fervent sage May 5, 2021, 9:00 AM

#

sadge

#

ok so uh, what out of

This snippet could get more explicit variable names too, it is a bit hard to follow right now. Also using the integer 420 would be less noisy than [1, 2, 3] and keep the same behavior.
isnt sarcastic?

green oriole May 5, 2021, 9:01 AM

#

all of it lol

#

What about just [420] then @cold island

fervent sage May 5, 2021, 9:02 AM

#

this is why i shouldnt program right after waking up kek

anyways, what var names do you think would fit better?

#

that being said before i saw xith messaged me i was about to dive into css so that would have been far worse lmao

green oriole May 5, 2021, 9:03 AM

#

x => first_list
y => second_list
z => reference_to_first_list
maybe? the last one is a bit long but it is at least explicit

fervent sage May 5, 2021, 9:04 AM

#

how about list_1 list_2 reference_to_list_1

#

a little more concise

green oriole May 5, 2021, 9:04 AM

#

Sounds good

fervent sage May 5, 2021, 9:06 AM

#

"list_1 is reference_to_list_1" sounds a bit weird, should it maybe be "list_1 is identical to refer..."

#

actually that doesnt sound better

vocal wolf May 5, 2021, 9:15 AM

#

#

the holy grail of reviews

green oriole May 5, 2021, 9:16 AM

#

Haha

stable mountainBOT May 5, 2021, 9:16 AM

#

Identity vs. Equality

Should I be using is or ==?

To check if two objects are equal, use the equality operator (==).

x = 5
if x == 5:
    print("x equals 5")
if x == 3:
    print("x equals 3")
# Prints 'x equals 5'

To check if two objects are actually the same thing in memory, use the identity comparison operator (is).

list_1 = [1, 2, 3]
list_2 = [1, 2, 3]
if list_1 is [1, 2, 3]:
    print("list_1 is list_2")
reference_to_list_1 = list_1
if list_1 is reference_to_list_1:
    print("list_1 is reference_to_list_1")
# Prints 'list_1 is reference_to_list_1'

green oriole May 5, 2021, 9:17 AM

#

... I am disqualified from the review count because I made a merge commit

vocal wolf May 5, 2021, 9:17 AM

#

noice noice

vocal wolf May 5, 2021, 9:17 AM

#

stable mountain

@cold island thoughts?

short snow May 5, 2021, 9:19 AM

#

cold island Tempban should error if no duration is provided (e.g forgot). Tempban dealing a ...

Right, I will just update the doc-strings then.

green oriole May 5, 2021, 9:19 AM

#

https://github.com/python-discord/.github/pull/3 please 🥺

GitHub

Add the ignore update merged option to tag speedrun by Akarys42 · P...

patent pivot May 5, 2021, 9:22 AM

#

green oriole https://github.com/python-discord/.github/pull/3 please 🥺

greenTick

green oriole May 5, 2021, 9:23 AM

#

Thanko

short snow May 5, 2021, 9:49 AM

#

What does that PR do?

green oriole May 5, 2021, 9:50 AM

#

It will still count staff approval even if you click the update branch button on github

short snow May 5, 2021, 9:51 AM

#

Ah ok 👍

vocal wolf May 5, 2021, 10:48 AM

#

@balmy sparrow @vale ibex did y'all test bot#1441?

dusky shoreBOT May 5, 2021, 10:48 AM

#

GitHub

PROpen Unescape html escape characters in reddit text and titles

vale ibex May 5, 2021, 10:49 AM

#

I tested pre the last commit, and the last commit didn't affect function imo

vocal wolf May 5, 2021, 10:50 AM

#

merged

#

mmmmmm down to 16 PRs

#

I like this

#

can we get 15

fervent sage May 5, 2021, 10:51 AM

#

whats the smallest one that wont take me an hour to review kek

balmy sparrow May 5, 2021, 10:51 AM

#

vocal wolf <@!271586885346918400> <@!126811506632294400> did y'all test bot#1441?

i didn't review this though

vale ibex May 5, 2021, 10:51 AM

#

t'was vco lul

fervent sage May 5, 2021, 10:51 AM

#

did you mean to

vocal wolf May 5, 2021, 10:51 AM

#

oh, whoops

fervent sage May 5, 2021, 10:51 AM

#

ye

vocal wolf May 5, 2021, 10:51 AM

#

pinged wrong person lol

vale ibex May 5, 2021, 10:51 AM

#

smh xith bot

vocal wolf May 5, 2021, 10:51 AM

#

D:

vale ibex May 5, 2021, 10:51 AM

#

someone open a pr

fervent sage May 5, 2021, 10:51 AM

#

lol

vocal wolf May 5, 2021, 10:52 AM

#

fervent sage whats the smallest one that wont take me an hour to review <:kek:585617800257863...

what about one that would take 59 minutes?

balmy sparrow May 5, 2021, 10:52 AM

#

oh, i have some time too now

#

i'd began going through that spring cleaning pr

fervent sage May 5, 2021, 10:52 AM

#

balmy sparrow i'd began going through that spring cleaning pr

pls continue :D :P

balmy sparrow May 5, 2021, 10:53 AM

#

but i gave up at 18/80 files

vocal wolf May 5, 2021, 10:53 AM

#

I'd really like Reddit migration to just be done and over with

#

I'm not sure if it should take too long

#

I'll acquire the links

balmy sparrow May 5, 2021, 10:53 AM

#

alright, i'll try to get to it now

vocal wolf May 5, 2021, 10:54 AM

#

So we're moving the Reddit cog from bot#1542 to sir-lancebot#569

dusky shoreBOT May 5, 2021, 10:54 AM

#

GitHub

PROpen Annihilate reddit cog
PROpen Reddit migration

vocal wolf May 5, 2021, 10:54 AM

#

@fervent sage there's something that you can review lol

fervent sage May 5, 2021, 10:55 AM

#

will do soon™️

vocal wolf May 5, 2021, 10:55 AM

#

thank

short snow May 5, 2021, 10:56 AM

#

My incidents enhancement is also up for review

#

Just need to mention a embed look change

vocal wolf May 5, 2021, 10:56 AM

#

It's currently 4am, if I suddenly become inactive, that means I've fallen asleep

short snow May 5, 2021, 10:56 AM

#

lemon_sleepy

vocal wolf May 5, 2021, 10:57 AM

#

short snow My incidents enhancement is also up for review

Eventually.

fervent sage May 5, 2021, 10:57 AM

#

eventually

vocal wolf May 5, 2021, 10:58 AM

#

As triage lead I command you all to go work on Reddit migration /s

short snow May 5, 2021, 10:58 AM

#

Yeah, just telling

vocal wolf May 5, 2021, 10:58 AM

#

Or else, death by poking.

cold island May 5, 2021, 11:19 AM

#

vocal wolf <@!512354988157173763> thoughts?

oh wow, I'll stop reviewing on mobile, I missed that change

#

it looks really noisy tbh

vocal wolf May 5, 2021, 11:20 AM

#

I agree, to some extent

#

wait it got merged?

#

oh, it did

#

we can always make another PR to clean it up

short snow May 5, 2021, 12:02 PM

#

🤦‍♂️ made a commit called mend

#

force pushes ducky_devil

vale ibex May 5, 2021, 12:07 PM

#

git reset --soft HEAD~1

#

unless you already pushed

vocal wolf May 5, 2021, 12:08 PM

#

@cold island as you are moderation lead, I need a review from you on bot#1564.

dusky shoreBOT May 5, 2021, 12:08 PM

#

GitHub

PROpen When duration specified for !ban, apply a !tempban

short snow May 5, 2021, 12:11 PM

#

following xithrus, helps alot 🙃

#

he has got stars on some really good repos

vocal wolf May 5, 2021, 12:12 PM

#

I do?

#

huh

#

thanks lol

green oriole May 5, 2021, 12:17 PM

#

Hah thanks for the link Chris, was about to ask for that

short snow May 5, 2021, 12:17 PM

#

this one

vocal wolf May 5, 2021, 12:17 PM

#

short snow this one

oh yeah that's some good stuff, gonna set up an alias soon so I can get beautiful diffs

short snow May 5, 2021, 12:18 PM

#

already did

green oriole May 5, 2021, 12:19 PM

#

iirc it is just textwrap pretty much sucking

vocal wolf May 5, 2021, 12:19 PM

#

not with a git config, but instead with a one liner

#

(unless that's not an option)

green oriole May 5, 2021, 12:20 PM

#

I feel like this will end up with a custom partition function

sleek steppe May 5, 2021, 12:30 PM

#

vocal wolf (unless that's not an option)

~~Have you gotten to testing my PR yet?~~

vocal wolf May 5, 2021, 12:31 PM

#

which PR was that?

#

I've forgotten lol

sleek steppe May 5, 2021, 12:31 PM

#

The one that redirect's eval output

vocal wolf May 5, 2021, 12:32 PM

#

oh right

#

that one lol

#

lol I self-requested a review and proceeded to completely forget about it

short snow May 5, 2021, 12:33 PM

#

looks ok @cold island ?

vocal wolf May 5, 2021, 12:33 PM

#

@sleek steppe do you think you could include a screenshot of what it currently does?

#

since it doesn't happen often, I think it'd be worth it

short snow May 5, 2021, 12:34 PM

#

max

sleek steppe May 5, 2021, 12:35 PM

#

vocal wolf May 5, 2021, 12:35 PM

#

oh

#

right, that makes sense lol

#

that should've been obvious to me

#

anyway

#

@sleek steppe I've never actually had to test the eval command before, did you clone the repo and start it up along with the bot, or just do docker-compose up on the bot repo?

sleek steppe May 5, 2021, 12:40 PM

#

I mean I cloned it and did docker-compose up

vocal wolf May 5, 2021, 12:40 PM

#

for snekbox?

cold island May 5, 2021, 12:41 PM

#

vocal wolf <@!512354988157173763> as you are moderation lead, I need a review from you on b...

I can't really test it right now, but did someone check it perma bans without a duration?

vocal prairie May 5, 2021, 12:41 PM

#

Yeah

#

Jason tested it on me

cold island May 5, 2021, 12:42 PM

#

hmm.. I'm wondering what to do about pban now...

short snow May 5, 2021, 12:42 PM

#

vocal wolf <@!506618674921340958> I've never actually had to test the eval command before, ...

yes that would work. no need to clone snekbox

#

but you would need to edit the config.yml

#

one sec

vocal wolf May 5, 2021, 12:42 PM

#

right, that makes sense

#

I'd think I just have an incorrect var in there

#

prob the api url

cold island May 5, 2021, 12:44 PM

#

short snow looks ok <@!512354988157173763> ?

mmm how does it look with max length?

vocal wolf May 5, 2021, 12:44 PM

#

cold island hmm.. I'm wondering what to do about pban now...

I think we were going to hold off on that given that if we wanted to purge for an amount of days and tempban would lead to confusion between command args.

At the same time I don't see us doing anything besides 1d purges, except in very rare cases.

#

Something something slash commands

short snow May 5, 2021, 12:45 PM

#

cold island mmm how does it look with max length?

we already max legth it at 300

short snow May 5, 2021, 12:45 PM

#

vocal wolf I'd think I just have an incorrect var in there

#

i had it pinned in my test server, the docker config

vocal wolf May 5, 2021, 12:45 PM

#

that font lol

sleek steppe May 5, 2021, 12:46 PM

#

short snow i had it pinned in my test server, the docker config

Didn't that get deleted

cold island May 5, 2021, 12:46 PM

#

@vocal wolf @vocal prairie @short snow I approved it, though it creates inconsistency with pban and vban

vocal wolf May 5, 2021, 12:46 PM

#

yeah

cold island May 5, 2021, 12:46 PM

#

short snow we already max legth it at 300

yeah I'm just asking how it looks like at 300+

short snow May 5, 2021, 12:46 PM

#

one sec

vocal wolf May 5, 2021, 12:47 PM

#

cold island <@!196664644113268736> <@!788119126450765844> <@!736234578745884682> I approved ...

I'll get back to you on that when I wake up.

vocal prairie May 5, 2021, 12:47 PM

#

Night Xithrius

short snow May 5, 2021, 12:47 PM

#

sleek steppe Didn't that get deleted

not deleted, just a removal of all, add emojis and configuring the ids took time

sleek steppe May 5, 2021, 12:47 PM

#

😔

vocal wolf May 5, 2021, 12:47 PM

#

vocal prairie Night Xithrius

👋

#

I'm testing Toxic's PR first tho

sleek steppe May 5, 2021, 12:49 PM

#

My goal is to get 1k additions/deletions on the spring cleanup PR

short snow May 5, 2021, 12:50 PM

#

cold island yeah I'm just asking how it looks like at 300+

cold island May 5, 2021, 12:53 PM

#

Sorry, I know I'm nitpicking at this point, but I don't like that it creates another line to say that it was truncated

vocal wolf May 5, 2021, 12:53 PM

#

@sleek steppe Does redirection usually ping twice?

#

for other things

thorny obsidian May 5, 2021, 12:54 PM

#

cold island Sorry, I know I'm nitpicking at this point, but I don't like that it creates ano...

should we have vban operate similarly? Where if you specify a time it doesn't perma vban?

cold island May 5, 2021, 12:54 PM

#

cold island Sorry, I know I'm nitpicking at this point, but I don't like that it creates ano...

Maybe just append " [...]" in all truncation cases

cold island May 5, 2021, 12:54 PM

#

thorny obsidian should we have vban operate similarly? Where if you specify a time it doesn't pe...

You probably didn't mean to reply to this message, but yes this is what I'm saying

thorny obsidian May 5, 2021, 12:54 PM

#

(maybe I'll stop getting that command wrong too if it behaves like that >_>)

sleek steppe May 5, 2021, 12:55 PM

#

vocal wolf <@!506618674921340958> Does redirection usually ping twice?

No because eval pings and help doesn't

thorny obsidian May 5, 2021, 12:55 PM

#

dammit discord web app, no. I did not mean to reply to that message, I mean to just mention

vocal wolf May 5, 2021, 12:55 PM

#

ah

sleek steppe May 5, 2021, 12:55 PM

#

Maybe a ping_user kwarg for redirect output?

vocal wolf May 5, 2021, 12:55 PM

#

uhhhhhhhhhh

short snow May 5, 2021, 12:55 PM

#

cold island Maybe just append " [...]" in all truncation cases

that would be good, i will do that

vocal wolf May 5, 2021, 12:56 PM

#

sleek steppe Maybe a ping_user kwarg for redirect output?

ye

cold island May 5, 2021, 12:56 PM

#

thorny obsidian (maybe I'll stop getting that command wrong too if it behaves like that >_>)

The only issue I have is that with pban you already have a duration argument, but for the deletion

vocal wolf May 5, 2021, 12:56 PM

#

@sleek steppe I'll check on it when I wake, have a night.

cold island May 5, 2021, 12:56 PM

#

Which is why we don't have a ptempban so far

sleek steppe May 5, 2021, 12:56 PM

#

gn

vocal wolf May 5, 2021, 12:56 PM

#

but so far it looks good

thorny obsidian May 5, 2021, 12:56 PM

#

cold island The only issue I have is that with pban you already have a duration argument, bu...

oh wait, really? I thought pban was just all messages

cold island May 5, 2021, 12:56 PM

#

nope

vocal wolf May 5, 2021, 12:56 PM

#

thorny obsidian oh wait, really? I thought pban was just all messages

default is 1d, max 7

cold island May 5, 2021, 12:56 PM

#

!help pban

stable mountainBOT May 5, 2021, 12:56 PM

#

Command Help

!purgeban <user> [purge_days=1] [reason]
Can also use: pban

*Same as ban but removes all their messages for the given number of days, default being 1.

purge_days can only be values between 0 and 7.
Anything outside these bounds are automatically adjusted to their respective limits.*

sleek steppe May 5, 2021, 12:58 PM

#

vocal wolf ye

Would would it say if ping_user is false?

thorny obsidian May 5, 2021, 12:58 PM

#

Would we ever tempban and purge?

vocal wolf May 5, 2021, 12:58 PM

#

sleek steppe Would would it say if ping_user is false?

see how it looks, I guess

#

my mind is tired, responses aren't very good

cold island May 5, 2021, 1:00 PM

#

@vocal wolf go sleep

thorny obsidian May 5, 2021, 1:00 PM

#

My thoughts:
ban and vban should allow for optional duration arguments. If provided, it becomes temp.
pban should stay the same because we should only use that command in urgent situations where we need to clean and permaban quickly because the messages are that bad. pban should be the outlier because of the severity that requires its usage.

sleek steppe May 5, 2021, 1:01 PM

#

maybe await ctx.send(..., allowed_mentions=AllowedMentions(users=ping_user))

short snow May 5, 2021, 1:02 PM

#

thorny obsidian My thoughts: ban and vban should allow for optional duration arguments. If provi...

not a moderator, but I agree

fervent sage May 5, 2021, 1:04 PM

#

not a mod but i also agree

deft patrol May 5, 2021, 1:04 PM

#

Same.

cold island May 5, 2021, 1:14 PM

#

short snow not a moderator, but I agree

feel free to add it to vban in the same pr

#

although you don't have to for it to be merged

short snow May 5, 2021, 1:22 PM

#

ok cool will do

sleek steppe May 5, 2021, 1:27 PM

#

I used allowed_mentions so you could still see it's talking to you

gritty wind May 5, 2021, 1:29 PM

#

Is this for the #bot-commands redirection issue?

#

If so, can you just remove the first message?

#

Seems redundant

sleek steppe May 5, 2021, 1:31 PM

#

Makes sense

dry folio May 5, 2021, 2:45 PM

#

hello! does anyone know how do i clone the sir-lancebot for testing? i did
git clone https://github.com/cat-dev-group/sir-lancebot but i'm not sure what to do next

clever wraith May 5, 2021, 2:47 PM

#

Contributing guide

vocal prairie May 5, 2021, 2:47 PM

#

https://pythondiscord.com/pages/contributing/sir-lancebot/

Python Discord | Contributing to Sir Lancebot

We're a large, friendly community focused around the Python programming language. Our community is open to those who wish to learn the language, as well as those looking to help others.

clever wraith May 5, 2021, 2:47 PM

#

this

sleek steppe May 5, 2021, 2:47 PM

#

Wait is that an org lol

fallen patrol May 5, 2021, 2:50 PM

#

sleek steppe Wait is that an org lol

<

vale ibex May 5, 2021, 2:56 PM

#

Could someone add a cooldown on .catify to avoid situations like this? #bot-commands message

#

User bucket is fine, with 5seconds

sleek steppe May 5, 2021, 2:57 PM

#

Sure I could do that

fervent sage May 5, 2021, 4:04 PM

#

thanks github... ???

patent pivot May 5, 2021, 4:05 PM

#

lfmao

short snow May 5, 2021, 4:09 PM

#

Weird, happens on IOS

stable mountainBOT May 5, 2021, 4:09 PM

#

@green oriole

It has arrived!

Here's your reminder: Suggest to use the Redis container on the Python docs.
[Jump back to when you created the reminder](#dev-contrib message)

fervent sage May 5, 2021, 4:10 PM

#

short snow Weird, happens on IOS

mobile's embed rendering in general is ??!!?!??!!!?

balmy sparrow May 5, 2021, 4:10 PM

#

discord in general is ??!?!

short snow May 5, 2021, 4:10 PM

#

Unpredictable

#

God, when would discord fix this

sleek steppe May 5, 2021, 4:11 PM

#

fervent sage thanks github... ???

lmao I was wondering why you appproved after it was merged

short snow May 5, 2021, 4:11 PM

#

Alec there is a pr on quackstack

#

Really really short one

#

5 seconds most

fervent sage May 5, 2021, 4:12 PM

#

quackstack#38

dusky shoreBOT May 5, 2021, 4:12 PM

#

GitHub

PRMerged Move bill to the lowest layer.

fervent sage May 5, 2021, 4:12 PM

#

ik it isnt that one

#

thats just the fastest way for me to get to the repo kek

short snow May 5, 2021, 4:12 PM

#

Just to get the link ik

#

Indeed, that was the fastest approve

fervent sage May 5, 2021, 4:13 PM

#

ngl im proud of the sentence i just wrote

balmy sparrow May 5, 2021, 4:14 PM

#

unfucky the ducky should be an otn

patent pivot May 5, 2021, 4:17 PM

#

unrelated but related note: I forked this bitnami project to the org and tinkered with it to add discord support, it's pretty nice https://github.com/python-discord/kubewatch/

GitHub

python-discord/kubewatch

Watch k8s events and trigger Handlers. Contribute to python-discord/kubewatch development by creating an account on GitHub.

short snow May 5, 2021, 4:18 PM

#

That’s nice 😊

patent pivot May 5, 2021, 4:18 PM

#

just a shame it's golang mmLul

short snow May 5, 2021, 4:19 PM

#

lemon_pensive

sleek steppe May 5, 2021, 4:22 PM

#

patent pivot unrelated but related note: I forked this bitnami project to the org and tinkere...

https://github.com/python-discord/kubewatch/blob/master/pkg/handlers/discord/discord.go#L23 missed a \ in .com

stable mountainBOT May 5, 2021, 4:22 PM

#

pkg/handlers/discord/discord.go line 23

var DISCORD_HOOK = regexp.MustCompile(`https?://(?:(?:canary|ptb)\.)?discord(?:app)?.com/api/webhooks/(?P<id>\d+)/(?P<token>.*)`)```

green oriole May 5, 2021, 4:30 PM

#

stable mountain <@407110650217627658>

!remind 1M maybe dewiki is done by then?

stable mountainBOT May 5, 2021, 4:30 PM

#

Absolutely!

Your reminder will arrive in 1 minute!

sleek steppe May 5, 2021, 4:31 PM

#

It's gonna be done in a minute?

stable mountainBOT May 5, 2021, 4:31 PM

#

@green oriole

It has arrived!

Here's your reminder: maybe dewiki is done by then?.
[Jump back to when you created the reminder](#dev-contrib message)

green oriole May 5, 2021, 4:32 PM

#

Lmao

#

Okay

fallen patrol May 5, 2021, 4:32 PM

#

green oriole !remind 1M maybe dewiki is done by then?

lmfao

green oriole May 5, 2021, 4:32 PM

#

Well

fallen patrol May 5, 2021, 4:32 PM

#

minutes-is-not-months

green oriole May 5, 2021, 4:32 PM

#

!remind 1m Try again Akarys

stable mountainBOT May 5, 2021, 4:32 PM

#

ROGER THAT

Your reminder will arrive in 1 month!

fallen patrol May 5, 2021, 4:32 PM

#

green oriole !remind 1M maybe dewiki is done by then?

#

i was actually rewriting the time parser for a different bot, aka stealing the parser from this bot

vocal wolf May 5, 2021, 4:34 PM

#

@sleek steppe tested thing. Works well.

fallen patrol May 5, 2021, 4:34 PM

#

~~which is just the dateutil library but still lol~~

green oriole May 5, 2021, 4:35 PM

#

fallen patrol

@patent pivot ye wise man lemon_pensive

patent pivot May 5, 2021, 4:35 PM

#

🥸

fallen patrol May 5, 2021, 4:35 PM

#

green oriole <@165023948638126080> ye wise man <:lemon_pensive:754441880246419486>

~~hey no, lemme give you the context so you can thank me~~

#

patent pivot May 5, 2021, 4:36 PM

#

I added it 🐒

green oriole May 5, 2021, 4:36 PM

#

That's an important fact

fallen patrol May 5, 2021, 4:36 PM

#

i suggested it :read_no_evil:

green oriole May 5, 2021, 4:37 PM

#

That remind me of the pyfact thing for some reason

#

Is it even live

#

I remember we had a Github discussion for it, dunno if we ever received a message on it

#

.pyfact

dusky shoreBOT May 5, 2021, 4:37 PM

#

Python Facts

Python was named after Monty Python, a British Comedy Troupe, which Guido van Rossum likes.

Suggestions

Suggest more facts here!

sleek steppe May 5, 2021, 4:37 PM

#

vocal wolf <@!506618674921340958> tested thing. Works well.

thanks!

green oriole May 5, 2021, 4:37 PM

#

What a fact

vocal prairie May 5, 2021, 4:37 PM

#

fallen patrol ~~hey no, lemme give you the context so you can thank me~~

That's not the only context

patent pivot May 5, 2021, 4:38 PM

#

ya, we did receive stuff, we should prob add some 🙂

vocal wolf May 5, 2021, 4:38 PM

#

~~I may or may not be running on 2 hours of sleep~~

patent pivot May 5, 2021, 4:38 PM

#

we don't need to discuss the etymology of off topic names here

vocal wolf May 5, 2021, 4:38 PM

#

it's ok though, I'm going back to sleep after this zoom class

green oriole May 5, 2021, 4:39 PM

#

!e

print(f"Xith's uptime: {2 / 24 * 100}%")

stable mountainBOT May 5, 2021, 4:39 PM

#

@green oriole :white_check_mark: Your eval job has completed with return code 0.

Xith's uptime: 8.333333333333332%

fallen patrol May 5, 2021, 4:39 PM

#

vocal prairie That's not the only context

~~yes, it was actually vco who first did it~~

green oriole May 5, 2021, 4:39 PM

#

Wait, it is the other way around

#

I thought we paid the hosting for more than 91% of uptime

sleek steppe May 5, 2021, 4:48 PM

#

I feel bad for anand, he's going through the 84 files of my PR 😔

vocal wolf May 5, 2021, 4:49 PM

#

I'm going to have to review it as well lol

sleek steppe May 5, 2021, 4:50 PM

#

Is this the biggest PR on lancebot

#

in terms of additions/deletions

vocal wolf May 5, 2021, 4:50 PM

#

D:

patent pivot May 5, 2021, 4:51 PM

#

lol

vocal wolf May 5, 2021, 5:04 PM

#

15 PRs on bot, but can it go to 10?

vale ibex May 5, 2021, 5:08 PM

#

is there a reason https://github.com/python-discord/bot/pull/1561 is still open?

#

I just approved the workflow, should I just merge?

#

ahh Akarys has auto merge on

#

but after merging in main, we had to approve the lint action

#

it should merge after linting now

green oriole May 5, 2021, 5:09 PM

#

Why did it not merge

#

Huh interesting

sleek steppe May 5, 2021, 5:09 PM

#

Oh yeah my PR is the biggest on lancebot lol

green oriole May 5, 2021, 5:10 PM

#

The spring cleanup one?

gritty wind May 5, 2021, 5:10 PM

#

I wonder if the check is stuck

vale ibex May 5, 2021, 5:10 PM

#

nah, it's not

gritty wind May 5, 2021, 5:10 PM

#

Oh it was out of date

vale ibex May 5, 2021, 5:10 PM

#

the lint action needs to be approved to be ran for first time contribs

gritty wind May 5, 2021, 5:10 PM

#

I thought the merge was 30 minutes ago

#

It was

#

ah

green oriole May 5, 2021, 5:11 PM

#

Flashback to my open 1.4k lines PR on the site, my god, why did I step into

vale ibex May 5, 2021, 5:11 PM

#

even if its a core dev that does an action (such as merge in main)

#

we still need to approve lol

vocal prairie May 5, 2021, 5:11 PM

#

vale ibex is there a reason https://github.com/python-discord/bot/pull/1561 is still open?

It was a draft for a while

vale ibex May 5, 2021, 5:11 PM

#

Yea, that wasn't the issue 😄 It's all of the above

#

it's merged now anyway

#

Congrats 😄

vocal wolf May 5, 2021, 5:15 PM

#

but can we get 13 hyperlemon

fervent sage May 5, 2021, 5:17 PM

#

is today the pydis pr sprint lol

sleek steppe May 5, 2021, 5:17 PM

#

vocal wolf but can we get 13 <:hyperlemon:435707922563989524>

Depends, can bot#1564 get merged

dusky shoreBOT May 5, 2021, 5:17 PM

#

GitHub

PROpen When duration specified for !ban, apply a !tempban

vocal wolf May 5, 2021, 5:17 PM

#

poor @balmy sparrow

balmy sparrow May 5, 2021, 5:17 PM

#

😔

#

im half way through

vocal wolf May 5, 2021, 5:18 PM

#

you won't be forgotten

balmy sparrow May 5, 2021, 5:18 PM

#

guys can someone explain this to me my brain has shrunk https://why.life-is-pa.in/0wrlyp

0wrlyp.png

fervent sage May 5, 2021, 5:18 PM

#

me and toxic: 👁️

sleek steppe May 5, 2021, 5:18 PM

#

Reviewing is easier than doing it 😔

vocal wolf May 5, 2021, 5:18 PM

#

sleek steppe Depends, can bot#1564 get merged

Aren't we thinking about doing this for other commands, or have admins said that we're ok with this merging?

balmy sparrow May 5, 2021, 5:19 PM

#

fervent sage me and toxic: 👁️

99% of my comments are nitpicks on other parts of the code that wasn't touched by the PR 👁️

fervent sage May 5, 2021, 5:20 PM

#

lol

vocal prairie May 5, 2021, 5:20 PM

#

balmy sparrow 99% of my comments are nitpicks on other parts of the code that wasn't touched b...

It is a cleanup 🤷‍♂️