#dev-contrib | Python | Page 114

gritty wind May 16, 2021, 2:59 PM

#

looks like the codeowners was missed

#

I'll check out the PR and give it a whirl

vale ibex May 16, 2021, 2:59 PM

#

# Extensions groups
bot/exts/christmas/**                   @ks129
bot/exts/halloween/**                   @ks129

# CI & Docker
.github/workflows/**                    @Akarys42 @SebastiaanZ @Den4200
Dockerfile                              @Akarys42 @Den4200
docker-compose.yml                      @Akarys42 @Den4200

# Tools
Pipfile*                                @Akarys42

gritty wind May 16, 2021, 2:59 PM

#

Should hopefully be able to review now

vale ibex May 16, 2021, 2:59 PM

#

What's missing?

#

I'll change it, so your review counts lol

gritty wind May 16, 2021, 3:00 PM

#

Pipfile is no longer a thing

vale ibex May 16, 2021, 3:00 PM

#

ah yis

gritty wind May 16, 2021, 3:00 PM

#

Have to manually add the pyproject and lock

vale ibex May 16, 2021, 3:02 PM

#

Done

gritty wind May 16, 2021, 3:02 PM

#

noice

vale ibex May 16, 2021, 3:02 PM

#

This'll need some site changes too

gritty wind May 16, 2021, 3:03 PM

#

btw we'll probably be switching around over the next few days

#

you may find pipenv --rm to be a useful command 👀

sleek steppe May 16, 2021, 3:03 PM

#

yes

vale ibex May 16, 2021, 3:03 PM

#

hah true

#

I'm heading out for a few hours before the staff meeting now

gritty wind May 16, 2021, 3:03 PM

#

now if someone knows the poetry equivalent

#

that would just be

#

👌

placid ermine May 16, 2021, 3:03 PM

#

uh pipenv loads env variables automatically when i do pipenv run, that won't happen with poetry will it

vale ibex May 16, 2021, 3:03 PM

#

I'll raise a site PR if someone doesn't by then

gritty wind May 16, 2021, 3:03 PM

#

placid ermine uh pipenv loads env variables automatically when i do `pipenv run`, that won't h...

we added python-dotenv for that

#

should be in the constants file iirc

vale ibex May 16, 2021, 3:04 PM

#

gritty wind now if someone knows the poetry equivalent

poetry env remove

gritty wind May 16, 2021, 3:04 PM

#

shweet

#

thanks

vale ibex May 16, 2021, 3:04 PM

#

gritty wind should be in the constants file iirc

__init__.py but yea, it's there

placid ermine May 16, 2021, 3:04 PM

#

gritty wind we added python-dotenv for that

ok_handbutflipped

sleek steppe May 16, 2021, 3:16 PM

#

Should I make a PR to site since lance's env var reference is missing CHANNEL_REDDIT? https://pythondiscord.com/pages/guides/pydis-guides/contributing/sir-lancebot/env-var-reference/

gritty wind May 16, 2021, 3:17 PM

#

dew it

#

Actually, while you're at it

#

can you make sure the page is up to date

#

just look for any env vars in the project

sleek steppe May 16, 2021, 3:18 PM

#

will do

vocal prairie May 16, 2021, 3:27 PM

#

Oh, did I forget that one yesterday? My bad

gritty wind May 16, 2021, 3:29 PM

#

https://pythondiscord.com/pages/guides/pydis-guides/contributing/installing-project-dependencies/

Has anyone ever seen this page

#

I just ran into it for the first time

#

I don't think it's linked anywhere

#

I found it in the repo

#

I think I'll just delete it

green oriole May 16, 2021, 3:30 PM

#

It used to be linked somewhere

gritty wind May 16, 2021, 3:30 PM

#

huh

#

it is actually about halfway down on the guides

#

alright, I'll fix the page

#

do you happen to know it on other systems?

#

right, so what's the shortcut on mac?

#

there's only one shortcut in the guide

#

settings

#

Now I just need a linux dev

#

probably still ctrl + shift +s

sleek steppe May 16, 2021, 3:46 PM

#

What description should I put for this one 🤔 (and I'm assuming it goes in the debug section)

gritty wind May 16, 2021, 3:46 PM

#

GIT_SHA may actually be better left out

#

it's mostly a CI thing

sleek steppe May 16, 2021, 3:46 PM

#

ah ok

#

I added IGDB_CLIENT_ID, IGDB_CLIENT_SECRET, AOC_COMMANDS_CHANNEL_ID, AOC_SESSION_COOKIE, and CHANNEL_REDDIT

gritty wind May 16, 2021, 3:48 PM

#

those should already all be there

sleek steppe May 16, 2021, 3:48 PM

#

gritty wind those should already all be there

yep, but none of them are

gritty wind May 16, 2021, 3:48 PM

#

linux and windows probably have the shortcuts

lunar hull May 16, 2021, 3:59 PM

#

Hey, so I'm trying to contribute a feature to Python, but am a little unclear on what the steps to do it are based on the Devguide posted. The devguide says that my branch should be named with the issue number (and to create it if it doesn't exist). From what I understand I need to do this before I create my branch, commit and make a pull request. However on the New Issue page on the issue tracker it also asks me for a link to the PR, so I'm not quite sure which order to be doing things in

gritty wind May 16, 2021, 4:01 PM

#

Hi, could you send a link to the page asking for the PR link?

vale ibex May 16, 2021, 4:10 PM

#

lance, bot and site all refer to http://pythondiscord.local:8000/pages/guides/pydis-guides/contributing/installing-project-dependencies/

gritty wind May 16, 2021, 4:10 PM

#

ye

vale ibex May 16, 2021, 4:10 PM

#

Do we want to update this to have both (poetry and pipenv) right now, or should we wait until we migrate site and bot?

gritty wind May 16, 2021, 4:10 PM

#

I don't know if site will be migrated

#

I already updated the guide to both

#

check dev-log lol

vale ibex May 16, 2021, 4:11 PM

#

oh lol, I won't update then lmfao

#

LOL I've also fixed the images

#

smh

#

guess i should have checked when i got back

#

oh well

gritty wind May 16, 2021, 4:11 PM

#

I wonder if we have more broken images like that elsewhere

vale ibex May 16, 2021, 4:12 PM

#

joe should have 404 logs

#

would be easier to look via those

lunar hull May 16, 2021, 4:12 PM

#

@gritty wind https://bugs.python.org/issue?@template=item

gritty wind May 16, 2021, 4:12 PM

#

Oh

#

You are looking at bug.python.org, which is for the python language itself

#

this is not for our projects

#

If you mean to ask for the python language, check out #internals-and-peps

lunar hull May 16, 2021, 4:13 PM

#

oh LOL

#

sorry

gritty wind May 16, 2021, 4:13 PM

#

no worries

gritty wind May 16, 2021, 4:13 PM

#

vale ibex joe should have 404 logs

they can't 404 if no one ever visits those pages lol

#

damn we are so close I can taste it

sleek steppe May 16, 2021, 4:15 PM

#

site#501 if anyone wants to review

dusky shoreBOT May 16, 2021, 4:15 PM

#

GitHub

PROpen Update sir-lancebot's env var reference

vale ibex May 16, 2021, 4:33 PM

#

gritty wind they can't 404 if no one ever visits those pages lol

🤔

#

https://github.com/python-discord/site/pull/502/files the diff says the images have gotten smaller lol

gritty wind May 16, 2021, 4:33 PM

#

that is indeed true

#

I redid the screenshots

#

to add poetry

vale ibex May 16, 2021, 4:33 PM

#

ahhhh kewl

gritty wind May 16, 2021, 4:34 PM

#

you can see image diffs on git

#

just select the rich diff

#

dusky shoreBOT May 16, 2021, 4:38 PM

#

GitHub

PROpen Add the stackoverflow command

short snow May 16, 2021, 4:41 PM

#

gritty wind just select the rich diff

Where do we select rich diff? I can’t see the image diff

gritty wind May 16, 2021, 4:42 PM

#

in that screenshot, in the top right

#

there are two side by side buttons

#

click the one on the right

short snow May 16, 2021, 4:42 PM

#

Oo, lemme see

#

Nice, tyy

patent pivot May 16, 2021, 4:49 PM

#

bot#1582 looks good

dusky shoreBOT May 16, 2021, 4:49 PM

#

GitHub

PROpen Upgrade To 3.9 and Poetry

gritty wind May 16, 2021, 4:50 PM

#

I think we're ready to merge, but we probably want to get the guides ready first

patent pivot May 16, 2021, 4:50 PM

#

Yeah

gritty wind May 16, 2021, 4:50 PM

#

PR is ready for that, but it upgrades both sir-lance and python

patent pivot May 16, 2021, 4:50 PM

#

What's the plan for lancebot? My PR bumps to 3.9 and poetry but I see there is another PR just for the py version bump

gritty wind May 16, 2021, 4:50 PM

#

We'll merge yours, and I think the plan was to just close the 3.9 upgrade

#

it bumped a lot of dependencies for no real reason

patent pivot May 16, 2021, 4:51 PM

#

gotcha

gritty wind May 16, 2021, 4:51 PM

#

Main selling point was 3.9 anyways

patent pivot May 16, 2021, 4:51 PM

#

yep

#

cc <@&409416496733880320> approval needed on sir-lancebot#623

dusky shoreBOT May 16, 2021, 4:52 PM

#

GitHub

PROpen Pipenv to poetry

vale ibex May 16, 2021, 4:52 PM

#

Need a staff on site#502 too

dusky shoreBOT May 16, 2021, 4:52 PM

#

GitHub

PROpen Poetry In Dev Guides

patent pivot May 16, 2021, 4:53 PM

#

I'll give that a look

#

yeah all looks fine there

#

alright dope

gritty wind May 16, 2021, 4:54 PM

#

One more approval on sir-lance and all 3 can be merged

patent pivot May 16, 2021, 4:54 PM

#

yeah

#

@green oriole are you around to sign off on sir-lancebot#623

dusky shoreBOT May 16, 2021, 4:55 PM

#

GitHub

PROpen Pipenv to poetry

green oriole May 16, 2021, 4:57 PM

#

yessir

#

watch it all go down in flames now

patent pivot May 16, 2021, 4:57 PM

#

lmfao

#

if someone can write up a dev announce as well that'd be pretty dope, otherwise I can write one up later/tomorrow

#

good work everybody

green oriole May 16, 2021, 4:58 PM

#

I could do that although that'd be pretty unfair lol

patent pivot May 16, 2021, 4:59 PM

#

scaleios is typing lol

gritty wind May 16, 2021, 4:59 PM

#

you saw nothing

green oriole May 16, 2021, 5:00 PM

#

so we are all migrated to poetry?

gritty wind May 16, 2021, 5:00 PM

#

site is still pipenv

#

amongst a few other projects

#

but for python and sir-lance, we're all migrated

green oriole May 16, 2021, 5:01 PM

#

Noice

vale ibex May 16, 2021, 5:01 PM

#

🎉

green oriole May 16, 2021, 5:01 PM

#

!help

#

It is running 👀

vale ibex May 16, 2021, 5:01 PM

#

We should write up a migration guide for contribs

#

and announce

green oriole May 16, 2021, 5:02 PM

#

oh wait, not this one

gritty wind May 16, 2021, 5:02 PM

#

Waaaay ahead of y'all lol

green oriole May 16, 2021, 5:02 PM

#

.help

dusky shoreBOT May 16, 2021, 5:02 PM

#

Command Help

AdventOfCode
.adventofcode
All of the Advent of Code commands.

AprilFoolVideos
.fool
Get a random April Fools' video from Youtube.

AvatarModify
.8bitify
Pixelates your avatar and changes the palette to an 8bit one.
.avatar_modify
Groups all of the pfp modifying commands to allow a single concurrency limit.
.avatareasterify [colours...]
This "Easterifies" the user's avatar.
.mosaic [squares=16]
Splits your avatar into x squares, randomizes them and stitches them back into a new image!
.prideavatar [option=lgbt] [pixels=64]
This surrounds an avatar with a border of a specified LGBT flag.
.spookyavatar [member]
Spookify an user's avatar.

green oriole May 16, 2021, 5:07 PM

#

Uh oh @gritty wind

#

It failed

gritty wind May 16, 2021, 5:07 PM

#

hmmm

#

reviewing now

#

we need git

#

ughhhh

#

or do we

#

I'll run some tests

patent pivot May 16, 2021, 5:12 PM

#

uh

#

don't think we do

gritty wind May 16, 2021, 5:12 PM

#

why does prod not exist

#

this is hell

patent pivot May 16, 2021, 5:12 PM

#

hmmm

gritty wind May 16, 2021, 5:12 PM

#

CI is hell

#

it's --no-dev

#

why was I using prod

#

I'm so confused what's happening ugghhhh

patent pivot May 16, 2021, 5:14 PM

#

ohhh

#

was it uhhh

#

isn't that yarn

#

lol

gritty wind May 16, 2021, 5:14 PM

#

uhhhh

patent pivot May 16, 2021, 5:14 PM

#

hahahahaha yeah

gritty wind May 16, 2021, 5:14 PM

#

I can neither confirm nor deny

#

but the docker file works fine locally

#

hmm

patent pivot May 16, 2021, 5:14 PM

#

#

huh

#

old version of poetry?

vale ibex May 16, 2021, 5:15 PM

#

gritty wind but the docker file works fine locally

well yea, it doesn't have --prod on it

#

lol

gritty wind May 16, 2021, 5:15 PM

#

it does

#

line 16

#

https://github.com/python-discord/bot/blob/1db39d0c3cd1c3f4731f4c494a892d15a07f00fe/Dockerfile#L16

stable mountainBOT May 16, 2021, 5:15 PM

#

Dockerfile line 16

RUN poetry install --prod```

gritty wind May 16, 2021, 5:15 PM

#

I'll just update it and stop asking questions

vale ibex May 16, 2021, 5:16 PM

#

oh yea, it's in the lint ci it doesn't

#

mb

sleek steppe May 16, 2021, 5:16 PM

#

site#501 needs 1 staff/contrib review

dusky shoreBOT May 16, 2021, 5:16 PM

#

GitHub

PROpen Update sir-lancebot's env var reference

gritty wind May 16, 2021, 5:17 PM

#

did I not test the docker file

#

I'm so confused

#

I tested it with the compose

#

which may or may not have not actually rebuilt the image

vale ibex May 16, 2021, 5:18 PM

#

gritty wind which may or may not have not actually rebuilt the image

oh lol, I did the same

#

docker-compose up

#

missed off --build

#

so it just used my old image

gritty wind May 16, 2021, 5:19 PM

#

alright, tested it again

#

this time correctly

#

all things seem fine

#

hotfix incoming

vale ibex May 16, 2021, 5:21 PM

#

When writing the PR takes longer than the fix

#

Sadge

sleek steppe May 16, 2021, 5:21 PM

#

lol

gritty wind May 16, 2021, 5:21 PM

#

alright

#

1589

#

go do things

sleek steppe May 16, 2021, 5:21 PM

#

Is there gonna be a dev announcement for the migration 👀

gritty wind May 16, 2021, 5:22 PM

#

no

sleek steppe May 16, 2021, 5:22 PM

#

ah ic

gritty wind May 16, 2021, 5:22 PM

#

does git pull --all update forks

#

hmm

vale ibex May 16, 2021, 5:22 PM

#

<@&409416496733880320> bot#1589 pls

dusky shoreBOT May 16, 2021, 5:22 PM

#

GitHub

PROpen Hotfix: Fixes Dependency Install Flag In Dockerfile

patent pivot May 16, 2021, 5:23 PM

#

lookin

vale ibex May 16, 2021, 5:23 PM

#

gritty wind does `git pull --all` update forks

iirc it does if their upstream is setup

patent pivot May 16, 2021, 5:23 PM

#

donezo

vale ibex May 16, 2021, 5:25 PM

#

gritty wind hmm

I can test if you aren't already

gritty wind May 16, 2021, 5:25 PM

#

I tested with an origin

#

that does work

vale ibex May 16, 2021, 5:25 PM

#

cool

gritty wind May 16, 2021, 5:25 PM

#

Please do if you've got a minute

vale ibex May 16, 2021, 5:25 PM

#

mhmm

sleek steppe May 16, 2021, 5:26 PM

#

@fervent sage we've upgraded to 1.7.2, so feel free to make the changes in your PR

gritty wind May 16, 2021, 5:27 PM

#

nice

patent pivot May 16, 2021, 5:28 PM

#

rock solid

gritty wind May 16, 2021, 5:29 PM

#

We're a go for the announcement once we confirm the git pull works lol

vale ibex May 16, 2021, 5:29 PM

#

gritty wind Please do if you've got a minute

hmm, didn't seem to no

#

had to do git pull upstream main

gritty wind May 16, 2021, 5:29 PM

#

Hmm alright

#

That's fine too

#

thank you

fervent sage May 16, 2021, 5:31 PM

#

sleek steppe <@!297045071457681409> we've upgraded to 1.7.2, so feel free to make the changes...

ok_handbutflipped

vale ibex May 16, 2021, 5:32 PM

#

!ping

stable mountainBOT May 16, 2021, 5:32 PM

#

Pong!

Command processing time

75.362 ms

Python Discord website latency

0.843 ms

Discord API latency

116.167 ms

vale ibex May 16, 2021, 5:32 PM

#

Nice

patent pivot May 16, 2021, 5:33 PM

#

winner

eager fable May 16, 2021, 5:34 PM

#

@patent pivot I'm sure you already know that we hit 200k members, when will the announcement come out?

gritty wind May 16, 2021, 5:35 PM

#

I looked through the PRs again

#

did I miss anyone in the pings at the end

patent pivot May 16, 2021, 5:40 PM

#

eager fable <@!165023948638126080> I'm sure you already know that we hit 200k members, when ...

We don't have anything lined up in terms of a specific 200k announcement, I'm afraid, but we do have exciting things to announce over the coming weeks and months so keep your eyes peeled for that.

vocal wolf May 16, 2021, 5:46 PM

#

@cold moon Hello, I hope you've been well. What's the status of bot#1487?

dusky shoreBOT May 16, 2021, 5:46 PM

#

GitHub

PROpen Goodbye, #talent-pool

fallen patrol May 16, 2021, 5:46 PM

#

@gritty wind smh

#

You have two tldrs in the same message

gritty wind May 16, 2021, 5:48 PM

#

it's a complex topic

fallen patrol May 16, 2021, 5:48 PM

#

vocal wolf <@!446310592425361418> Hello, I hope you've been well. What's the status of bot#...

Why is talent pool being removed

vocal wolf May 16, 2021, 5:48 PM

#

don't need it

gritty wind May 16, 2021, 5:48 PM

#

have you ever stopped to consider how sane it would be to read every single message sent by a user for months

patent pivot May 16, 2021, 5:48 PM

#

lul

vale ibex May 16, 2021, 5:49 PM

#

fallen patrol Why is talent pool being removed

The relay channel is being removed, we will still have a pool of candidates

sleek steppe May 16, 2021, 5:49 PM

#

I'd say sir-lancebot#683 can be closed right?

dusky shoreBOT May 16, 2021, 5:49 PM

#

GitHub

PROpen Updates Sir-lancebot To Python3.9.4

fallen patrol May 16, 2021, 5:49 PM

#

gritty wind have you ever stopped to consider how sane it would be to read every single mess...

Well isnt that the purpose of big brother?

vale ibex May 16, 2021, 5:49 PM

#

Yea, i think an issue should be made for bumping the versions of the deps though

gritty wind May 16, 2021, 5:50 PM

#

i don't use big brother either

#

some do

fallen patrol May 16, 2021, 5:50 PM

#

Or do people who get watched often get removed from it shortly?

#

Either by ban or by improved behavior

thorny obsidian May 16, 2021, 5:51 PM

#

Big brother is used differently. I do use big brother and the goal is that a user's behavior either improves enough that we remove them or their behavior is poor enough that we remove them from the community.

vale ibex May 16, 2021, 6:05 PM

#

I'm just making @vocal wolf's job harder

#

you better have an upto date list of merged PRs lul

vocal wolf May 16, 2021, 6:06 PM

#

y u do dis

vale ibex May 16, 2021, 6:06 PM

#

❤️

vocal wolf May 16, 2021, 6:06 PM

#

what did you do

vale ibex May 16, 2021, 6:06 PM

#

some bot, site and lance prs

fervent sage May 16, 2021, 6:06 PM

#

oops i totally forgot about my async await tag creation kek

vocal wolf May 16, 2021, 6:07 PM

#

oh snap

#

13 PRs on bot

fallen patrol May 16, 2021, 6:08 PM

#

🤔

#

pipx 0.16.1.0 has requirement importlib-metadata>=3.3.0; python_version < "3.8", but you'll have importlib-metadata 1.7.0 which is incompatible.

sleek steppe May 16, 2021, 6:08 PM

#

Could we get sir-lancebot to 16 👀? sir-lancebot#733

dusky shoreBOT May 16, 2021, 6:09 PM

#

GitHub

PROpen Http status command randomness

fallen patrol May 16, 2021, 6:09 PM

#

trying to upgrade lance to poetry

sleek steppe May 16, 2021, 6:09 PM

#

Are you using python 3.9?

fallen patrol May 16, 2021, 6:09 PM

#

no

#

3.7 is my system python

sleek steppe May 16, 2021, 6:09 PM

#

Well then, time to upgrade

fallen patrol May 16, 2021, 6:09 PM

#

so i have to install poetry on the 3.9 python?

sleek steppe May 16, 2021, 6:09 PM

#

yes

fallen patrol May 16, 2021, 6:10 PM

#

@gritty wind pls update #dev-announcements

gritty wind May 16, 2021, 6:10 PM

#

you don't need to do that

#

as long as you can install the dep manager

#

if you can't install the dep manager at all, and it's a supported python version

#

it's a bug that should be raised with poetry

fallen patrol May 16, 2021, 6:11 PM

#

ay now i know my path got fucked up

gritty wind May 16, 2021, 6:11 PM

#

as for having python 3.9 on your system, that's the first thing in the announcement

sleek steppe May 16, 2021, 6:12 PM

#

ah I thought it said somewhere that poetry has trouble with using a different version of python that it isn't installed with

gritty wind May 16, 2021, 6:12 PM

#

Yeah, I said that

#

But that's a bit more complex than that

#

If you're on a system that supports pipx at all, poetry should actually be able to handle itself fine

#

if you can't install poetry at all

#

that's a completely different matter

vale ibex May 16, 2021, 6:14 PM

#

vocal wolf 13 PRs on bot

can we get <10 lemonshake

vocal wolf May 16, 2021, 6:14 PM

#

Yes of course hyperlemon

#

@subtle kraken will you be going over the requested changes sometime soon? Thanks.

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

#

@gritty wind what do you think we should do about this PR? Changing docstrings would probably not be the best solution, unfortunately.

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

fallen patrol May 16, 2021, 6:19 PM

#

works fine

#

@sleek steppe@gritty wind had to fix my path. I changed shells and fix it as it happens and pyenv wasn't on it

gritty wind May 16, 2021, 6:21 PM

#

@vocal wolf two possible solutions. We change the commands that have time units (such as mute) to a more sane formatting, or we close the PR)

vocal wolf May 16, 2021, 6:21 PM

#

You have 1 hour remaining hyperlemon

sleek steppe May 16, 2021, 6:22 PM

#

I feel like we can just add \s to the eval help command

vale ibex May 16, 2021, 6:22 PM

#

I think it's across a few commands

vocal wolf May 16, 2021, 6:22 PM

#

gritty wind <@!196664644113268736> two possible solutions. We change the commands that have ...

Define "more sane formatting". What are you suggesting?

vale ibex May 16, 2021, 6:22 PM

#

eval was just used as an example

sleek steppe May 16, 2021, 6:22 PM

#

ah

vale ibex May 16, 2021, 6:22 PM

#

So we wanted a bot wide approach

gritty wind May 16, 2021, 6:25 PM

#

Basically it currently uses something like

\u2003\`h\` - hours

The suggestion is to switch that format into regular line breaks around the units, so they'd look like:

\`y\` - years

\`m\` - months∗

\`w\` - weeks

...

or an even better solution would be to add a <br> character, that is treated differently, so you'd get:

\`y\` - years<br>
\`m\` - months∗<br>
\`w\` - weeks<br>
...

vocal wolf May 16, 2021, 6:25 PM

#

oh jeez

placid ermine May 16, 2021, 6:25 PM

#

formatting ok_handbutflipped

fervent sage May 16, 2021, 6:25 PM

#

hahaha

gritty wind May 16, 2021, 6:26 PM

#

idk man

sleek steppe May 16, 2021, 6:26 PM

#

that looks bad lol

vocal wolf May 16, 2021, 6:26 PM

#

That does look like more sane formatting (as far as I can tell)

gritty wind May 16, 2021, 6:26 PM

#

fixed it for all you haters

vocal wolf May 16, 2021, 6:27 PM

#

lmao

gritty wind May 16, 2021, 6:27 PM

#

ima do that right now

sleek steppe May 16, 2021, 6:28 PM

#

sleek steppe Could we get sir-lancebot to 16 👀? sir-lancebot#733

if anyone wants to review this ^

vocal wolf May 16, 2021, 6:28 PM

#

gritty wind Basically it currently uses something like ``` \u2003\`h\` - hours ``` The sugg...

The first solution with linebreaks looks better, I don't think we'd want to actually include <br> in a docstring.

gritty wind May 16, 2021, 6:28 PM

#

well that's easy enough to do now

vocal wolf May 16, 2021, 6:29 PM

#

With any new command we'd have to test if it doesn't explode within the help command

fervent sage May 16, 2021, 6:31 PM

#

idk i think its more fun to play blind help command roulette mmLol

sleek steppe May 16, 2021, 6:32 PM

#

!int e [await ctx.send_help(cmd) for cmd in bot.walk_commands()]

vale ibex May 16, 2021, 6:36 PM

#

Hey @eternal owl I'm going over your voice gate PR bot#1587 and noticed that we don't catch discord.Forbidden on this line. I realise it's not related to your changes, but could you add it to the PR? 😄 https://github.com/python-discord/bot/blob/fea8300a7d84c0efec04bd1315fb3dfc48cbd488/bot/exts/moderation/voice_gate.py#L148

dusky shoreBOT May 16, 2021, 6:36 PM

#

GitHub

PROpen Prioratize DM over channel message for voice verification ping.

stable mountainBOT May 16, 2021, 6:36 PM

#

bot/exts/moderation/voice_gate.py line 148

await ctx.author.send(embed=embed)```

eternal owl May 16, 2021, 6:37 PM

#

vale ibex Hey <@!263560579770220554> I'm going over your voice gate PR bot#1587 and notice...

sure, will do now

vale ibex May 16, 2021, 6:37 PM

#

Thanks 😄

sleek steppe May 16, 2021, 6:38 PM

#

spring cleanup for bot when? 👀

vale ibex May 16, 2021, 6:38 PM

#

IG on fail, send it to the voice channel and schedule delete

eternal owl May 16, 2021, 6:38 PM

#

~~what do we want to do if dm is off~~

#

alright

vale ibex May 16, 2021, 6:38 PM

#

lol had the same thought

eternal owl May 16, 2021, 6:39 PM

#

sleek steppe spring cleanup for bot when? 👀

👀

fervent sage May 16, 2021, 6:39 PM

#

sleek steppe spring cleanup for bot when? 👀

ha

#

haha

#

ha

#

ha.

vale ibex May 16, 2021, 6:39 PM

#

1 more and you would have been auto-muted 👀

#

lol

fervent sage May 16, 2021, 6:39 PM

#

ik

#

:)

#

my sarcastic has were carefully designed by complete accident

vale ibex May 16, 2021, 6:40 PM

#

So, when using the docker-compose setup on the bot, it doesn't run the metabase migrations on the db

#

Has anyone solved this locally already?

patent pivot May 16, 2021, 6:42 PM

#

you mean docker-compose?

#

wait

#

you mean metricity

#

brain moment

#

hmmm

#

yeah, we don't have the init script on bot

eternal owl May 16, 2021, 6:45 PM

#

can you add a try-except inside an except block?

fervent sage May 16, 2021, 6:46 PM

#

yep

eternal owl May 16, 2021, 6:46 PM

#

thought so, but kinda looks weird, lol

clever wraith May 16, 2021, 6:49 PM

#

eternal owl can you add a try-except inside an except block?

yeah

vale ibex May 16, 2021, 6:50 PM

#

patent pivot you mean metricity

yes, lol my bad

clever wraith May 16, 2021, 6:50 PM

#

eternal owl thought so, but kinda looks weird, lol

yeah lol

eternal owl May 16, 2021, 6:50 PM

#

I'm gonna make some code changes to avoid code redundancy/repeated code

#

@vale ibex

clever wraith May 16, 2021, 6:50 PM

#

hmm

vale ibex May 16, 2021, 6:54 PM

#

patent pivot yeah, we don't have the init script on bot

I guess there's no way around it, unless we include the init file in the image

patent pivot May 16, 2021, 6:54 PM

#

hmm yeah

vocal wolf May 16, 2021, 6:55 PM

#

@gritty wind close? https://github.com/python-discord/sir-lancebot/pull/683

#

Look at them merge conflicts

vale ibex May 16, 2021, 6:55 PM

#

Close + make issue to bump deps?

#

I think now is a good time to update them

vocal wolf May 16, 2021, 6:55 PM

#

vale ibex Close + make issue to bump deps?

prob

eternal owl May 16, 2021, 6:58 PM

#

why are we doing this await self.bot.http.delete_message(Channels.voice_gate, message_id) instead of await message.delete()

#

in voice gate

gritty wind May 16, 2021, 6:59 PM

#

probably something to do with reliability

fervent sage May 16, 2021, 6:59 PM

#

doesnt require a msg object i assume

sleek steppe May 16, 2021, 6:59 PM

#

yes, the delete_message endpoint only needs ID while Message.delete needs the object

eternal owl May 16, 2021, 6:59 PM

#

should I be using that to delete trival messages?

fervent sage May 16, 2021, 6:59 PM

#

if you have a msg object use message.delete()

fallen patrol May 16, 2021, 6:59 PM

#

U can create an object with discord.Object tho

gritty wind May 16, 2021, 7:00 PM

#

isn't there a partial message

sleek steppe May 16, 2021, 7:00 PM

#

yes

eternal owl May 16, 2021, 7:00 PM

#

I mean, if you have message_id, you can get the message object 😬

fervent sage May 16, 2021, 7:00 PM

#

gritty wind isn't there a partial message

doesnt have delete for whatever dumb reason

sleek steppe May 16, 2021, 7:00 PM

#

!d discord.PartialMessage.delete

stable mountainBOT May 16, 2021, 7:00 PM

#

discord.PartialMessage.delete


await delete(*, delay=None)```
This function is a [*coroutine*](https://docs.python.org/3/library/asyncio-task.html#coroutine).

Deletes the message.

Your own messages could be deleted without any proper permissions. However to delete other people’s messages, you need the [`manage_messages`](https://discordpy.readthedocs.io/en/stable/api.html#discord.Permissions.manage_messages "discord.Permissions.manage_messages") permission.

Changed in version 1.1: Added the new `delay` keyword-only parameter.

fervent sage May 16, 2021, 7:00 PM

#

eternal owl I mean, if you have message_id, you can get the message object 😬

yes but thats an api call

#

huh

eternal owl May 16, 2021, 7:00 PM

#

ah

#

so

#

we are avoiding an extra api call

#

smart

tough imp May 16, 2021, 7:01 PM

#

its bc the ID comes from the cache

#

so you dont have the obj

#

and dont need it either

eternal owl May 16, 2021, 7:01 PM

#

cool cool

sleek steppe May 16, 2021, 7:02 PM

#

fallen patrol U can create an object with discord.Object tho

That wouldn't work at all, because it still needs the channel ID as well

fervent sage May 16, 2021, 7:02 PM

#

a discord.Object() is literally just a thing with a snowflake

#

nothing more nothing less

cold moon May 16, 2021, 7:04 PM

#

vocal wolf <@!446310592425361418> Hello, I hope you've been well. What's the status of bot#...

#dev-core message

tough imp May 16, 2021, 7:04 PM

#

be careful with the cog, its prone to race conditions due to having to check the cache often and relying on voice events which sometimes fire a few times in succession

vocal wolf May 16, 2021, 7:05 PM

#

cold moon https://discord.com/channels/267624335836053506/411200599653351425/8434550139798...

ty

tough imp May 16, 2021, 7:05 PM

#

thats why the helpers enforce atomicity

eternal owl May 16, 2021, 7:06 PM

#

I just made a simple change instead of changing the _delete_ping func to handle message IDs as well, as it would not make much of a diff

#

@vale ibex

vale ibex May 16, 2021, 7:07 PM

#

Nice, I'll give it a look after this admin call

eternal owl May 16, 2021, 7:07 PM

#

oki dokies

#

yes?

stable mountainBOT May 16, 2021, 7:13 PM

#

if __name__ == '__main__'

This is a statement that is only true if the module (your source code) it appears in is being run directly, as opposed to being imported into another module. When you run your module, the __name__ special variable is automatically set to the string '__main__'. Conversely, when you import that same module into a different one, and run that, __name__ is instead set to the filename of your module minus the .py extension.

Example

# foo.py

print('spam')

if __name__ == '__main__':
    print('eggs')

If you run the above module foo.py directly, both 'spam'and 'eggs' will be printed. Now consider this next example:

# bar.py

import foo

If you run this module named bar.py, it will execute the code in foo.py. First it will print 'spam', and then the if statement will fail, because __name__ will now be the string 'foo'.

Why would I do this?

• Your module is a library, but also has a special case where it can be run directly
• Your module is a library and you want to safeguard it against people running it directly (like what pip does)
• Your module is the main program, but has unit tests and the testing framework works by importing your module, and you want to avoid having your main code run during the test

vocal wolf May 16, 2021, 7:20 PM

#

@clever wraith I think the PR may no longer be a draft, what do you think?

#

also ty for updating

sleek steppe May 16, 2021, 7:21 PM

#

I think the tag should be updated to talk more about blocking, since that's what the tag is about

vocal wolf May 16, 2021, 7:24 PM

#

it still be a draft

#

thank

fervent sage May 16, 2021, 7:26 PM

#

poetry is love poetry is life

fallen patrol May 16, 2021, 7:32 PM

#

faster.

#

Much faster.

fervent sage May 16, 2021, 7:32 PM

#

ok_handbutflipped

gritty wind May 16, 2021, 7:33 PM

#

if the tag isn't loaded during tests, it's an increase in line count without an increase in lines hit

fallen patrol May 16, 2021, 7:34 PM

#

the coverage rating is how much of the code was covered by the tests

dry folio May 16, 2021, 7:43 PM

#

hey, does anyone know what do these "conflicts" mean and how should i resolve them? thanks!

gritty wind May 16, 2021, 7:44 PM

#

one branch has the content above the =====, and another branch has the content below

#

Git can't figure out how to merge these differences by itself

#

so you are expected to solve it yourself

#

imports are usually the easiest

dry folio May 16, 2021, 7:45 PM

#

gritty wind so you are expected to solve it yourself

oh... and so i just delete the things below ======?

gritty wind May 16, 2021, 7:45 PM

#

No,

#

you want to merge the content of both

dry folio May 16, 2021, 7:45 PM

#

gritty wind you want to merge the content of both

oh...?

gritty wind May 16, 2021, 7:45 PM

#

in this case, it's just going to be:

from bot.bot import Bot
from bot.constant import Colours, NEGATIVE_REPLIES, Roles```

dry folio May 16, 2021, 7:45 PM

#

gritty wind in this case, it's just going to be: ```py from bot.bot import Bot from bot.cons...

ah alright

gritty wind May 16, 2021, 7:45 PM

#

that way the imports satisfy the code from both branches

dry folio May 16, 2021, 7:46 PM

#

gritty wind that way the imports satisfy the code from both branches

but i don't understand why these conflicts occur... they didn't before, and i didn't even commit they just suddenly appeared

dry folio May 16, 2021, 7:47 PM

#

gritty wind that way the imports satisfy the code from both branches

what about this? i want to keep mine (above ===)

sleek steppe May 16, 2021, 7:47 PM

#

whatever change actually works

gritty wind May 16, 2021, 7:48 PM

#

Yeah ^

#

Did you try updating your branch or something?

fallen patrol May 16, 2021, 7:50 PM

#

dry folio but i don't understand why these conflicts occur... they didn't before, and i di...

Fwiw if you aren't already you shouldn't be commiting to main/master

dry folio May 16, 2021, 7:51 PM

#

gritty wind Did you try updating your branch or something?

i'm not sure, but the conflicts are resolved now :D

#

back to waiting for the last approval lol

fallen patrol May 16, 2021, 7:54 PM

#

.source help

#

#

Sounds about right

sleek steppe May 16, 2021, 7:55 PM

#

oh no

#

ah I see it

vocal wolf May 16, 2021, 8:10 PM

#

@dry folio merged.

#

Thank you for the very large contribution!

#

@short snow Will you be updating this sometime soon?

https://github.com/python-discord/sir-lancebot/pull/618

fallen patrol May 16, 2021, 8:22 PM

#

🤔

fallen patrol May 16, 2021, 8:22 PM

#

vocal wolf <@!769221607888912487> merged.

Didn't Joe say that pr should be squash merged?

vocal wolf May 16, 2021, 8:23 PM

#

no idea

patent pivot May 16, 2021, 8:24 PM

#

lol

#

it's good

#

but ya prob should have policy of squashing if there are lint or fmting commits

sleek steppe May 16, 2021, 8:24 PM

#

.source help

dusky shoreBOT May 16, 2021, 8:24 PM

#

Help Command

A discord.ext.commands.Command subclass which supports root aliases.

Source Code

Go to GitHub

sleek steppe May 16, 2021, 8:25 PM

#

^ It's getting the docstring for bot.command.Command, can I make a fix or it?

fallen patrol May 16, 2021, 8:26 PM

#

patent pivot but ya prob should have policy of squashing if there are lint or fmting commits

Did you squash the site one you just made :P

fallen patrol May 16, 2021, 8:26 PM

#

vocal wolf Thank you for the very large contribution!

Just caused a sentry issue lol

patent pivot May 16, 2021, 8:27 PM

#

fallen patrol Did you squash the site one you just made :P

yes, it had 2 useless commits

brazen charm May 16, 2021, 8:27 PM

#

sleek steppe ^ It's getting the docstring for bot.command.Command, can I make a fix or it?

Doesn't feel like something that needs to be discussed much, can also thrown in an unwrap on the source object like @stable mountain does while you're doing that

patent pivot May 16, 2021, 8:27 PM

#

fallen patrol Just caused a sentry issue lol

ya race condition

sleek steppe May 16, 2021, 8:28 PM

#

I can write a fix for that one too

remote wigeon May 16, 2021, 8:28 PM

#

laughs in rust

#

KEKW

#

more like sadkek tho

sleek steppe May 16, 2021, 8:29 PM

#

brazen charm Doesn't feel like something that needs to be discussed much, can also thrown in ...

Question, what's unwrap?

brazen charm May 16, 2021, 8:30 PM

#

!d inspect.unwrap

stable mountainBOT May 16, 2021, 8:30 PM

#

inspect.unwrap


inspect.unwrap(func, *, stop=None)```
Get the object wrapped by *func*. It follows the chain of `__wrapped__` attributes returning the last object in the chain.

*stop* is an optional callback accepting an object in the wrapper chain as its sole argument that allows the unwrapping to be terminated early if the callback returns a true value. If the callback never returns a true value, the last object in the chain is returned as usual. For example, [`signature()`](https://docs.python.org/3/library/inspect.html#inspect.signature "inspect.signature") uses this to stop unwrapping if any object in the chain has a `__signature__` attribute defined.

[`ValueError`](https://docs.python.org/3/library/exceptions.html#ValueError "ValueError") is raised if a cycle is encountered.

New in version 3.4.

brazen charm May 16, 2021, 8:31 PM

#

When decorators (with inspect.update_wrapper) are used on the function object itself, it'll fetch the source of that instead of the actual command function. If you unwrap it it'll get to the right function through __wrapped__. I'm not sure if there's any deco present in lancebot that behaves like that now but it can't hurt when you're working around in the cog

sleek steppe May 16, 2021, 8:32 PM

#

ah, maybe we could just use Command.short_doc?

brazen charm May 16, 2021, 8:35 PM

#

The unwrapping is done in the part where it gets the url as it needs to get the right url https://github.com/python-discord/bot/blob/67934bd582ae5a6fac459c02ba3fa6c17d085d1a/bot/exts/info/source.py#L72-L74

stable mountainBOT May 16, 2021, 8:35 PM

#

bot/exts/info/source.py lines 72 to 74

source_item = inspect.unwrap(source_item.callback)
src = source_item.__code__
filename = src.co_filename```

sleek steppe May 16, 2021, 8:37 PM

#

ah that's what you mean

vale ibex May 16, 2021, 8:40 PM

#

!remind 30M iceman's PR

stable mountainBOT May 16, 2021, 8:40 PM

#

Yeah okay.

Your reminder will arrive in 30 minutes!

brazen charm May 16, 2021, 8:43 PM

#

sleek steppe ah, maybe we could just use Command.short_doc?

The unwrap didn't really have anything to do with it directly. I assume not using short_doc was done for a reason but I think the help command changed since that implemented so removing the special case may be the best.

sleek steppe May 16, 2021, 8:50 PM

#

I think it just wasn't changed when porting the source command, because it's using splitlines[1] when the doc string for the help command is one line

#

I think I'll remove the check since this looks fine

dry folio May 16, 2021, 9:01 PM

#

vocal wolf Thank you for the very large contribution!

THANK YOU! Z E T A P O G G E R SSSSS

dry folio May 16, 2021, 9:04 PM

#

vocal wolf <@!769221607888912487> merged.

just in time for our cat dev's one month mensiversary :D best present ever

stable mountainBOT May 16, 2021, 9:10 PM

#

@vale ibex

It has arrived!

Here's your reminder: iceman's PR.
[Jump back to when you created the reminder](#dev-contrib message)

eternal owl May 16, 2021, 9:12 PM

#

I am still around if you need any more changes, chris

#

haha it was always like that until it melted for a short time

thorny obsidian May 16, 2021, 9:17 PM

#

@eternal owlbtw not ignoring your DM, just getting my thoughts together so it's coherent

eternal owl May 16, 2021, 9:17 PM

#

👍

eternal owl May 16, 2021, 9:32 PM

#

hey @patent pivot, can I get an approval on this. It is regarding the soft delete of the off topic names. Implementation mentioned in the PR. Just making sure before I start working on it

#

https://github.com/python-discord/site/issues/490

GitHub

Re-roll off topic names backend changes. · Issue #490 · python-disc...

Bot side issue: python-discord/bot#1559 Context The re-roll command on the bot side re-names an off-topic channel and marks the previous off-topic channel name to be devoid of being used again. We ...

patent pivot May 16, 2021, 9:34 PM

#

having a look

eternal owl May 16, 2021, 9:34 PM

#

ty

patent pivot May 16, 2021, 9:35 PM

#

eternal owl https://github.com/python-discord/site/issues/490

looks good 👍 though I don't like the to_display name, I'd say we use something like active

eternal owl May 16, 2021, 9:35 PM

#

alright, will edit the issue description

vale ibex May 16, 2021, 9:37 PM

#

Hmmm, how can I shove data into metricity for testing this command

#

Since I'm in docker, not sure how I can get the metricity bot to hook into it

#

might need to run it on the host

#

Think I'm just gonna manually insert into db

#

probably easier

patent pivot May 16, 2021, 9:41 PM

#

lookin for some approvals on https://github.com/python-discord/site/pull/503

GitHub

Add Moderation Policies to redirects by jb3 · Pull Request #503 · p...

Redirects pages/code-of-conduct/policies/ to pages/code-of-conduct/.

tough imp May 16, 2021, 9:42 PM

#

vale ibex Hmmm, how can I shove data into metricity for testing this command

iirc when I tested something with metricity I execed into the postge container and ran insert queries

vale ibex May 16, 2021, 9:42 PM

#

Yea, that's what I'm doing now 😄

tough imp May 16, 2021, 9:43 PM

#

😌

#

theres also an init script in the site repo

patent pivot May 16, 2021, 9:43 PM

#

patent pivot lookin for some approvals on https://github.com/python-discord/site/pull/503

actually i'm just gonna force merge this lol

vale ibex May 16, 2021, 9:43 PM

#

tough imp theres also an init script in the site repo

Yea, I ran that one 😄

patent pivot May 16, 2021, 9:44 PM

#

easter egg in site#503 checks lol

dusky shoreBOT May 16, 2021, 9:44 PM

#

GitHub

PROpen Add Moderation Policies to redirects

vocal prairie May 16, 2021, 9:44 PM

#

Hmmm. shipit didn't show up in #dev-log

sleek steppe May 16, 2021, 9:44 PM

#

How did I get rickrolled by clicking "Show all checks" Sadge

patent pivot May 16, 2021, 9:45 PM

#

LMFAO

sleek steppe May 16, 2021, 9:45 PM

#

I'm never clicking links again

patent pivot May 16, 2021, 9:45 PM

#

my magnum opus, inserting rickrolls into our review requirement compliance checks

vale ibex May 16, 2021, 9:45 PM

#

...

#

too powerful

fervent sage May 16, 2021, 9:50 PM

#

patent pivot my magnum opus, inserting rickrolls into our review requirement compliance check...

this is why i dont touch links at all if there's any possibility that you could have touched it

patent pivot May 16, 2021, 9:50 PM

#

lmfao

fervent sage May 16, 2021, 9:50 PM

#

ive legit run links you sent before crosslink before to check

#

but now that ive told you that my technique will no longer work

sleek steppe May 16, 2021, 9:52 PM

#

it literally said "check overridden by joe" I should've listened 😔

patent pivot May 16, 2021, 9:55 PM

#

lmfao

fervent sage May 16, 2021, 10:04 PM

#

12 prs on bot

vale ibex May 16, 2021, 10:05 PM

#

8 of which are stalled/deferred/waiting on author/WIP

vocal prairie May 16, 2021, 10:11 PM

#

Maybe we can get site to 0.

#

It's at 4 right now

vale ibex May 16, 2021, 10:12 PM

#

hmmm site#474 and site#477 seem to be duplicates

dusky shoreBOT May 16, 2021, 10:12 PM

#

GitHub

PROpen Added a Favicon, and changed the 404-support text.
PROpen Added Favicon

vocal prairie May 16, 2021, 10:13 PM

#

Opened by the same person too

vale ibex May 16, 2021, 10:13 PM

#

or at least 477 is a subset

#

ahhh, their fork is deleted too

#

@patent pivot @gritty wind I see you both commented on the two above PRs, should we close them now that the fork has been deleted?

#

Yea, it doesn't really help that they lose order too

#

https://github.com/python-discord/sir-lancebot/blob/a5d138ef76fec71b2d737e24889c7de494fdbc0d/bot/exts/evergreen/issues.py#L246 ```diff

issues = set(issues)

issues = list(dict.fromkeys(issues))

stable mountainBOT May 16, 2021, 10:15 PM

#

bot/exts/evergreen/issues.py line 246

issues = set(issues)```

vale ibex May 16, 2021, 10:15 PM

#

simple change really

fallen patrol May 16, 2021, 10:15 PM

#

it used to note which was one

#

but it stopped noting which was which

patent pivot May 16, 2021, 10:16 PM

#

vale ibex <@!165023948638126080> <@!386599446571384843> I see you both commented on the tw...

probably ya

fallen patrol May 16, 2021, 10:16 PM

#

#

that format was better

vale ibex May 16, 2021, 10:17 PM

#

Feel free to open an issue to solve the ordering and formatting

brazen charm May 16, 2021, 10:26 PM

#

vale ibex https://github.com/python-discord/sir-lancebot/blob/a5d138ef76fec71b2d737e24889c...

I think if it's needed, unique_everseen communicates what's happening there much better than fromkeys. Not sure if lancebot has more-itertools but it is a tiny library

vale ibex May 16, 2021, 10:27 PM

#

brazen charm I think if it's needed, unique_everseen communicates what's happening there much...

Agreed, it doesn't but as you say it's small enough to add

#

And could be useful elsewhere

#

Sure 🙂

fervent sage May 16, 2021, 10:48 PM

#

btw are you planning to implement the issue too?

#

pensivewobble kk

#

!d dict.fromkeys

stable mountainBOT May 16, 2021, 10:54 PM

#

dict.fromkeys


classmethod fromkeys(iterable[, value])```
Create a new dictionary with keys from *iterable* and values set to *value*.

[`fromkeys()`](https://docs.python.org/3/library/stdtypes.html#dict.fromkeys "dict.fromkeys") is a class method that returns a new dictionary. *value* defaults to `None`. All of the values refer to just a single instance, so it generally doesn’t make sense for *value* to be a mutable object such as an empty list. To get distinct values, use a [dict comprehension](https://docs.python.org/3/reference/expressions.html#dict) instead.

#

@clever wraith :white_check_mark: Your eval job has completed with return code 0.

['a', 'b']

fervent sage May 16, 2021, 10:59 PM

#

so basically the dict is acting like an ordered set

#

since a dict can only have 1 value per key, setting it multiple times doesnt add new keys

#

and since iirc 3.6 dicts retain item order

#

when you call list you get the (ordered) keys of the dict, which are all unique

vale ibex May 16, 2021, 11:01 PM

#

fervent sage and since iirc 3.6 dicts retain item order

Yea, 3.6 implementation feature of CPython, 3.7 language feature

fervent sage May 16, 2021, 11:03 PM

#

what're the chances of me making a PR that makes everything into a standard storage format, by which i mean:

turning the 3 yaml files and 1 txt file into json files like every other file
making the indentation in the json files consistent 😩

vale ibex May 16, 2021, 11:03 PM

#

brazen charm I think if it's needed, unique_everseen communicates what's happening there much...

But as Numerlor said here, unique_everseen would read better

vocal prairie May 16, 2021, 11:03 PM

#

fervent sage what're the chances of me making a PR that makes everything into a standard stor...

Pretty high

fervent sage May 16, 2021, 11:03 PM

#

we dont have more_itertools

#

but if we did there's other places i can think of using it to shorten things

vale ibex May 16, 2021, 11:03 PM

#

fervent sage what're the chances of me making a PR that makes everything into a standard stor...

If suggest making an issue for that one to see what peoples appetite is

#

Yea it's a small package, I wouldn't mind having it

#

Could be useful

fervent sage May 16, 2021, 11:04 PM

#

more itertools is one of those tiny things that so many people who need it dont know about

#

specificially more_itertools.chunked()

vale ibex May 16, 2021, 11:04 PM

#

Oh yes

sleek steppe May 16, 2021, 11:23 PM

#

https://github.com/python-discord/sir-lancebot/tree/main/bot/resources/evergreen

fervent sage May 17, 2021, 12:18 AM

#

@patent pivot as of the no-opt-out pr metricity no longer has any commands, given that, is it cool if I make a PR:

removing bot_commands_channel config option, it's now redundent
removing command_prefix config option, there are no commands, prefix now set to ""
adding help_command=None to the contructor, else you just get this:

patent pivot May 17, 2021, 12:19 AM

#

fervent sage <@!165023948638126080> as of the no-opt-out pr metricity no longer has any comma...

greenTick

#

eventually

#

we might like

#

add commands to metricity to pull out charts

#

but for now, that sounds wise

fervent sage May 17, 2021, 12:26 AM

#

tfw you make a quick commit and it takes 10x longer because you accidentally spell import as impoty and have to fix it blob_pain

patent pivot May 17, 2021, 12:26 AM

#

lmfao

sleek steppe May 17, 2021, 12:27 AM

#

Shouldn't we just use discord.Client if we aren't using the commands extension?

patent pivot May 17, 2021, 12:28 AM

#

toxic making more work 😔

fervent sage May 17, 2021, 12:28 AM

#

can you think of that 5 minutes ago please :)

patent pivot May 17, 2021, 12:28 AM

#

eventually we will use commands

fervent sage May 17, 2021, 12:28 AM

#

thats a good enough reason for me, no point changing it again mmLol

short snow May 17, 2021, 12:54 AM

#

vocal wolf <@!736234578745884682> Will you be updating this sometime soon? <https://github...

yeah will do that, am thinking of merging all the files together, into just different classes, possibly game and cog, i don't really like the jumping around

dry folio May 17, 2021, 1:13 AM

#

patent pivot but ya prob should have policy of squashing if there are lint or fmting commits

oop- flashbacks to all the lint fail fixes and copy error fixes
precommit doesn't work 😢 (it says precommit command not found or smth)

patent pivot May 17, 2021, 1:13 AM

#

hmmmm interesting

dry folio May 17, 2021, 1:14 AM

#

but also, WE GOT MERGED BEFORE 200! WOOHOO

patent pivot May 17, 2021, 1:14 AM

#

next time you are playing with bot, try run poetry install and poetry run task precommit

short snow May 17, 2021, 1:15 AM

#

So i want to run this command before loading all the extensions, which is the best way I can do that?

#

umm, like htis?

fallen patrol May 17, 2021, 1:25 AM

#

👀 apparently I'll be continuing progress on the colors issue very soon

#

sorry i've been distracted and also trying to graduate from high school

dry folio May 17, 2021, 1:41 AM

#

patent pivot next time you are playing with bot, try run `poetry install` and `poetry run tas...

Doesn't it use pipenv?

vocal prairie May 17, 2021, 1:41 AM

#

#dev-announcements tl;dr They migrated to poetry and Python 3.9

patent pivot May 17, 2021, 1:42 AM

#

yeah, migrated as of a few hours ago

short snow May 17, 2021, 1:42 AM

#

short snow So i want to run this command before loading all the extensions, which is the be...

ping me when you respond

dry folio May 17, 2021, 1:44 AM

#

patent pivot yeah, migrated as of a few hours ago

Wow, so i'd have to reconstruct the entire virtual environment and config settings- woohoo!

#

Lol prolly won't be too long tbh

#

I'll do it tmr

patent pivot May 17, 2021, 1:44 AM

#

yeah it's not too big of a task

vocal prairie May 17, 2021, 1:44 AM

#

#dev-announcements has the exact 5 commands

patent pivot May 17, 2021, 1:44 AM

#

yeah

dry folio May 17, 2021, 1:45 AM

#

WAIT WAT IT'S THAT SIMPLE?!

#

Wow, i really overestimated the complexity

#

I thought I'd have to do a lot of manual crap

#

Thanks @vocal prairie and @patent pivot

patent pivot May 17, 2021, 1:48 AM

#

👍

vocal prairie May 17, 2021, 1:50 AM

#

pydis_strong

stable mountainBOT May 17, 2021, 2:03 AM

#

@short snow

It has arrived!

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

patent pivot May 17, 2021, 2:05 AM

#

hmmm

#

is that the dependencies PR

fervent sage May 17, 2021, 2:07 AM

#

stable mountain <@736234578745884682>

ill be honest i was hoping i'd be asleep before this reminder triggered

#

sleep schedules SadCatThumb

short snow May 17, 2021, 2:36 AM

#

patent pivot is that the dependencies PR

that's not needed rn right?

#

according to the discussions did yesterday

patent pivot May 17, 2021, 2:37 AM

#

yeah I don't think we should pursue dependency upgrades like that

short snow May 17, 2021, 2:37 AM

#

(for me)

#

alright, close the pr?

patent pivot May 17, 2021, 2:37 AM

#

I'll get a consensus from core devs and then close based on that or let you know otherwise

subtle kraken May 17, 2021, 3:42 AM

#

vocal wolf <@!431861021968302108> will you be going over the requested changes sometime soo...

I will whenever I will have time to go through it
Which is presumably later this week if school doesn't stop me once again

vocal wolf May 17, 2021, 3:42 AM

#

Thank you for the update

#

good luck with school

subtle kraken May 17, 2021, 3:42 AM

#

how do you answer so fast

#

wow

vocal wolf May 17, 2021, 3:43 AM

#

am writing boring essay

subtle kraken May 17, 2021, 3:43 AM

#

good luck friend

vocal wolf May 17, 2021, 3:43 AM

#

u2

dry folio May 17, 2021, 12:39 PM

#

nightmarish

clever wraith May 17, 2021, 12:54 PM

#

yo guys

#

i opened a issue about a anagrams minigame but no explicit approval / denial

gritty wind May 17, 2021, 1:15 PM

#

damn chris

vale ibex May 17, 2021, 1:16 PM

#

ah lol, you got in there just a few seconds before me

gritty wind May 17, 2021, 1:18 PM

#

We should really consider just pinning all these ideas in one place so they can stop being suggested

#

Ask tag, OSS channel

#

Probably 90% of suggestions for the help channels

vale ibex May 17, 2021, 1:18 PM

#

yea, just mentioned in #dev-core

mellow hare May 17, 2021, 1:38 PM

#

@vale ibex I'm trying to understand what change you wanted on bot#1587

dusky shoreBOT May 17, 2021, 1:38 PM

#

GitHub

PROpen Prioratize DM over channel message for voice verification ping.

vale ibex May 17, 2021, 1:39 PM

#

delete the two lines I marked, since the message is already being deleted in the lines I linked

#

otherwise the bot errors out, since 2 parts of the code try to delete the same message

#

Sorry if that wasn't clear in the comment

mellow hare May 17, 2021, 1:43 PM

#

Gotcha, okay

#

That's what I figured

vale ibex May 17, 2021, 1:53 PM

#

I've update the comment anyway

#

hopefully it's more clear now

#

shrugR

#

I did test further after making that change locally and it all looks good

clever wraith May 17, 2021, 2:11 PM

#

@sleek steppe , regarding your review, bot.Bot and bot.bot.Bot both fail, should i keep as commands.Bot?

sleek steppe May 17, 2021, 2:12 PM

#

What do you mean by "fail"?

#

from bot.bot import Bot

clever wraith May 17, 2021, 2:12 PM

#

shit

#

im super dumb

sleek steppe May 17, 2021, 2:12 PM

#

Also since we migrated to poetry, I think you need to run poetry lock since you updated dependancies

clever wraith May 17, 2021, 2:13 PM

#

i did

#

i made sure

sleek steppe May 17, 2021, 2:13 PM

#

Well, it isn't in your changed files

clever wraith May 17, 2021, 2:15 PM

#

I havent committed that yet

#

also @sleek steppe Commands.BadArgument does not accept an embed

sleek steppe May 17, 2021, 2:23 PM

#

I said that the error handler catches it

clever wraith May 17, 2021, 2:27 PM

#

it catches but it wont let me use the embed

#

oh wait nvm

clever wraith May 17, 2021, 2:43 PM

#

any changes you asked for have been made toxic

green oriole May 17, 2021, 2:47 PM

#

don’t make toxic changes D:

vale ibex May 17, 2021, 2:47 PM

#

lol

clever wraith May 17, 2021, 2:49 PM

#

green oriole *don’t make toxic changes D:*

Lmao

fervent sage May 17, 2021, 2:51 PM

#

green oriole *don’t make toxic changes D:*

was the spring cleanup pr a toxic change mmLol

vale ibex May 17, 2021, 2:57 PM

#

Looks like the git actions concurrency change is working nicely 😄

#

#dev-log

sleek steppe May 17, 2021, 2:58 PM

#

ah I was interested what that colour was about lol

vale ibex May 17, 2021, 2:58 PM

#

I made a change on Saturday, so git actions are cancelled if a new commit on the same PR comes in

#

since we don't care about that action any more, we only care about whether the new action passes/fails

#

limited to lance right now, just to test the waters

#

https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#concurrency and https://github.com/python-discord/sir-lancebot/blob/a5d138ef76fec71b2d737e24889c7de494fdbc0d/.github/workflows/build.yaml#L11-L13 if you're interested

stable mountainBOT May 17, 2021, 3:01 PM

#

.github/workflows/build.yaml lines 11 to 13

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true```

clever wraith May 17, 2021, 3:02 PM

#

Alright @sleek steppe , finished review(again)

sleek steppe May 17, 2021, 3:08 PM

#

It's still gonna raise Exception if you pass an empty string, so maybe you should change it to if not text.strip()

granite plaza May 17, 2021, 3:09 PM

#

Do i need to change anything from my recent pull request because of the recent change in #dev-announcements?

sleek steppe May 17, 2021, 3:10 PM

#

Nah, you're good

granite plaza May 17, 2021, 3:10 PM

#

K that's what i thought but wanted to make sure.

sleek steppe May 17, 2021, 3:11 PM

#

But you still need to resolve merge conflicts

clever wraith May 17, 2021, 3:11 PM

#

sleek steppe It's still gonna raise `Exception` if you pass an empty string, so maybe you sho...

alright

granite plaza May 17, 2021, 3:12 PM

#

I dont understand how to read the conflict log thing.
This is what was highlighted

<<<<<<< http_status_command_randomness
=======
from bot.bot import Bot
from bot.utils.extensions import invoke_help_command

>>>>>>> main

clever wraith May 17, 2021, 3:12 PM

#

@sleek steppe actually no, if there isnt any body given, it sends a default message

clever wraith May 17, 2021, 3:12 PM

#

granite plaza I dont understand how to read the conflict log thing. This is what was highlight...

for this, youll wanna delete anything under main and keep from your branch in most cases

sleek steppe May 17, 2021, 3:14 PM

#

granite plaza I dont understand how to read the conflict log thing. This is what was highlight...

You can keep the from bot.bot import Bot part and delete that other line. The lines above the ===== are the changes you made, the lines below it is the branch you're pulling from

granite plaza May 17, 2021, 3:15 PM

#

ok 👍

clever wraith May 17, 2021, 3:25 PM

#

thanks toxic, can people review sir-lancebot#699

dusky shoreBOT May 17, 2021, 3:25 PM

#

GitHub

PROpen Cowsay

mellow hare May 17, 2021, 3:27 PM

#

I swear to god, I can never figure out how to properly make a multi-line review comment work

clever wraith May 17, 2021, 3:27 PM

#

mellow hare I swear to god, I can never figure out how to properly make a multi-line review ...

they're so weird

#

you have to drag click

sleek steppe May 17, 2021, 3:27 PM

#

You drag down on the side of the line where the + is

clever wraith May 17, 2021, 3:27 PM

#

why cant you use ctrl+click to select multiple lines

mellow hare May 17, 2021, 3:27 PM

#

Right and that works within the GitHub one

#

But it doesn't work quite right in the PyCharm integration

short snow May 17, 2021, 3:40 PM

#

Yeah, on pycharm it is not supported ig, never figured it out

sleek steppe May 17, 2021, 3:44 PM

#

Doesn't work for me on the vscode github extension either

short snow May 17, 2021, 3:45 PM

#

Oo, I need to complete my review on cowsay

stable mountainBOT May 17, 2021, 3:45 PM

#

Missing required argument

content

short snow May 17, 2021, 3:45 PM

#

!remind 11h do now

stable mountainBOT May 17, 2021, 3:45 PM

#

Absolutely!

Your reminder will arrive in 11 hours!

green oriole May 17, 2021, 4:32 PM

#

Hey, that’s a good first start

fervent sage May 17, 2021, 4:34 PM

#

Yesterday I was setting up metricity for my server and I found some stuff in metricity that didn't need to be there while doing config so I made a pr to remove it and totally forgot to keep setting it up for my server KEKW

green oriole May 17, 2021, 4:34 PM

#

Hah

vocal prairie May 17, 2021, 4:34 PM

#

Hey, at least it was a good contribution

#

That's a good thing

gritty wind May 17, 2021, 4:38 PM

#

Yeah

short snow May 17, 2021, 4:38 PM

#

Yes

gritty wind May 17, 2021, 4:39 PM

#

We never really discussed it

vale ibex May 17, 2021, 4:39 PM

#

I don't see any reason not to

gritty wind May 17, 2021, 4:39 PM

#

The conversation was always in the context of the btos

#

but yeah, what chris said

vale ibex May 17, 2021, 4:39 PM

#

It'll simplify the contrib guide too

gritty wind May 17, 2021, 4:39 PM

#

yuuuup

vocal prairie May 17, 2021, 4:40 PM

#

Setting up the site with Docker is so easy, but with pipenv it can be a little more complicated, so this would be nice.

sleek steppe May 17, 2021, 4:40 PM

#

snekbox/nsjail.py:181:101: E501 line too long (113 > 100 characters)

😔

#

why is it even 100?

short snow May 17, 2021, 4:41 PM

#

up and then -d

#

I think so

vocal prairie May 17, 2021, 4:41 PM

#

docker-compose up -d

#

You need to modify your hosts file first though

short snow May 17, 2021, 4:42 PM

#

0.0.0.0:8080 mapped to pythondiscord.local

#

it’s on the site docs iirc

gritty wind May 17, 2021, 4:42 PM

#

You don't actually need to do that for the site

sleek steppe May 17, 2021, 4:42 PM

#

snekbox/nsjail.py:181:101: E501 line too long (101 > 100 characters)

😔

gritty wind May 17, 2021, 4:42 PM

#

It's more for the bots

vocal prairie May 17, 2021, 4:42 PM

#

gritty wind You don't actually need to do that for the site

Oh you don't?

#

Going to 0.0.0.0:8000 doesn't work for me

gritty wind May 17, 2021, 4:43 PM

#

the bots can't access something like api.localhost, but your browser should be able to

sleek steppe May 17, 2021, 4:43 PM

#

vocal prairie Going to 0.0.0.0:8000 doesn't work for me

127.0.0.1:8000 works for me though

gritty wind May 17, 2021, 4:43 PM

#

Yeah ^

#

or localhost

vocal prairie May 17, 2021, 4:43 PM

#

Oh, I should do that

gritty wind May 17, 2021, 4:43 PM

#

Anyways, it's easy enough to do and get out of the way

sleek steppe May 17, 2021, 4:44 PM

#

sleek steppe ``` snekbox/nsjail.py:181:101: E501 line too long (101 > 100 characters) ``` 😔

Shouldn't this be 120 characters?

short snow May 17, 2021, 4:44 PM

#

No /etc/hosts

vocal prairie May 17, 2021, 4:44 PM

#

You don't even need it if you're running with Docker apparently

short snow May 17, 2021, 4:44 PM

#

Or something similar, don’t remember the file name

gritty wind May 17, 2021, 4:44 PM

#

https://pythondiscord.com/pages/guides/pydis-guides/contributing/hosts-file/

#

It's linked in all the contributing guides

#

That should be it

#

yeah

short snow May 17, 2021, 4:45 PM

#

Yeah just write it towards the end

gritty wind May 17, 2021, 4:45 PM

#

sleek steppe Shouldn't this be 120 characters?

https://github.com/python-discord/snekbox/blob/main/.flake8#L7

stable mountainBOT May 17, 2021, 4:45 PM

#

.flake8 line 7

max-line-length = 100```

gritty wind May 17, 2021, 4:45 PM

#

the other projects do use 120

sleek steppe May 17, 2021, 4:45 PM

#

yeah, I was wondering why it isn't 120 like the other projects

gritty wind May 17, 2021, 4:46 PM

#

hit save and you're good

vale ibex May 17, 2021, 4:50 PM

#

There may be more to it, since we want to move to 3.9 at the same time

#

the site was previously on 3.9, but we had to downgrade

#

I'd prefer if you leave that to us for now

#

It was a hotfix, rather than linked to a PR, so we're still trying to figure out why it was needed

#

since it was done a few months ago

#

So we're just going to look into why the downgrade was needed, and if its still an issue

vocal prairie May 17, 2021, 4:53 PM

#

If you want something to do something on the site Rick, site#459 would be nice to have

dusky shoreBOT May 17, 2021, 4:53 PM

#

GitHub

IssueOpen Edit on GitHub for content articles.

gritty wind May 17, 2021, 4:53 PM

#

Could you take a screenshot?

#

Can't say I've run into that

#

Have you tried the support forums?

#

what address did you go to, and what's the output of docker container ps

vocal prairie May 17, 2021, 4:57 PM

#

Does http://pythondiscord.local:8000 work?

sleek steppe May 17, 2021, 4:57 PM

#

yes

vocal prairie May 17, 2021, 4:57 PM

#

sleek steppe yes

I was asking if it worked for Rick

sleek steppe May 17, 2021, 4:57 PM

#

ah

gritty wind May 17, 2021, 4:57 PM

#

🥳

vocal prairie May 17, 2021, 4:58 PM

#

Depends on what you want to do in the site

#

But probably not, most of what you need shouldn't be much more than the template language and knowledge of html/bulma

short snow May 17, 2021, 4:59 PM

#

Then not really, but you should know have basic django knowledge

vocal prairie May 17, 2021, 5:01 PM

#

Yeah, but the thing where it renders the markdown is what you want

#

Why do you need it for admins only?

#

That button should appear for everyone

#

Not everyone has write access

#

See ya Rick

sleek steppe May 17, 2021, 5:20 PM

#

@vale ibex shouldn't we update the site in this PR to say poetry intead of pipenv in the guides?

#

For example, https://github.com/python-discord/site/blob/main/pydis_site/apps/content/resources/guides/pydis-guides/contributing/installing-project-dependencies.md#L23

stable mountainBOT May 17, 2021, 5:22 PM

#

pydis_site/apps/content/resources/guides/pydis-guides/contributing/installing-project-dependencies.md line 23

5. If installing dependencies for the site, click `Pipenv Environment`, otherwise, click `Poetry Environment`, then click `OK`.```

vale ibex May 17, 2021, 5:23 PM

#

Yes, we should 😄

sleek steppe May 17, 2021, 5:24 PM

#

Also snekbox#108 👀

dusky shoreBOT May 17, 2021, 5:24 PM

#

GitHub

PROpen feat: Allow custom arguments to be passed to eval

gritty wind May 17, 2021, 5:24 PM

#

I could get you updated screenshots

vale ibex May 17, 2021, 5:25 PM

#

Yes please 😄 I don't use pycharm

gritty wind May 17, 2021, 5:25 PM

#

okey

#

did you push the branch yet?

vale ibex May 17, 2021, 5:25 PM

#

I have

gritty wind May 17, 2021, 5:26 PM

#

oh right

#

I have the PR open lmao

vale ibex May 17, 2021, 5:26 PM

#

lol

sleek steppe May 17, 2021, 5:26 PM

#

lol

vale ibex May 17, 2021, 5:26 PM

#

but if you push to branch you lose voting rights lol

#

ah, it doesn't matter anyway, since we need devops

gritty wind May 17, 2021, 5:26 PM

#

I'll just send you the screenshot lol

vale ibex May 17, 2021, 5:26 PM

#

kk

gritty wind May 17, 2021, 5:26 PM

#

time to test my migration guide

vale ibex May 17, 2021, 5:27 PM

#

lol

#

Do you use a fork? 🤔

gritty wind May 17, 2021, 5:27 PM

#

nah lol

#

plus pycharm can just pull in PR branches with one button

vale ibex May 17, 2021, 5:28 PM

#

That's what aliases are for 😄

#

git cpr 505

#

easy

gritty wind May 17, 2021, 5:28 PM

#

hahaha

#

I'll be back once the install is done

vale ibex May 17, 2021, 5:28 PM

#

cpr = "!cd ${GIT_PREFIX:-.} && git fetch origin refs/pull/${1}/head && git checkout FETCH_HEAD #"

sleek steppe May 17, 2021, 5:29 PM

#

gh pr checkout 505

vale ibex May 17, 2021, 5:29 PM

#

!e print( len("gh pr checkout 505") - len("git cpr 505") )

stable mountainBOT May 17, 2021, 5:29 PM

#

@vale ibex :white_check_mark: Your eval job has completed with return code 0.

vale ibex May 17, 2021, 5:29 PM

#

smh 7 precious key stokes longer

sleek steppe May 17, 2021, 5:30 PM

#

cpr = "gh pr checkout"

vale ibex May 17, 2021, 5:30 PM

#

lol

vale ibex May 17, 2021, 5:30 PM

#

gritty wind I'll be back once the install is done

It's not instant 👀

#

takes like 10s for me lol

gritty wind May 17, 2021, 5:31 PM

#

I can count the KBs of my internet speed on my hands

#

It'll take a minute

vale ibex May 17, 2021, 5:31 PM

#

ah lol

#

Just download the internet shrugR

gritty wind May 17, 2021, 5:32 PM

#

My god, you are a genius

fallen patrol May 17, 2021, 5:32 PM

#

vale ibex git cpr 505

don't kill the computer

#

😄

vale ibex May 17, 2021, 5:33 PM

#

nono, it's cpr, so I'm saving it

#

I've updated the docs now anyway, so just need to drop in the images

#

gonna head out to the shop to buy some ingredients now though

#

back in 20

gritty wind May 17, 2021, 5:39 PM

#

you can copy this straight into the root directory, and it'll work as expected, but you may also wanna rename the file from pipenv

📎 virus.rar

granite plaza May 17, 2021, 5:40 PM

#

sleek steppe You can keep the `from bot.bot import Bot` part and delete that other line. The ...

The line you say to keep is unused in the file. So I shouldent keep it right (becouse it fails the lint)

vale ibex May 17, 2021, 5:41 PM

#

Lmfao

sleek steppe May 17, 2021, 5:42 PM

#

granite plaza The line you say to keep is unused in the file. So I shouldent keep it right (be...

Did you pull correctly? Because it's used here https://github.com/python-discord/sir-lancebot/blob/main/bot/exts/evergreen/status_codes.py#L16

stable mountainBOT May 17, 2021, 5:42 PM

#

bot/exts/evergreen/status_codes.py line 16

def __init__(self, bot: Bot):```

granite plaza May 17, 2021, 5:45 PM

#

weird. my version had def __init__(self, bot: commands.Bot):

clever wraith May 17, 2021, 5:46 PM

#

gritty wind you can copy this straight into the root directory, and it'll work as expected, ...

hey scaleios, thanks for the file, however it seems to have changed my computers background to rick astley and plays the never gonna give you up intro on boot. Any way to fix this?

gritty wind May 17, 2021, 5:47 PM

#

Oh yea, sorry about that. I'll DM you the number for Microsoft support, and they should be able to fix it remotely for ya

clever wraith May 17, 2021, 5:47 PM

#

hey thanks

#

how convenient, it seems to have also installed teamviewer

vale ibex May 17, 2021, 5:49 PM

#

Smh, you're really gonna make me download winrar aren't you 😅

gritty wind May 17, 2021, 5:49 PM

#

I'll send you the raw images lol

vale ibex May 17, 2021, 5:50 PM

#

If I get an email from the licence police its on you 😅

clever wraith May 17, 2021, 5:50 PM

#

vale ibex Smh, you're really gonna make me download winrar aren't you 😅

@vale ibex you can open them from 7z for free

gritty wind May 17, 2021, 5:50 PM

#

do you legitimately not have any archiving software lmao

#

#

vale ibex May 17, 2021, 5:51 PM

#

Oh I didn't realise 7zip supported rar files

#

Lol

#

I'm not at pc anyways

clever wraith May 17, 2021, 5:51 PM

#

vale ibex Oh I didn't realise 7zip supported rar files

it supports damn nearly any compression type under the sun, you can extract .docx files

#

or rather the xml

vale ibex May 17, 2021, 5:52 PM

#

Lol

clever wraith May 17, 2021, 5:52 PM

#

well im gonna go for a walk before class, see yall later

gritty wind May 17, 2021, 5:53 PM

#

laters

granite plaza May 17, 2021, 5:53 PM

#

sleek steppe But you still need to resolve merge conflicts

Well i think i finished this for sir-lancebot#733

dusky shoreBOT May 17, 2021, 5:53 PM

#

GitHub

PROpen Http status command randomness

sleek steppe May 17, 2021, 5:55 PM

#

yeah, a core dev has to approve before this gets merged

granite plaza May 17, 2021, 5:56 PM

#

sleek steppe yeah, a core dev has to approve before this gets merged

lemon_fingerguns_shades

vocal prairie May 17, 2021, 5:56 PM

#

Noice

#

What page is that?

#

The site uses Bulma just so you know

#

It shouldn't be too much css

#

https://bulma.io

fallen patrol May 17, 2021, 6:00 PM

#

#

@vale ibex btw when you update the guide, snekbox still uses pipfile

#

anyways, i can't start my snekbox

sleek steppe May 17, 2021, 6:02 PM

#

a lot of our projects still use pipenv

fallen patrol May 17, 2021, 6:02 PM

#

ik what you meant smh

vocal prairie May 17, 2021, 6:02 PM

#

Yeah uh, I realized I was talking to the wrong person

fallen patrol May 17, 2021, 6:02 PM

#

however, i can't start it regardless

gritty wind May 17, 2021, 6:02 PM

#

Pipenv is not used by any projects in the contrib guides

#

Namely: lance, site, bot

fallen patrol May 17, 2021, 6:03 PM

#

okay so anyways how do i start my snekbox

vale ibex May 17, 2021, 6:03 PM

#

fallen patrol anyways, i can't start my snekbox

what version of docker-compose do you have?

fallen patrol May 17, 2021, 6:03 PM

#

😳

#

#

how do i update docker compose*

gritty wind May 17, 2021, 6:03 PM

#

strip the path, and replace it with github prefix

vale ibex May 17, 2021, 6:04 PM

#

fallen patrol how do i update docker compose*

what os?

fallen patrol May 17, 2021, 6:04 PM

#

That feeling when you realize you use debian so everything is out of date all of the time

#

~~debipain~~ debian lol

gritty wind May 17, 2021, 6:04 PM

#

- https://pythondiscord.com/pages/guides/pydis-guides/contributing/
+ https://github.com/python-discord/site/tree/main/pydis_site/apps/content/resources/guides/pydis-guides/contributing```

#

hmm

#

can probably get you a better diff than that

#

Site prefix: https://pythondiscord.com/pages
Github prefix: https://github.com/python-discord/site/tree/main/pydis_site/apps/content/resources

vale ibex May 17, 2021, 6:05 PM

#

fallen patrol That feeling when you realize you use debian so everything is out of date all of...

how did you install it?

#

regardless, this will be the answer most likely https://stackoverflow.com/a/49839172

fallen patrol May 17, 2021, 6:06 PM

#

apt

#

lol

gritty wind May 17, 2021, 6:06 PM

#

Is that the banner?

fallen patrol May 17, 2021, 6:06 PM

#

probably

gritty wind May 17, 2021, 6:06 PM

#

We wanted to remove it completely

fallen patrol May 17, 2021, 6:06 PM

#

that should honestly be expedited

gritty wind May 17, 2021, 6:06 PM

#

we removed it on desktop

vale ibex May 17, 2021, 6:06 PM

#

ugggh pain

gritty wind May 17, 2021, 6:06 PM

#

it's there on mobile

#

wtf lol

fallen patrol May 17, 2021, 6:07 PM

#

vale ibex ugggh pain

why is that image so small wtf

vale ibex May 17, 2021, 6:07 PM

#

there aren't new lines between the images

#

and since the middle one is smaller than width, it's in-line with text

gritty wind May 17, 2021, 6:07 PM

#

yeah just remove that lol

#

do you want a different size?

patent pivot May 17, 2021, 6:08 PM

#

we should try make the timeline visible in some other way on mob

vale ibex May 17, 2021, 6:08 PM

#

nah, it'll likely be an issue on different sizes anyway, I'll just throw in line breaks

gritty wind May 17, 2021, 6:08 PM

#

patent pivot we should try make the timeline visible in some other way on mob

we can keep that banner

patent pivot May 17, 2021, 6:08 PM

#

but probably not through that banner thing

gritty wind May 17, 2021, 6:08 PM

#

just remove 100

vocal prairie May 17, 2021, 6:08 PM

#

patent pivot we should try make the timeline visible in some other way on mob

It's in the navbar now

patent pivot May 17, 2021, 6:08 PM

#

yeah, but we can make it more visible than navbar

#

I would have thought we can pivot this to have that righthand box on mobile as well

fallen patrol May 17, 2021, 6:10 PM

#

fallen patrol

patent pivot May 17, 2021, 6:11 PM

#

yeah, that sounds wise

vocal prairie May 17, 2021, 6:11 PM

#

Does Bulma support css grid format? I'm trying to line up the event boxes on the event page

#

Yup, that's a Bulma thing

fallen patrol May 17, 2021, 6:12 PM

#

~~heads up if you send so many seperate messages you'll be muted automatically~~

vocal prairie May 17, 2021, 6:13 PM

#

is-hidden-mobile I think, lemme check

#

Yeah, it's is-hidden-mobile if you want Desktop only

fallen patrol May 17, 2021, 6:14 PM

#

lol @vale ibex now its worse. Much worse.

vocal prairie May 17, 2021, 6:16 PM

#

Much better

patent pivot May 17, 2021, 6:22 PM

#

it has django admin, we just disabled it

#

we did have discord auth

vocal prairie May 17, 2021, 6:23 PM

#

The site is migrating to FastAPI right?

patent pivot May 17, 2021, 6:23 PM

#

It is, yeah

#

FastAPI w/ SQLAlchemy iirc

vocal prairie May 17, 2021, 6:24 PM

#

Hmmm. How do I make this not look ugly

#

The events page redesign

#

I know they all look like the same thing

#

But my problem is that the things are stuck together

dusky shoreBOT May 17, 2021, 6:25 PM

#

GitHub

IssueOpen Hacktoberfest branding asset link is dead.

vocal prairie May 17, 2021, 6:25 PM

#

If you want to you can

#

But I'll fix it anyway

#

I know how to do it

#

But columns aren't really responsive enough

#

CSS grid would be better

#

Uh, just changing the size of the window

#

I use Chrome

#

And it's inbuilt mobile feature too

#

Maybe I'll manually implement breakpoints and use that for columns

#

Nah, Bulma has breakpoints built in

#

The only thing is the "Show More" button

clever wraith May 17, 2021, 6:30 PM

#

hey, looking for a good issue as a first time contributor for the bot repo

#

any good ones to try out?

vale ibex May 17, 2021, 6:31 PM

#

That's annoying

#

adding line breaks restarts the numbering

#

is there md syntax to make it not?

patent pivot May 17, 2021, 6:32 PM

#

@fervent sage the tag content looks great, but I think that now we've got the guides pipeline unblocked on site we said a while ago that we'd do short tags on bot and if they needed longer form content we'd link to a guide on site instead of having huge embeds on Discord. We're more free with site guides as we can have sections with a ToC, images and so on (example: https://pythondiscord.com/pages/guides/python-guides/mutability/). Would you be willing to convert the proposed tag to a guide and then shorten the bot tag to link through to the site?

sleek steppe May 17, 2021, 6:32 PM

#

clever wraith hey, looking for a good issue as a first time contributor for the bot repo

There's bot#1588 👀

dusky shoreBOT May 17, 2021, 6:32 PM

#

GitHub

IssueOpen Modpings schedule feature

clever wraith May 17, 2021, 6:32 PM

#

alright

patent pivot May 17, 2021, 6:33 PM

#

I think modpings might be a bit too low level for a brand new contributor

fervent sage May 17, 2021, 6:33 PM

#

patent pivot <@!297045071457681409> the tag content looks great, but I think that now we've g...

!remind 2h this

stable mountainBOT May 17, 2021, 6:33 PM

#

You got it!

Your reminder will arrive in 2 hours!

clever wraith May 17, 2021, 6:33 PM

#

id be fine, except ive no idea how to work with redis

patent pivot May 17, 2021, 6:33 PM

#

ty vco very pog

placid ermine May 17, 2021, 6:34 PM

#

speaking of tags
I suggested this a few days ago in #community-meta, could we have a tag for why eval/exec shouldn't normally be used?

#

psvm said nedbat has a blog post about this

dusky shoreBOT May 17, 2021, 6:35 PM

#

GitHub

IssueOpen Add a more centered invite link

patent pivot May 17, 2021, 6:35 PM

#

clever wraith hey, looking for a good issue as a first time contributor for the bot repo

I'd recommend going through https://github.com/python-discord/bot/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+no%3Aassignee and asking about any that interest you.

GitHub

python-discord/bot

The community bot for the Python Discord community - python-discord/bot

patent pivot May 17, 2021, 6:36 PM

#

placid ermine speaking of tags I suggested this a few days ago in <#429409067623251969>, could...

I'm alright with that, I don't think it needs to be super long, as you mentioned we can link through to https://nedbatchelder.com/blog/201206/eval_really_is_dangerous.html

Eval really is dangerous

eval() is tempting, but it’s truly dangerous. Here’s how common safety attempts can be circumvented.

placid ermine May 17, 2021, 6:37 PM

#

yeah

patent pivot May 17, 2021, 6:38 PM

#

hmm

#

i'm not sure about bot#1111

dusky shoreBOT May 17, 2021, 6:38 PM

#

GitHub

IssueOpen Write a new !ask tag

patent pivot May 17, 2021, 6:39 PM

#

We already have pretty much what we'd put at the top of an ask tag in the help embeds

#

#

and we link through to https://pythondiscord.com/pages/asking-good-questions/, which we wrote about exactly that

clever wraith May 17, 2021, 6:39 PM

#

damn that was one i was considering

patent pivot May 17, 2021, 6:40 PM

#

yeah, your comment was what prompted my thinking, apologies

#

but yeah, it's less so the content of the ask tag, and more how it is used

#

when we had an ask tag we just constantly saw things like !ask @user with no preample

fervent sage May 17, 2021, 6:40 PM

#

patent pivot <@!297045071457681409> the tag content looks great, but I think that now we've g...

on this, does that mean i can go into a bit more detail and add some extra info too since i dont need to account for embed size

patent pivot May 17, 2021, 6:40 PM

#

yeah for sure

#

sections, images, other markdown features are all fine in guides

#

you get a ToC autogenerated so you can split it logically and have it longer

clever wraith May 17, 2021, 6:42 PM

#

joe, for bot#1588, im assuming id be using async-rediscache. Where are the docs kept for it?

dusky shoreBOT May 17, 2021, 6:42 PM

#

GitHub

IssueOpen Modpings schedule feature

patent pivot May 17, 2021, 6:43 PM

#

I don't think there are any hosted docs at the moment, but you can inspect the source code at https://github.com/SebastiaanZ/async-rediscache, the best source of docs is looking at how other cogs use it in the bot

clever wraith May 17, 2021, 6:43 PM

#

alright, thanks!

#

ill give bot#1588 a try

dusky shoreBOT May 17, 2021, 6:43 PM

#

GitHub

IssueOpen Modpings schedule feature

patent pivot May 17, 2021, 6:44 PM

#

👍, if you leave a comment there then I can assign you

clever wraith May 17, 2021, 6:45 PM

#

actually looking at the command i think its beyond my ability atm

patent pivot May 17, 2021, 6:46 PM

#

that's fine, feel free to take another look, or maybe look at lancebot issues as well

clever wraith May 17, 2021, 6:48 PM

#

patent pivot that's fine, feel free to take another look, or maybe look at lancebot issues as...

ive been working on lancebot, seeing if theres anything to do in the bot repo

#

i decided to look at bot#1316

dusky shoreBOT May 17, 2021, 6:52 PM

#

GitHub

IssueOpen Command to teach how to add to Path

vocal prairie May 17, 2021, 6:54 PM

#

That sounds like it would be tag, not a normal command

clever wraith May 17, 2021, 6:55 PM

#

vocal prairie That sounds like it would be tag, not a normal command

ik, ive never done anything with the bot repo before and wanna start off easy

vocal prairie May 17, 2021, 7:00 PM

#

It's lined up for me on pydis.com

#

🤷‍♂️

clever wraith May 17, 2021, 7:23 PM

#

hey, dockers being pissy with the bot, telling me that docker is not started. what am i supposed to do?

green oriole May 17, 2021, 7:25 PM

#

sudo systemctl start docker

#

If you are on Linux

vocal prairie May 17, 2021, 7:26 PM

#

If you're on Windows, just open Docker Desktop

clever wraith May 17, 2021, 7:28 PM

#

vocal prairie If you're on Windows, just open Docker Desktop

thats the problem, scoops a bitch and it didnt install docker desktop

vale ibex May 17, 2021, 7:33 PM

#

https://github.com/python-discord/site/pull/505 is ready

#

will need a devops review

patent pivot May 17, 2021, 7:33 PM

#

devops

#

i am devops

sleek steppe May 17, 2021, 7:34 PM

#

yes

gritty wind May 17, 2021, 7:34 PM

#

left a comment on the PATH tag

#

Could a core dev look at it and remove the planning label?

#

(bot#1316)

dusky shoreBOT May 17, 2021, 7:35 PM

#

GitHub

IssueOpen Command to teach how to add to Path

brazen charm May 17, 2021, 7:36 PM

#

Isn't there a tag for it already?

vale ibex May 17, 2021, 7:36 PM

#

There's uhh

#

!venv

#

!t

stable mountainBOT May 17, 2021, 7:36 PM

#

**Current tags**

» voice-verification
» windows-path
» with
» xy-problem
» ytdl
» zip

vale ibex May 17, 2021, 7:36 PM

#

!windows-path

stable mountainBOT May 17, 2021, 7:36 PM

#

PATH on Windows

If you have installed Python but you forgot to check the Add Python to PATH option during the installation you may still be able to access your installation with ease.

If you did not uncheck the option to install the Python launcher then you will find a py command on your system. If you want to be able to open your Python installation by running python then your best option is to re-install Python.

Otherwise, you can access your install using the py command in Command Prompt. Where you may type something with the python command like:

C:\Users\Username> python3 my_application_file.py

You can achieve the same result using the py command like this:

C:\Users\Username> py -3 my_application_file.py

You can pass any options to the Python interpreter after you specify a version, for example, to install a Python module using pip you can run:

C:\Users\Username> py -3 -m pip install numpy

You can also access different versions of Python using the version flag, like so:

C:\Users\Username> py -3.7
... Python 3.7 starts ...
C:\Users\Username> py -3.6
... Python 3.6 stars ...
C:\Users\Username> py -2
... Python 2 (any version installed) starts ...

gritty wind May 17, 2021, 7:37 PM

#

Ah

#

The issue is a bit different

#

It's specifically about how to add to PATH

#

but I don't particularly think it's needed either way

#

since my advice is rerun the installer and select the option

#

which can be typed out in 5 seconds

brazen charm May 17, 2021, 7:38 PM

#

There is a script that comes with python that can be used to do it without reinstalling (on win), may be good to note in a tag. But with path issues there's usually more to it than just adding python's folders to path

clever wraith May 17, 2021, 7:40 PM

#

patent pivot i am devops

joe is one and all