#dev-contrib

1 messages · Page 61 of 1

crude gyro
#

!otn a hemlock's pipe dreams

stable mountainBOT
#

:ok_hand: Added hemlock’s-pipe-dreams to the names list.

mellow hare
#

I have to grow them in the field because I can't grow them on my face

obsidian patio
#

Here we have a visual of what it could look like

#

This is built on square pieces (except for the face, but we'll fix that)

#

@mellow hare @crude gyro

crude gyro
#

this is fun, but it's simple

obsidian patio
#

And this one assumes that we fill up the squares completely. There is an alternative, which could look better

crude gyro
#

I think more entropic and uneven tiles would look better

obsidian patio
#

Something like a slight curve on the edges?

crude gyro
#

this looks like a dirty yellow centipede

obsidian patio
crude gyro
#

MAKE IT STOP

obsidian patio
#

😆

crude gyro
#

that's fun, though

mellow hare
#

I think the tip of the beard is....

obsidian patio
#

This is just so confusing to the brain. I don't know if I love or hate it

mellow hare
#

Needs some work

crude gyro
#

I'm not a fan of the sideways bases though

#

they make me wanna turn my head

mellow hare
#

I still think it should be the slight angle we're used to but it turns the beard directly to the right

#

Or whichever direction

obsidian patio
#

I'll give that a try. It's probably a good idea

crude gyro
#

it's harder

#

but that's often a good sign

late wolf
#

it seems the meme of this channel is lemon's beard

mellow hare
#

Meme of the server more like

#

But in fairness, it truly is long and majestic

crude gyro
obsidian patio
#

Woww

#

That's not even a beard

#

We're gonna need longer emojis

crude gyro
#

it appears my most excellent co-owner @hardy gorge has gone ahead and created #dev-branding for us to discuss lemojis, ducks, animated server icons, banners, and other pydis assets in, gentlemen.

#

so let's make sure we use that

short snow
#

lemon is tha you?

crude gyro
#

sure is

hardy gorge
#

That's lemon's beard

short snow
#

such a long beard

obsidian patio
#

Here's a concept for making it go sideways, but it would be a lot more difficult than having a 90° or 0° rotation

#

Aah, that's awesome!

long brook
#

you could make it smoother without the lines

obsidian patio
#

They wouldn't be there when it's ready. We moved the conversation to #dev-branding, though

green mesa
#

Bot is giving same pride fact over and over again😅

#

.pridefact

short snow
#

yes it does

#

it changes everyday

#

that's what some mod said

#

i don't remeber who

green mesa
#

But he isn't sure 100%

#

Groot

#

Said

obsidian patio
#

Look at the code if you wanna know

green mesa
#

Umm ok but ......

obsidian patio
#

But what?

eternal owl
green oriole
#

@green mesa it changes daily, they are right

vocal wolf
late wolf
tawdry vapor
#

Sorry, but why are you posting this here? Seems off topic.

ancient salmon
#

hi

obsidian patio
green mesa
#

i asked in python genral but no one responded

long brook
#

the help channels are for help

#

just claim one of them

green mesa
#

tried again in python genral and got help from wattle

#

now problem solved 😄

#

@long brook

long brook
#

nice

mellow hare
tough imp
#

Hi I will work on it today

#

I made the changes a few days ago but I hated the way it turned out so I didnt push yet

#

going to give it another try

green oriole
#

We won’t reach 100k any time soon, will we

obsidian patio
#

I talked to Joe about that a few days back. We aren't planning on hitting it before "the purge"

patent pivot
#

we have a day and a bit

green oriole
#

You always !int e the purge

patent pivot
#

huh?

green oriole
#

You can use an internal eval to do the purge if the PR isn’t merged yet, using a little script

mellow hare
#

@tough imp No worries! I just appreciate you taking the time

patent pivot
#

ah right

#

we could

green mesa
#

i am waiting for one more check to pass only 1 check left on my pr at seasonal bot

#

😄

#

then wiki command will be added

obsidian patio
neon phoenix
#

@obsidian patio there actually are

#

click the title of the card

obsidian patio
#

I thought that would bring me to github

#

Maybe specify that it'll take you to the guide?

green oriole
#

It isn't the first time that I hear about someone having issues figuring out how to access the guides tbh

obsidian patio
#

I could open an issue about it on the site. Feels like something we should fix

green oriole
#

Yup

obsidian patio
#

Or if someone else could do it? I'm working on the ultimate lemoji beard right now

stiff marsh
#

Is this channel for contribution to Python or Python?

#

Like CPython or @stable mountain?

subtle kraken
#

Its about talks regarding contributions to our projects

#

Be it SeasonalBot, site, @stable mountain

obsidian patio
#

@stiff marsh I think #internals-and-peps would be the place for discussing contributions to CPython. Read the description there

sullen phoenix
#

yeah we are aware the UX of the site is not great

obsidian patio
#

Yeah hahah. I’ll just add the fact that the menu is hidden, while the social links are prominent

ancient salmon
#

hi

#

can anyone tell me Define statement

#

it means

#

def

green mesa
obsidian patio
#

The bot seems to have been quite slow at closing help channels today. Takes a few seconds

#

Here’s a case when someone closed it twice, with two responses from @stable mountain

patent pivot
#

The verification process is running, so ratelimits will be tight for a few hours

obsidian patio
#

Aah, alright. That explains things

#

A help channel just opened at the top of the list. Was that an admin moving it or was it an issue with @stable mountain?

subtle kraken
#

We always have 2 help channels available so not sure whats wrong

obsidian patio
#

When it was claimed, or sometime around there, it was at the top of the occupied category @subtle kraken

subtle kraken
#

Interesting

obsidian patio
#

Maybe it was an admin moving it. Otherwise, it’s very, very odd

subtle kraken
#

I suppose it could be some issue on discord API side of things

obsidian patio
#

How could that happen? Would Discord just place it somewhere other than instructed?

subtle kraken
#

It could be accidental drag but in such case it shouldnt matter too much

#

Once the channel gets closed it should all be good again

wild ridge
#

ya it's most likely someone's misclick

obsidian patio
#

Yeah

hardy gorge
#

Could be

#

If it happens more frequently we could introduce a bit of logging of "position after the move" in that category

#

Or a check if it's in the correct position, just to make sure it's not a bug on our side somewhere

obsidian patio
#

@hardy gorge Can't you see that in the audit log? At 12:30, the channel was opened, I believe

subtle kraken
#

discord doesn't log moving of positions

obsidian patio
#

Really? I thought it did

patent pivot
#

nope

subtle kraken
#

it'd be very spammy, especially here

#

channel position isn't really too important

patent pivot
#

(as in, 5 minute cache)

sullen phoenix
#

i'm surprised our member count is still moving up

obsidian patio
#

It has actually started to decrease, it seems

#

I checked the count in #bot-commands a few minutes back

sullen phoenix
#

that would make more sense

obsidian patio
patent pivot
#

loll

obsidian patio
#

It's going down for real

patent pivot
sullen phoenix
#

lmfao

#

i realized i was looked at a month instead of day

patent pivot
#

localStorage caught you out

obsidian patio
#

This is officially bullying joe

sullen phoenix
#

i like grafana better

obsidian patio
sullen phoenix
subtle kraken
#

I to agree grafana is better for stats

patent pivot
#

grafana will update faster yeah

subtle kraken
#

no I mean it looks nicer

patent pivot
#

😔

green oriole
#

The 10k purge has started?

subtle kraken
#

it did start

sullen phoenix
#

it'sa 15k purge lol

subtle kraken
#

we are optimistic of 5000 people realising what is going on /s

green oriole
#

The Great Purge

#

it'sa 15k purge lol
@sullen phoenix eeeh the issue said 10k lol

short snow
#

so 15k ppl get kicked out? in this

subtle kraken
#

could be miscalculation slash we got 5000 people out of nowhere

short snow
#

i dunno what is purge

subtle kraken
#

we are kicking unverified people

sullen phoenix
#

we've grown a lot since then

short snow
#

we are kicking unverified people
@subtle kraken okh

#

but what about those who just got in?

subtle kraken
#

if you stay in our server for 30d without verifying yourself you will be informed that you were kicked, why, invite link to join back

#

nothing happens to them

short snow
#

ok

subtle kraken
#

we already evaluated who deserves what

#

after 3 days with no roles you are unverified
after 30d you are kicked if you still only have that role

#

you can read about it more in detail in the amazing PR @tough imp did

patent pivot
#

@sullen phoenix eeeh the issue said 10k lol
@green oriole lol

short snow
#

damn!

sullen phoenix
obsidian patio
#

Quite insane to watch

#

I actually checked it out a few minutes back. Is there a bump in members that are joining, considering the invite DM?

green oriole
#

and are they verifying this time?

sullen phoenix
#

there was like a 1:1 ratio of people joining and verifying

#

amazing

#

last 7 hours

obsidian patio
#

Do we have a graph of the amount of members that are joining? Not amount of members

sullen phoenix
obsidian patio
#

That's quite a bit higher than usual, right?

sullen phoenix
#

yes it is

obsidian patio
#

Those red blocks look like some admin had gone rouge hahaha

green oriole
#

I like how that's not consistent

sullen phoenix
obsidian patio
#

That's awesome. Seems like people join when being kicked and DMed

sullen phoenix
#

yeah, the DM on join has helped a lot as well i suspect

obsidian patio
#

Aah, yeah

wintry trellis
#

Hi. How do I make a contribution to python discord projects? I had gone through this site https://pythondiscord.com/pages/contributing/bot/. But if anyone can explain me from 'docker' section, it would be really helpful. Thanks!

green mesa
#

@wintry trellis in guide everything is mention just follow it

#

U don't need to worry about docker

sullen phoenix
#

well, you kind of do have to worry about docker

green mesa
#

Can we contribute to @viscid badge ?

sullen phoenix
green mesa
#

Ty 🔥

sullen phoenix
#

there's not much that needs doing

green mesa
#

Oooo

sullen phoenix
#

the things we need to do with it are admin tasks

#

which joe just does himself

green mesa
#

Yes joe told me it gives metrics

#

I thought we can also contribute 😅

#

But np I will contribute to seasonal and py bot🙂

sullen phoenix
#

@wintry trellis what were you confused about in the "docker" section?

patent pivot
#

people can contribute to metricity for refactors/optimisation, but there isn't much that needs adding as of now. long term goal is to replace some more graphite stuff

fierce willow
#

For the Python Bot, could you make it detect when anyone asks for instance "what is a while loop" or other newbie questions and have the bot post a link to something that tells the user all about it?

patent pivot
#

it's been considered, but so far no proposed implementation has covered everything

green mesa
#

We have tags command which covers some stuff

#

Which need explanation

fierce willow
#

Couldn't you make the bot search for the answer at a certain website like Python.org? Then tag the user to the link? Or is that hard to do

green mesa
#

We are adding more to it

patent pivot
#

It's simpler said than done 😄

fierce willow
#

Ok lol

patent pivot
#

How do you determine what a user is talking about? We'd need to invest in advanced natural language stuff or list off a bunch of buzzwords, some which may not be accurate

#

butu yeah, as Anubhav mentions we do have tags

#

for example

#

!f-strings

stable mountainBOT
#

In Python, there are several ways to do string interpolation, including using %s's and by using the + operator to concatenate strings together. However, because some of these methods offer poor readability and require typecasting to prevent errors, you should for the most part be using a feature called format strings.

In Python 3.6 or later, we can use f-strings like this:

snake = "Pythons"
print(f"{snake} are some of the largest snakes in the world")

In earlier versions of Python or in projects where backwards compatibility is very important, use str.format() like this:

snake = "Pythons"

# With str.format() you can either use indexes
print("{0} are some of the largest snakes in the world".format(snake))

# Or keyword arguments
print("{family} are some of the largest snakes in the world".format(family=snake))
fierce willow
#

Like if statements. But I know that would be a lot of ifs. Could you have the bot get most of the simple functions (like certain links) from the website then have ifs that get what the user said and if they said something that relates most to a certain link? Would that be easier?

short snow
patent pivot
#

Like if statements. But I know that would be a lot of ifs. Could you have the bot get most of the simple functions (like certain links) from the website then have ifs that get what the user said and if they said something that relates most to a certain link? Would that be easier?
not really, we just can't programatically determine what a user is talking about, the tags system is closest to what you mean I think

green mesa
#

@fierce willow py official documentation is not beginner friendly

sullen phoenix
fierce willow
#

@patent pivot ok. Thanks! Sorry for using up your time

patent pivot
#

no need to apologise!

short snow
fierce willow
#

@green mesa I was using it as an example. I couldn't think of another off the top of my head

green mesa
#

Ok

sullen phoenix
#

@sullen phoenix in a forked repo?
@short snow you should have the repository locally

fierce willow
#

one more thing. I downloaded the python bot script. Is it supposed to look like this on some of the py files?

sullen phoenix
#

looks fine to me, you just have some odd syntax highlighting

obsidian patio
#

If you’re referring to the large blocks of text, they’re simply docstrings. They’re worth learning about, if you don’t know about them already

fierce willow
#

Ok. Thanks. But where do I put a bot token for testing?

sullen phoenix
#

the code is very well documented

fierce willow
#

Ok. Thanks again!

tawdry vapor
#

The project directory means the root of the repository you cloned

#

The bot is a subdirectory of the project, so no, it would not be in the bot folder.

fierce willow
#

Ok. Thank you Mark!

short snow
#

what all environment variables should be there in the .env file @green mesa ?

green mesa
#

@f1re Sorry to keep coming back but I'm a little confused about this step at the bottom > <https://pythondiscord.com/pages/contributing/installing-project-dependencies/#installing-dependencies-with-the-command-line >
@fierce willow in your editor terminal run both command

short snow
#

ah thanks, how do we get a github token?

green mesa
#

Go to your profile then setting then developer option

short snow
#

for the .env

green mesa
#

Not sure buddy😅

short snow
#

np @sullen phoenix sorry for the ping, do i need to install those plugins?

#

they were auto detected in the .env file

sullen phoenix
#

if you want to, sure

#

i have no idea what IDE you're using lol

#

it looks very unfamiliar to me

short snow
#

pycharm

#

just white theme

#

they are 3rd party that's why i asked

sullen phoenix
#

ahh okay

#

sure, they’re just helpful

short snow
#

what should be put in CYCLE_FREQUENCY=? in the .env

#
    cycle_frequency = int(environ.get("CYCLE_FREQUENCY", 3))  # 0: never, 1: every day, 2: every other day, ...
ValueError: invalid literal for int() with base 10: ''
``` since when doing `pipenv run start` i got this error in the `/PycharmProjects/seasonalbot/bot/constants.py` file
#

happened becuase here was nothing with the cycle freq

#

so what shold i put

#

(in the .env)

sullen phoenix
#

don't put anything

#

you can just use the default

#

remove CYCLE_FREQUENCY= from the .env file

short snow
#

ok

#

what does it mean btw

#

thanks, it is working now, i will start contributing the issues and a few games

#

discord.errors.HTTPException: 400 Bad Request (error code: 10014): Unknown Emoji to overcome thi error what all emojis are needed?

green mesa
#

@short snow yes last is dustbin emoji

#

In your server it's not there

#

So it's raising that error

short snow
#

what else?

green mesa
#

U don't need to worry

#

About this

#

U can start working on the issue if u wish

short snow
#

ok

#

i added the transh emoji still not working

#

nvm i will start working on an issue or making a small game like hangman or mastermind

green mesa
#

Ok

clever wraith
#

!reddit command doesn’t filter nsfw subreddits.

#

idk if i should submit this issue by making a format or something.

#

#bot-commands message

patent pivot
#

so

#

I thought we did have checks in place for this

#

I did think it was there already, but evidently not

#

peculiar

clever wraith
#

yeah i opened an issue.

green oriole
#

I remember checking it out, and it did checked if the first post has the over_18 attribute set to true

#

Wait a minute

#

We have a reddit command on @dusky shore and @stable mountain

clever wraith
#

i used @stable mountain. !reddit

green oriole
#

Yeah

#

But it is really weird, why do we have two commands doing basically the same thing?

patent pivot
#

hmmmm

#

lol

#

.reddit

green oriole
#

Seems like we can get rid of the @dusky shore one, since @stable mountain needs it for #reddit

patent pivot
#

!reddit

stable mountainBOT
#
Command Help

!reddit
View the top posts from various subreddits.

Subcommands:

!reddit daily [subreddit=r/Python]
Send the top posts of today from a given subreddit.
!reddit subreddits
Send a paginated embed of all the subreddits we're relaying.
!reddit top [subreddit=r/Python]
Send the top posts of all time from a given subreddit.
!reddit weekly [subreddit=r/Python]
Send the top posts of this week from a given subreddit.

patent pivot
#

huh

green oriole
rapid swallow
#

I remember checking it out, and it did checked if the first post has the over_18 attribute set to true
@green oriole perhaps the first post was not marked NSFW?

#

still does not make sense

green oriole
#

Well, we have a reddit command on both bot, but one of them doesn't check if it is tagged as NSFW @rapid swallow

rapid swallow
#

yeah I got that

#

it did checked if the first post has the over_18 attribute set to true
so the first post might no have had nsfw tag for some reason

clever wraith
#

I can use the trash can emote and close joe’s output of the reddit command using @dusky shore

#

i believe this shows the first post. I might be wrong.

my bot :3

short snow
#

where is the statuscat code of the seasonalbot, i think i have a better which has result for all errors, just will replace and test on my testserver

green oriole
short snow
#

okie tthanks

green mesa
#

this much is enough for zodiac command

#

or i need to add more

short snow
green oriole
#

From the issue

Only alpha numerical or ASCII would be allowed as a start.

#

Using ascii would be the best IMO

green mesa
#

this would be fun

obsidian patio
#

It will depend on the font. If it’s a small font, alpha numerical + a few extra characters would be in place, while we could support much more of the ASCII range with a larger font

short snow
#

i will try it, just can you give me abcd..xyz in written emoji, so it will help me

obsidian patio
#

I’ve already kind of made it

#

Didn’t expect me to do it, but it just seemed too fun not to

green oriole
#

Good luck for the m

obsidian patio
#

Hahahah that one wasn’t too hard. The characters can very in width, as long as the height is the same

green oriole
#

That's easier then haha

obsidian patio
#

I have one font that’s 3 characters heigh so far. I’ll link you a demo

green oriole
#

That's pretty cool haha

obsidian patio
#

Hahah yeah. It’s super neat. Coming to seasonalbot, so that should be nice

wintry trellis
#

@wintry trellis what were you confused about in the "docker" section?
@sullen phoenix I had gone through the guide again and figured it out, thanks :)

green oriole
#

What if you replace the top right by a lonely lemon @obsidian patio?

obsidian patio
green oriole
#

Mhh, not sure that'd even work

crude gyro
#

make the right stem one step shorter than the left stem @obsidian patio , it'll look more k-like then

obsidian patio
#

Aah, the problem is that the top is only one block heigh

crude gyro
#

right.

obsidian patio
#

And we don't have "half-blocks"

crude gyro
#

haha.

obsidian patio
short snow
#

or make square with a break in middle K, like a groove in the square

obsidian patio
clever wraith
#

or do you mind if i take a crack at it

tough imp
#

hi

#

go for it

#

just so you know, we're planning some significant changes to the wiki pages, I'm not sure how much that would affect this issue

#

@crude gyro likely knows more about that

clever wraith
#

changes content-wise, or something to do with the backend/api?

crude gyro
#

I think a search bar will be great - but it is definitely blocked by the dewikification project.

#

I think perhaps a search bar can be part of this project

clever wraith
#

ok, yep, i can see why this is a blocker

crude gyro
#

I'd welcome any contribution towards this project, though. But there are several tasks that will need to be completed first.

#

they are quite well specified, though, if you'd like to take a crack at any of them

clever wraith
#

sure, i'll have a look

crude gyro
green mesa
#

need suggestion what i can add more

#

or its fine

green mesa
#

added moto also

crude gyro
#

do we really have an issue for this?

#

astrology commands in a programming server feels kinda out of place to me.

brazen charm
#

I believe that's just an extension of an existing command

green mesa
#

yes

#

we have

#

.issue 414

green mesa
#

lemon whats your zodiac sign

#

same question for others too 🙂

short snow
#

aries

eternal owl
#

libra

long brook
#

gemini

green mesa
#

noice

#

mine is Pisces

green oriole
#

Gemini

#

Wonder if we can find lemon's birth date somewhere

green mesa
#

yes 👀

crude gyro
#

ugh.

#

how dare you

green oriole
#

I see everything haha

crude gyro
#

!warn @green oriole please don't dox the owners

stable mountainBOT
#

:incoming_envelope: :ok_hand: applied warning to @green oriole.

neon phoenix
#

@crude gyro wait how did you change my issue description

#

well, imo your edit is better lol

obsidian patio
#

Wait what. Was that an actual warning? Did I miss something?

patent pivot
#

wait how did you change my issue description
@neon phoenix github issues & comments can be edited by repository admins

neon phoenix
#

:o

clever wraith
#

did he just warned that guy for that? ._.

crude gyro
#

I'm allowed to warn people for doxxing me, yes, even if they are akarys.

brazen charm
#

The previous iterations are always kept so it's really not an issue

patent pivot
#

yep

clever wraith
#

doxxing should be more than a warn >:)

#

swing the banhammer

neon phoenix
#

let's ban a core contributor yeah

clever wraith
#

yeah!

neon phoenix
#

no

#

well, lemon warned him and that's all
there's nothing more what to discuss, I guess

crude gyro
#

roles do not provide infraction immunity. a warn was the correct response in this case. can we please maybe not discuss infractions in here

clever wraith
#

ight

empty sluice
#

Did Akarys leave the server?

green mesa
#

Yes

#

He left

#

I am also leaving the server ,I might return someday

mellow hare
#

Why the leave?

#

If I may ask, of course

#

Not Ak's

#

Ah no? Anubhav is still here

empty sluice
mellow hare
#

Weird

#

It still shows his roles

empty sluice
#

Why did Akarys leave though?

crude gyro
#

that's his business. let's not discuss it here.

thorny obsidian
#

Does resolve and close mean different things in github?

tawdry vapor
#

No

crude gyro
#

Does resolve and close mean different things in github?
@thorny obsidian

Sorry, I just edited that comment because I wanted the comment to actually link the ticket so it would auto-close when the PR was merged - but it turns out you can't do that.

#

So then I edited it into the actual PR description instead, and that worked

thorny obsidian
#

Aaaah, okay. That makes sense

ashen isle
#

how are API keys stored for @stable mountain ? looking to make my own

radiant fossil
#

Environment variable

ashen isle
#

oh thanks, i didn't know that

obsidian patio
#

This is an example .env file from the guide:

BOT_TOKEN=YourDiscordBotTokenHere
BOT_API_KEY=badbot13m0n8f570f942013fc818f234916ca531
REDDIT_CLIENT_ID=YourRedditClientIDHere
REDDIT_SECRET=YourRedditSecretHere
ashen isle
#

ty!

obsidian patio
#

No problem

fierce willow
#

<@&267629731250176001> The server is being buggy. Every time I click on a help channel in occupied, it disappears until someone chats something then it reappears

crude gyro
#

@fierce willow the bots are down.

#

we're working on it

#

this affects the help channels.

fierce willow
#

Oh ok

crude gyro
#

what you're saying sounds more like you've collapsed the category though

#

just click the category and it'll stop doing that

fierce willow
#

Oh yeah... i feel dumb now XD

short snow
#

is there any beginner's issue on the seasonal bot

#

cuz i am literally a "beginner"

eternal owl
obsidian patio
#

Seems like the trash emoji isn’t available when testing out SeasonalBot on my test server

#

Do you know of a good solution?

brazen charm
#

change the constant to ❌

obsidian patio
#

That would work with @stable mountain, but I’m testing @dusky shore? Is there some config file that I’ve missed?

brazen charm
obsidian patio
#

Alright. Isn’t that file tracked by git though, so I can’t really modify it?

#

Well, not without it being listed as a changed file

brazen charm
#

You can just not add/commit it, I'm not aware of an another way of changing what emoji it's using

obsidian patio
#

Hmm, ok. Seems a bit finicky

patent pivot
#

git add . is typically a bad idea anyway, you should check what you are comitting

#

well, git add . is fine, just check git status to make sure nothing untoward is there

wild ridge
#

If I know exactly what I'm committing, I tend to git commit -am ... since that'll only add tracked files

#

As opposed to adding every file regardless of whether it's tracked

obsidian patio
#

Alright. I'll just have to remember not to commit that file hahah

wild ridge
#

Ah I wasn't looking at context

#

Suppose you might be able to use something like git add . ':!seasonalbot/bot/constants.py'

#

But that's bordering on unnecessary if you're only working on a couple files

brazen charm
#

For a change I needed the docker file for the site I created a git filter to avoid accidentally commiting it and pycharm highlighting it. Probably not the best practice but it works

crude gyro
#

staging changes in a UI is wildly superior to any form of git add. staging individual chunks or lines is really painful in git cli

patent pivot
#

eh, I think git add --interactive is pretty nice

obsidian patio
#

Gotta try that out next time

patent pivot
#

run git add --interactive and you can select things like viewing the diff, tracking files, reverting stuff

#

it's a great tool

wild ridge
#

oh that is quite nice

patent pivot
wild ridge
#

I got used to using git add --patch for chunks of code that I'd forgotten to add

patent pivot
#

but yeah, you can stage individual hunks super easy with that

#

yeah, git add --patch is a shortcut for git add --interactive and then selecting patch

wild ridge
#

right

#

might start using that then

patent pivot
#

yeah, you can often get away with just --patch but having interactive means you can pull up the diff and view status quickly as well

#

which makes is comparable to a GUI git client

#

you can see all files, tracked or untracked & stage individual hunks really easily. otherwise it would be painful

slender peak
#

do i need a website to run the bot?

#

as per docs

sullen phoenix
#

you need our site to run the bot

#

docker-compose takes care of that though

short snow
#

made a statuscat random command (which i know is useless, but just something new, now i need to understand the file structure and see how it works)

slender peak
#

i have added it

sullen phoenix
#

that’s not where you add it

#

did you read the docs?

slender peak
#

then 😢

#

Create a copy of config-default.yml named config.yml in the same directory.
?

sullen phoenix
#

yeah

slender peak
sullen phoenix
#

it does not say add your bot token to config.yml

slender peak
#

then 😭 ?

sullen phoenix
#

i'm not sure why you're crying

slender peak
#

this way?

sullen phoenix
#

yes, you also need the bot api key

slender peak
#

where is that

short snow
#

in the developer portal

slender peak
#

i'm dying for that

sullen phoenix
#

it says the api key right in the docs

slender peak
#

i mean which section?

sullen phoenix
#

the section i linked

#

BOT_API_KEY

#

by default, it's badbot13m0n8f570f942013fc818f234916ca531

slender peak
#

cool

#

oh those are default values?

sullen phoenix
#

that's the default api key for site

slender peak
#

we don't need to wrap the values with "/' ri8?

sullen phoenix
#

nope

slender peak
#

okie

sullen phoenix
#

just make sure you change up the config values for your guild, and you're set

slender peak
#

can i use invite link in guild as temp invite link or should i let that be like that?

sullen phoenix
#

you can change it to whatever you'd like, or you can keep it

#

are you trying to use this bot for your own guild, or are you setting up a dev environment to contribute?

slender peak
#

like for my own guild

#

I'm not that good at py

#

i must learn it more and more

sullen phoenix
#

this bot would not be good for your own guild

slender peak
#

why?

sullen phoenix
#

it's very specialised for our guild

#

you can try, but it probably won't go so well

slender peak
#

no i just want to manage help sections like you guys do

sullen phoenix
#

it does much more than that

slender peak
#

for that i can use it ri8?

#

okie cool but i want only that for now

sullen phoenix
#

then you probably don't want to use this bot

slender peak
#

so what can i use?

sullen phoenix
#

because it does like 40 other things as well

slender peak
#

that is awesome

sullen phoenix
#

i'd recommend extracting the help channel system code

#

and putting it into your own bot

slender peak
#

okie lemme check

#

where is the help section?

#

i couldn't find it

sullen phoenix
slender peak
#

so lemme know what file i can delete

sullen phoenix
#

you're giving yourself a lot more work by going the opposite route

#

adding this cog to a new bot would be much easier than deleting 60 other files

slender peak
#

so only cog file i can work with

#

so i will take that folder to another workspace and delete the remaining project is that okie?

short snow
#

you would need to understand this bot structure before copy-pasting it

#

when i went to contribute towards some issues i really couldn't get the file structure so for now i just am seeing each feature and see what i can add to them to make it better. Then once i am done with the file structure i start working on the issues

#

@slender peak this

slender peak
#

👍

#

😬 actually i don't have issue

short snow
#

yes i know that, i just tld about me, since you can't just cut paste, you would need to understand the code and file strucutre

slender peak
#

but i just wanna know cog is the only file handling the help channels and can i only get the cog folder and remove all other folders?

#

oh yeah okie

#

thank you so much everyone to helping me out

#

😉

#

shit did you guys say that i must work on cog folder?

#

i dleted all the other files and tried

#

XD

green mesa
#

Can I get my pr reviewed 😅 only 1 check is left

#

1 check is passed

short snow
#

you are back? even akarys is back , good to see you

green mesa
#

Yeh we are bck 😁 now time to contribute

#

Ty for welcoming

long brook
#

I typed 1914

#

is this a bug in the bot?

hardy gorge
#

weird

#

Is there an accidental space in the answer or something? I'm not sure

long brook
#

I dont think so

#

maybe

#

no there's no space

hardy gorge
#

Not sure what's going on then

long brook
#

ok

hardy gorge
#

Feel free to make an issue about it, maybe someone wants to have a bit of fun tracking it down and writing a patch for it

#

Are you familiar with making issues on GitHub?

long brook
#

yea I'll do that when I have time

#

probably by the end of the day

#

I'm in the middle of something currently

hollow orbit
#

How can i contribute Python

#

I will give one month 9 hours everyday for contribution

obsidian patio
#

@hollow orbit Do you mean the bot, this server or the language itself?

eternal owl
#

do we have a test server for @stable mountain so we can invite our own bots for development? (just to skip the whole config thing)

tawdry vapor
#

There isn't a public one

eternal owl
#

okay

short snow
#

ah same problem i had

#

can we have a server template?

#

for testing

eternal owl
#

i think it is in the pins

#

you still have to modify the config.yml file

short snow
#

yes i know

obsidian patio
#

Oh wow @crude gyro. That’s the most hilarious and great song I’ve heard. Made me start the day in a good mood hahahh

crude gyro
#

haha, thanks

green oriole
#

This is the greatest song ever related to open source development for sure haha

obsidian patio
#

For sure. It should get an award

short snow
#

this lemon_hearteyes for sure it should

green oriole
#

I'm surprised that this wasn't in the MTVs awards tbh

eternal owl
obsidian patio
#

Hey @subtle moss! This channel is intended for discussing the development of PyDis’ projects — like @stable mountain, @dusky shore and the site. Feel free to ask for a general code review in a help channel. Check out #❓|how-to-get-help is you need help claiming one

subtle moss
#

sorry about that.

obsidian patio
#

No worries :)

green oriole
#

Did crispy bulma got abandoned?

obsidian patio
#

crispy bulma? I only knew about simple bulma

green oriole
#

It is for bulma crispy forms

#

But it is now archived

obsidian patio
#

Never heard about it. Fun name though

long brook
#

yea why did it get archived

subtle kraken
#

It has been archived as it has been no longer maintained

long brook
#

ok

green oriole
#

That's a shame

green mesa
#

any changes do i need to do in embed or i can make pr for it 🙂

#

both partnerzodiac date and partnerzodiac zodiac is done

#

suggestions will be helpful 🙂

eternal owl
#

is there a chance the embed description will cross the limit?

green mesa
#

nope

eternal owl
#

okay

green mesa
#

i am fetching data from json and its hardcoded

eternal owl
#

but still, you could add a check, then paginate the embed?

#

ohk

#

btw, an embed title would be handy

green mesa
#

ok ty

#

i will add this

eternal owl
#

yep

green mesa
#

any more thing i need to add?

eternal owl
#

in the last 2 sections, the sentences are missing a full stop .

#

maybe thats from the json

green mesa
#

ok

#

yes bot is fetching data from json

crude gyro
#

@green oriole nobody was interested in working on it.

brazen charm
green oriole
#

Oh okay

eternal owl
green mesa
#

i wnt to make different pull request for different command but when i pushed the changes it pushed my changes to same pr why?

#

now wiki is having partnerzodiac subcommand also 😅

green oriole
#

You should push to two different branches, and open one PR from each branch

green mesa
#

ooo but by mistake i pushed to same branch

#

what should i do?

green oriole
#

You can reset the history and force push

#

First, you should create a new branch

green mesa
#

is there and docs how to do

green oriole
#

Then, switch back to the old branch, and use git log to see the last commit you want to keep

#

I can walk you through, that’ll be faster 🙂

green mesa
#

yes sure ty

#

this got messed up

#

😅

green oriole
#

Okay, have you made a new branch and switched back to the old one?

green mesa
#

ok let me do

#

done

#

new branch made and switched back to master

green oriole
#

Now you can run git reset 48a19c1 --hard

#

Although, that’s now too late, but you shoukd try to avoid working on master 🙂

green mesa
#

ok

#

48a19c1 this is my last commit hash?

green oriole
#

It is the last wiki related commit, yeah

green mesa
#

actually i am trying to remove partnerzodiac commit

#

its at last

#

😅

green oriole
#

Yeah, that should do it

green mesa
#

when i created pull request it has my old commit of wiki command also

#

how can i get rid of those

green oriole
#

Oooh, right

#

Hmm

#

You’d probably have to cherrypick or rebase, but I never did one of those operations, so I can’t really help you sorry

green mesa
#

np i will wait for someone else or ask in pygenral

obtuse arrow
#

Hi, what seems to be the problem?

#

You have a PR with an unwanted commit on it?

green oriole
#

Yes, there were two different feature on the same branch, we separated the first ones using a git reset, now we need to separate the last ones

obtuse arrow
#

I mean, if it's a single commit that you just want to undo, the easiest way is probably to create a revert commit on top of the branch.

green oriole
#

22 commits lemon_grimace

obtuse arrow
#

Ok, so one feature consists of 22 commits? And how many commits does the other feature consist of?

green oriole
#

I’ll let @green mesa answer, I don’t see that on gh

green mesa
#

i just closed on pr

#

u can check that

obtuse arrow
#

There are 26 commits in total

green mesa
#

last two commits are new one

#

rest are old

obtuse arrow
#

So, which ones do you want to keep and which ones do you want to undo?

#

@green oriole Are you allowed to force push to feature branches?

green oriole
#

I should be, but I’m not on my computer

obtuse arrow
#

I mean, are people in general allowed to force push to their own feature branches?

tough imp
#

Yes it's fine but it looks like those commits are on the fork's master, it'd probably be best to first move them over to a separate branch, edit it locally, then push it and open a new PR

obtuse arrow
#

Oh, right

#

Yeah, indeed

green oriole
#

Oh yeah, they are on their own fork

green mesa
#

@tough imp sorry for the ping but i have made new branch parterzodiac which has these commits

#

but still it shows all commit on either branch

tough imp
#

so what are you trying to do? drop commits in the history of your branch?

green oriole
#

I’m assuming you have pushed those branches already?

tough imp
#

I can show you how to drop the commit using interactive rebase if you want to do that, but I don't know if that's what you want

green mesa
#

yes i wnt to drop commits in new pr

#

i wnt to push only new commits

tough imp
#

are you using pycharm by any chance?

#

its gui makes it quite easy

green mesa
#

vsc

tough imp
#

ok, so when you do git log you can see the commit you want to drop?

#

and it's only in your branch, right?

green mesa
#

i have commits in 2 branch master and partnerzodiac

tough imp
#

so you want to throw away the other 21 commits in the branch?

#

probably better to just cherry pick these two

#

I think I know what you want to do but I don't want to mess up your history in case I'm wrong

green mesa
#

😅

tough imp
#

from your master branch

#

so we cannot reset your master to the upstream one just yet

#

what you need to do is create a new branch that will be a copy of the upstream master, and then cherry pick those two commits into it, I think

green mesa
#

that pr is waiting for review for more than 14 days

green mesa
subtle kraken
#

I don't think we ever had any issues with duplicate commits yet

green mesa
#

senjan i did this mistake ahh

#

my new pr has all commit of my old pr

subtle kraken
#

I don't think single mistake of single person warrants adding dependency for all our contribs

green mesa
#

well its a possible mistake which can happen

#

with new git users like me

subtle kraken
#

I'd list same reason as why we don't use auto formatters

#

you should be taking care of such things, it is a good way to learn how to use git in that case

green mesa
#

ok

subtle kraken
#

if you let some arbitrary program do it for you, you won't learn anything

green mesa
#

make sense

#

rebase can help with my issue?

green mesa
#

i have seen one thing whenever i do changes in ext folder of seasonal bot it adds all old commit

#

i created new branch at github and as soon as i copied my code and tried to add which are in ext folder github added my old commits

exotic ember
#

@neon garnet o/ it's alright if you don't have the time for it now but just letting you know I can merge the casear cipher PR once you addressed the nested function thingy

neon garnet
#

Sure, I should be able to work on it once I'm free with irl stuff

exotic ember
#

righto, you can just ping me once it's ready whenever

#

@green mesa Hello, re: your PRs for the wiki and partnerzodiac commands, they seem to have overlapping commits, as in wiki has partnerzodiac stuff, and partnerzodiac has wiki stuff. Can you please address that?

green mesa
#

@exotic ember yes I seen that thats y I commented on pr also

#

I tried to fix but it's detecting the commits again and again

#

When I try to create pull req at my own fork it shows 2 commit

#

But when I try at seasonal bot it shows old+new

#

Both are done same way but don't know y it's happening

exotic ember
#

hm, you have the wiki changes in your partnerzodiac branch, so when you try to PR from your partnerzodiac branch to our master branch, it takes the wiki additions too

green mesa
#

But when I created pr at my own fork

#

It didn't detected the old commits

#

Only new one's it detected

#

But at seasonal bot it detected both

exotic ember
#

that's because at the point of your new commit pr to your own master branch, the wikipedia commits have already been merged into your master branch

green mesa
#

So what should I do to fix this issue 😅

#

I am completely noob to git

#

Just know how to commit and push

#

I tried cherry pick method also of git that also didn't solved the problem😔

#

that's because at the point of your new commit pr to your own master branch, the wikipedia commits have > > already been merged into your master branch
I tried before merging the branch at my own fork also but that's also giving same result because of that I closed the previous pr

#

@exotic ember

exotic ember
#

hmm, one sec

green mesa
#

@exotic ember but my master branch have wiki commits

#

Will it allow me to push?

#

New changes

exotic ember
#

your partnerzodiac PR is from your partnerzodiac branch right? it should not concern master at all

green mesa
#

Ok

neon garnet
#

@exotic ember Do I need to resolve the merge conflicts if any of those arise?

green oriole
#

Make sure your branch is up-to-date, and then yeah, you’ll have to resolve the conflicts (if they are still there)

exotic ember
#

@neon garnet yes, that would be ideal

green oriole
#

We should reopen this issue I think

#

.issue 355 site

dusky shoreBOT
brazen charm
obsidian patio
#

Right now, the output looks like this:

obsidian patio
#

@sullen phoenix What do you think about renaming the emojis on the server as well?

sullen phoenix
#

what do you mean?

#

they've already been renamed

obsidian patio
#

Really?

#

They still look like this for me

sullen phoenix
#

yeah, they used to be terning

#

i don't think i understand what you mean

obsidian patio
#

Yess, I was referring to you requesting that we change it to die in the code ahah. We need it to be consistent

sullen phoenix
#

no, i only meant the variable name

#

since it's terning right now

obsidian patio
#

Oh, I thought it was a matter of singular/plural

#

I'll fix the variable name now. To dice, to keep it consistent

sullen phoenix
#

awesome, sounds good

obsidian patio
#

There we go. Should be ready to be merged once the checks are complete

#

@sullen phoenix It's ready to be merged now

sullen phoenix
#

needs 2 approvals to be merged, but you got my approval 😄

obsidian patio
#

Alright, thanks :)

tough imp
#

as a headsup @obsidian patio, it's generally better to do your work on a feature branch even if its on a fork, rather than opening a PR straight from master

#

there was another user here earlier today who ran into some git issues due to working on their fork's master

obsidian patio
#

Aah, alright. I thought this was tiny enough on the fork that it didn't matter. Will make sure to do that in the future 😄

tough imp
#

yeah it's fine either way

#

thanks for the fix

#

bot redeployed now

#

.roll 5

dusky shoreBOT
tough imp
#

good job rainbowcat

obsidian patio
#

Aah, lovely. Thanks 😄

#

Maybe we should add some space between multiple ones lemon_thinking Maybe later

green mesa
#

@sullen phoenix sorry for ping can u check wiki pr I need 1 more review so my check can pass actually I am requesting this cuz mistakly I did work on master branch of my fork and this wiki pr is not yet merged so whenever i try to pr with different branch it fetches old+new commits so if u have a look on my pr and if it's good then can u merge actually bcz of this dumb mistake my all pr is getting affected

#

This pr is on master now it's getting commits from every new pr of mine😔

sullen phoenix
#

sorry, i'm quite busy. i'll look over it when i can

green mesa
#

Ok

green mesa
#

now wiki command dont have any partnerzodiac commit if anyone wnts to check out ? any feedback will be helpful

short snow
#

is there any beginner's issue on the @dusky shore bot repo?

green oriole
#

Yes!

short snow
#

all are already in progress

green oriole
#

Welp

hardy gorge
#

Is there anything you'd like to see and work on?

#

Preferably not a magnus opus feature

short snow
#

not currently, but i just started d.py, and am making a small bot , so something for my lvl

#

not too above me

hardy gorge
#

You see, Seasonal Bot is for having a bit of fun with coding and for having a bit of fun playing with the feature you made

#

It doesn't have to be difficult, although it is important to note that other things become important when you're joining a collaborative project

#

It's not just about "making it work", but also about "making it readable" and "making it easy to maintain (including for others)"

short snow
#

👍 i saw the contributing guide, using the PEP8 and etc.

hardy gorge
#

Those are interesting things to get into and experiment with, which is exactly what we used Seasonal Bot for in the past: An introduction to collaborative, open-source projects.

green mesa
#

@nocturne hare sorry for ping, can u check my wiki pr only 1 check is pending 😁

#

My other pr is getting affected because of this pr becuase this pr is on my fork master

#

So if it get merged then my other pr will not fetch old+ new commit

#

.pr 430

dusky shoreBOT
short snow
green oriole
#

Lol what

green mesa
#

Weired

green oriole
#

It looks like it... isn't resizing?

short snow
#

well i dunno

#

no errors in terminal

green mesa
#

One thing happened with me ,I don't know y my bot don't know any games command even though branch master is upto date

#

When I use help game it gives None in log

#

I don't know y it happens¯_(ツ)_/¯

short snow
#

so i copied the command into another bot and it happened like this

#

i didn;t wanna mess the seasonal bot file up

patent pivot
#

lol what

#

.8bitify

dusky shoreBOT
#
Your 8-bit avatar

Here is your avatar. I think it looks all cool and "retro"

patent pivot
#

.help 8bitify

dusky shoreBOT
#
Command Help

.8bitify
Pixelates your avatar and changes the palette to an 8bit one.

short snow
#

why does mine happen like that?

patent pivot
#

no idea

short snow
#
from io import BytesIO

import discord
from PIL import Image
from discord.ext import commands


class EightBitify(commands.Cog):
    """Make your avatar 8bit!"""

    def __init__(self, bot: commands.Bot) -> None:
        self.bot = bot

    @staticmethod
    def pixelate(image: Image) -> Image:
        """Takes an image and pixelates it."""
        return image.resize((32, 32)).resize((1024, 1024))

    @staticmethod
    def quantize(image: Image) -> Image:
        """Reduces colour palette to 256 colours."""
        return image.quantize(colors=32)

    @commands.command(name="8bitify")
    async def eightbit_command(self, ctx: commands.Context) -> None:
        """Pixelates your avatar and changes the palette to an 8bit one."""
        async with ctx.typing():
            image_bytes = await ctx.author.avatar_url.read()
            avatar = Image.open(BytesIO(image_bytes))
            avatar = avatar.convert("RGBA").resize((1024, 1024))

            eightbit = self.pixelate(avatar)
            eightbit = self.quantize(eightbit)

            bufferedio = BytesIO()
            eightbit.save(bufferedio, format="PNG")
            bufferedio.seek(0)

            file = discord.File(bufferedio, filename="8bitavatar.png")

            embed = discord.Embed(
                title="Your 8-bit avatar",
                description='Here is your avatar. I think it looks all cool and "retro"'
            )

            embed.set_image(url="attachment://8bitavatar.png")
            embed.set_footer(text=f"Made by {ctx.author.display_name}", icon_url=ctx.author.avatar_url)

        await ctx.send(file=file, embed=embed)


def setup(bot: commands.Bot) -> None:
    """Cog load."""
    bot.add_cog(EightBitify(bot))
``` this code is correct right?
obsidian patio
#

@short snow after you clone your fork, do the development on a feature branch. There’s no need to copy the command files over to another bot, if that’s what you did

nocturne hare
#

No idea how strict the reviews are on seasonalbot, but you got mine @green mesa

green mesa
#

Ty for your review I will make changes according to what u said 🙂 ty very much

obsidian patio
#

Don't we have any command for reloading cogs in @dusky shore?

neon garnet
#

I don't believe we do

#

That'd be rather neat for dev purposes

obsidian patio
#

Yeah, for sure

#

We could probably borrow most of the logic from @stable mountain

#

I'll hopefully open an issue about it in a few hours, unless anyone is against the idea

green mesa
#

i have opened one issue for source command also

obsidian patio
#

Yep; that would be neat as well

green mesa
#

i am happy i finally got review on my wiki issue hope soon it will get merged

#

partnerzodiac sub commands also ready in advance UwU

green oriole
#

I'm curious about why we aren't using watchdog to auto reload with development bots

#

That would be so smooth

green mesa
#

watchdog?

green oriole
#

*watchman ooops

green mesa
#

ooo

green mesa
#

i dont know y but i am not able to request for review by chibli at github😅

#

is that normal?

green oriole
#

Which PR?

green mesa
#

same wiki pr

#

.pr 430

dusky shoreBOT
green oriole
#

The wiki command?

green mesa
#

yeh

tough imp
#

I think it's because Chibli reviewed using an accout that isn't in the org

green mesa
#

o

tough imp
#

he'll see your answers when he has time

#

or you can ping him here if you want to chat

green mesa
#

ok i am excited because this is my first PR 😅 for seasonal bot

#

@nocturne hare sorry for ping ,i pinged u here cuz i am not able to request for review at github,I have made all changes according to what i am told. when u have time please have a look 😄

green mesa
#

.pre-commit file only works in pipenv?

#

i wnt to use it without using pipenv,i tried at different repo and it not getting triggered at commit

green oriole
#

Well, it is used by the precommit package

#

If you look at the Pipfile, in the scripts category you can see what this command is actually doing

green mesa
#

i did that

#

but it didnt help its not getting triggered at commit

green oriole
#

Hmm

green mesa
#
  python: python3.7
fail_fast: true
repos:
-   repo: https://github.com/ambv/black
    rev: stable
    hooks:
    -    id: black
-   repo: https://github.com/PyCQA/flake8
    rev: master
    hooks:
    -    id: flake8
-   repo: https://github.com/pre-commit/mirrors-pylint
    rev: master
    hooks:
    -   id: pylint
        files: coc```
#

i have this, i am on rewrite branch of this repo

green oriole
#

What am I looking at? lemon_thinking

green mesa
#

.pre-commit-config.yml file

#

i did pre-commit install already and installed hooks

green oriole
#

If you updated the hooks, maybe you have to run it again?

green mesa
#

its an exisiting hook

#

but its not getting triggered

green oriole
green mesa
#

i already asked there

#

waiting for reply 😅

#

its triggering now

eternal owl
#

btw, its https://github.com/psf/black instead of https://github.com/ambv/black @green mesa

green mesa
#

ok

#

what does rev do?

eternal owl
#

not gonna make a whole lot diff tho but just keepin it updated xD

#

the version, basically

green mesa
#

ok

green mesa
#

Can we have one list of cursed word which we wnt to filter?

#

So it can be used in wiki command,reddit command and it can be used in upcoming pr🙂

#

If it's required

eternal owl
#

maybe we can have a list of words stored in a file/permanent storage, so that we can add more words as we go along with a command?

#

i guess we prolly won't be able to find out all the cursed words also

green mesa
#

But we can keep adding to list as we get to know

eternal owl
#

yep, thats what i said

#

Comment on the wiki issue/PR about this

green mesa
#

On my own pr or should I open one issue for it😅

eternal owl
#

the current one

#

as this is kinda related and can be implemented under the same topic

subtle kraken
#

list of words hasn't changed much

#

current one is all dynamic but you can find old one in one of commits on bot repo

green mesa
#

Under which pr?

subtle kraken
nocturne hare
#

@green mesa Ping me tomorrow and ill see if i can find time to go over it again

sullen phoenix
#

!remind 188735535169273858 20h do the thing

stable mountainBOT
#
I'll allow it.

Your reminder will arrive in 20 hours and will mention 1 other(s)!

obsidian patio
#

"Do the thing" hahah

green mesa
#

Ty senjan and ok chibli🙂

green mesa
#

And ty f1re

green mesa
#

Why we have name mentioned

#

And when someone join how it's time will None so it will fail or I am misunderstanding 😅

#

I am new to unit test

exotic ember
#

that's just an example to show how 100% test coverage doesn't mean everything

#

in that example, although every line is tested, not every case is tested

green mesa
#

Ok I will try to write one😅

green mesa
#

Y we use salt+azure together 😅

neon garnet
#

@exotic ember I've addressed the needed changes for the caesar cipher command, feel free to take a look if you have some time

exotic ember
#

coolio

obsidian patio
eternal owl
#

i actully setup in a different way which i find easier

#

that is to use the terminal

#

pipenv sync --dev

glass pecan
#

i also just sync --dev before opening the project and then pycharm autodetects the env

eternal owl
#

and then add a new project interpretor as pipenv, and it will detect it

glass pecan
#

is it just me, or is that attempting to run the .lock file as a py file though wat

#

wait no, it's thinking the .lock is the python executable

obsidian patio
#

Do you know why pipenv sync --dev would be needed?

glass pecan
#

it's not, there's a way to get it working without it, but i'm thinking you're in the wrong section to do it

#

but it's easier

green oriole
#

wait no, it's thinking the .lock is the python executable
Is your IDE drunk?

eternal owl
#

Do you know why pipenv sync --dev would be needed?
@obsidian patioits an alternative method, it will create the virtual env for u using the pipfile and the dev dependencies mentioned

glass pecan
#

oh you mean the dev flag?

#

yeah it's required

green oriole
#

You need it to fetch development tools

glass pecan
#

but you can do that in pycharm too pretty sure

green oriole
#

Pycharm uses it by default

obsidian patio
#

I mean why the Use pipenv interpreter won't set it up?

glass pecan
#

how about this @obsidian patio i'll setup my seasonalbot env at the same time as you and we can go through it step by step

#

since i need to do it anyway

green oriole
#

Because well, if you open it in PC, you are developing it

#

I mean why the Use pipenv interpreter won't set it up?
@obsidian patio PC creates a new pipenv interpreter everytime, even with this option afaik

obsidian patio
#

I've already set it up using VSC, but it's just the PyCharm part that's now working quite yet

glass pecan
#

oh so you already have a pipenv env?

#

then use the already created venv

obsidian patio
#

I believe so

eternal owl
#

what do you get when u do pipenv shell

glass pecan
#

pipenv --venv

#

will show location

#

then point pycharm to the python bin in it by going through venv interpreter setting

green oriole
#

But that won’t activate the PC pipenv integration

obsidian patio
#

Aah, alright

glass pecan
#

honestly, if you're needing multiple editors ak, it's better to not have pycharm take over control of pipenv

#

it'll still be capable of running pipenv commands in app/run configs though

green oriole
#

Well, pycharm doesn’t do anything by itself, it will always ask you like to relock or sync

#

But yeah

obsidian patio
#

This is my first time with PyCharm. What would you recommend that I do this time?

green oriole
#

Do what scrag said, it will probably be easier 🙂

glass pecan
#

im doing mine too atm and gonna see if pycharm picks up the venv still like it used to

obsidian patio
#

Not entirely sure how you mean

hardy gorge
#

I'm always running pipenv manually and just selecting the interpreter in PyCharm (adding the interpreter/selecting it)

glass pecan
#

i'll explain after my check

#

okay, so this is what i'm doing.
clone git repo to local
pipenv sync --dev
open directory in pycharm as project
wait for pycharm to index project
pycharm now has pipenv interpreter after a minute loaded

#

and i'm going to remove that autodetected one and go through the steps of me adding it back in manually

obsidian patio
#

That sounds really convenient. Sure didn't do that for me

glass pecan
#

Pick Venv, existing environment
pipenv --venv to get directory

hardy gorge
#

What I do is git clone, pipenv sync --dev from the command line. Open the folder as a project in PyCharm, when it asks for the interpreter, select existing interpreter and add the venv interpreter pipenv created

glass pecan
#

that's venv setup

obsidian patio
glass pecan
#

use bin/python