GitHub Actions run 3844008154 failed.
#dev-log
1 messages ยท Page 7 of 1
- Raised by @3ddP here: #the-pydis-mass-layoffs-lounge message (Helpers+ required)
We would like to add a nedbat tag that embeds Ned Batchelder's "Facts and Myths about Python names and values" PyCon 2015 talk.
Video is here https://www.youtube.com/watch?v=_AEJHKGk9ns
We already have a tag that links to that talk:
https://github.com/python-discord/bot/blob/main/bot/resources/tags/names.md
can it be added as an alias? ;(
It doesn't quite seem appropriate to have a command with Ned's name that links to a tag semi-related to a talk he gave 8 years ago.
I'd expect a nedbat command describe who Ned is, and mayve link to some of the content they have produced, which doesn't quite seem appropriate to do for one person in isolation.
The names tag is quite long and honestly includes a bit too much information to be useful. I think a !names-and-values tag that has a 2 sentence summary and a non-suppressed link to Ned's video would be much better. Alternatively, we could rework the names tag a bit to be more usable and approachable.
When creating a Post in help and attaching a file that you dont allow, PLEASE not delete the post after one sends it.
Typing out good questions AGAIN cause you deleted it because of smth I couldnt know is not cool
Discord msg about that:

This isn't possible as Discord only allows users to edit their own messages, not the messages of others.
We are working on a new filters system, which will include the ability to DM users with the content of the message that was deleted.
So that actual problem of not being able to access the message you pasted will be solved by https://github.com/python-discord/bot/issues/1530
Connected!
GitHub Actions run 3864920267 succeeded.
Hi, this seems like an issue with Docker on your system that is not specific to snekbox. Thus, this is not an appropriate place to get support for that issue. You're welcome to join Python Discord and get support there, or another community like the Docker forums or StackOverflow.
Looks good, just 2 small things to consider
[python-discord/bot] New review comment on pull request #2269: Allow evaling code from context menus
Don't we keep track of the python version that we usually use for evaluation ?
Having this here can lead to us forgetting about it once we update the py version
[python-discord/bot] New review comment on pull request #2269: Allow evaling code from context menus
I think that this code should go away once this PR is completed
GitHub Actions run 3866878420 succeeded.
You're right.
This is fixed in babf90649df6acb5e8ebda28e15b4e4b1ee9e2d2 by relying on the thread_id to turn into into a valid datetime object & use it for comparison
GitHub Actions run 3866926966 succeeded.
LGTM, tested & my commands sync as usual.
I'm just curious here, are you making this possibly optional for people who wouldn't want to sync their commands ?
If yes, is there particular reason behind this ? (Perf, etc.)
Yea, we might not always want slash commands loaded in dev if we end up with a lot of them, as there is a rate limit on how many new commands can be registered each day
Still don't really have PC access so I'm not able to review as thoroughly as I'd like, but it doesn't seem like there's anything that could go really wrong at this point with these changes.
GitHub Actions run 3870902734 succeeded.
Update the ANSI guide and sync with the original gist with better pictures, formatting & titles.
So, it seems there was a spelling mistake. So went ahead and fixed it!
GitHub Actions run 3889828041 succeeded.
Connected!
[python-discord/bot-core] branch deleted: sync\-tree\-on\-startup
[python-discord/bot-core] New tag created: v9\.4\.0
1b44290 Fix typo in AoC unlink command - shenanigansd
8268167 Merge pull request #73 from python-discord/shen... - D0rs4n
6736eb1 Start Completionist Role Assignment - janine9vn
7b808bb Merge pull request #74 from python-discord/jani... - janine9vn
64eeba9 Setup kubectl in deploy workflow - ChrisLovering
[python-discord/sir-robin] branch deleted: aoc\-join\-slash
Connected!
GitHub Actions run 3899312438 succeeded.
Connected!
Because I find myself pointing people to Asking Good Questions regularly.
Just my opinion, but this seems a little bare. Perhaps some other information could be useful
What do you think it needs? The page it links to has comprehensive detail and I didn't want to just repeat any of that.
Connected!
GitHub Actions run 3905552252 succeeded.
GitHub Actions run 3910271267 succeeded.
Connected!
GitHub Actions run 3913949451 succeeded.
The names tag is quite long and honestly includes a bit too much information to be useful. I think a
!names-and-valuestag that has a 2 sentence summary and a non-suppressed link to Ned's video would be much better. Alternatively, we could rework the names tag a bit to be more usable and approachable.
Either/both of those sound good to me.
GitHub Actions run 3914093298 succeeded.
GitHub Actions run 3914125010 succeeded.
Connected!
GitHub Actions run 3914169607 succeeded.
LGTM!
I didn't test this, but it looks pretty straightforward
[python-discord/bot] branch deleted: allow\-passing\-channel\-objets\-when\-checking\-ignore
GitHub Actions run 3914258418 succeeded.
Connected!
GitHub Actions run 3914292398 succeeded.
seems that worked, nice (tested with normal channel threads + forum channel posts)

GitHub Actions run 3914907646 succeeded.
Connected!
GitHub Actions run 3914952874 succeeded.
Nothing major. Thanks for the PR! I haven't tested it either, so I'm hoping we didn't letting anything slip by haha ๐
I'm not sure what this sentence is supposed to mean entirely. Perhaps it could be reworded to be concise?
Is there a reason we are returning None for the discord.ui.View? Couldn't we just use discord.ui.View.from_message?
Since this message is ephemeral, I don't think there's a need for the button that says "Click this to get your very own bookmark" or whatever it was. The whole point of the button was that multiple people wouldn't be running the .bookmark command in succession to the same message and flooding the channel - but with ephemeral messages, that's not an issue. So I think this button would just lead to more confusion and is unnecessary.
See my comment above. I don't think this description is necessary either. We just need something like "Bookmark sent! Check your DMs" or something similar
I believe PEP likes it when the first line of a docstring is on the same line as the initial triple quote
"""The form where a user can fill in a custom title for their bookmark & submit it.
GitHub Actions run 3917928659 succeeded.
Alright, comments have been addressed and it's ready for review again
I reverted it in e763175be4f93363ce3fc7060c7b19bfa4c7fe52, since we also return directly from build_error_embed
Good point, done in 61345d0abec9d5fb0f32f275ebab5adac12f9201
That's a lack of focus mistake, good catch.
Fixed here ecb41eb75cb0d6cd5b4cdc0111aebb29badaa1d8
Makes sense, done in cf4903561a222de9b7413c406b5d6e186f977332
GitHub Actions run 3917939857 succeeded.
That's supposed to be an "and", it's a typo.
Does it still appear unclear to you ?
This proposes to add to our deployment venv.
I think it would be quite useful in discussions in #internals-and-peps as well as showing implementation details of python object structure to beginners who are curious.
from einspect import view
a = [*range(18)]
print(view(a).allocated) # 18
b = list(i for i in range(18))
print(view(b).allocated) # 24
For example many have asked how python ints are actually implemented with base-30 arrays:
from einspect ...
We can't really recreate the view using from_message, as this persistent view needs to be instantiated with the self assignable roles, and from_message cannot pick that up.
GitHub Actions run 3917977095 succeeded.
GitHub Actions run 3919350404 succeeded.
GitHub Actions run 3919350630 succeeded.
GitHub Actions run 3919351557 succeeded.
GitHub Actions run 3919351532 failed.
GitHub Actions run 3919351556 succeeded.
GitHub Actions run 3919355281 succeeded.
Does this only apply to mobile users? If so, should it be more specific e.g. "Note that this feature doesn't work on mobile yet"
GitHub Actions run 3919359796 succeeded.
GitHub Actions run 3919359715 failed.
GitHub Actions run 3919361428 succeeded.
GitHub Actions run 3919361645 succeeded.
GitHub Actions run 3919366455 succeeded.
I heard from some users in the past few weeks they don't see the ANSI colors on their Linux/Windows & Stable client - will confirm that next week and update accordingly
GitHub Actions run 3919371786 failed.
GitHub Actions run 3919372830 failed.
GitHub Actions run 3919372421 succeeded.
GitHub Actions run 3919378149 succeeded.
wanted to rename the file for consistency, will need someone elses approval
I don't mind either way, thoughts @swfarnsworth?
GitHub Actions run 3919371786 succeeded.
GitHub Actions run 3919630904 succeeded.
GitHub Actions run 3919630457 succeeded.
GitHub Actions run 3919634935 succeeded.
Add description of what a decorator is actually syntactic sugar for.
GitHub Actions run 3919762963 succeeded.
I'll finish this off. I don't have commit permissions to this branch so i'll open a new PR.
7500eee Trim query in command not found error to avoid ... - wookie184
[python-discord/bot] New branch created: trim\-help\-command\-error\-title
Successor to #2286
Closes #2257
I ended up moving the trimming logic so only the query is trimmed, so you get something like Query "long..." not found rather than Query "long....
I also didn't use textwrap.shorten as that only breaks on words, so if the input had no spaces you would just get something like Query "..." not found.
GitHub Actions run 3919906853 succeeded.
minor suggestion, looks good otherwise
query = textwrap.shorten(query, 100, placeholder="โฆ")
Obviously requires importing textwrap too
See PR description :P
I also didn't use textwrap.shorten as that only breaks on words, so if the input had no spaces you would just get something like Query "..." not found.
Basically textwrap always ended up just deleting the input, since there aren't any spaces because the command only shows the first invalid bit (e.g. !abcd hwdhe just shows query "abcd" not found
>>> textwrap.shorten("helloworld", width=5)
'[...]'
a35b37b Make dependabot check for github actions updates - wookie184
[python-discord/site] New branch created: enable\-dependabot\-github\-actions\-updates
Noticed that there were some warnings to update in some workflow runs, so seems worth automating.
GitHub Actions run 3919979662 failed.
[site] Branch enable\-dependabot\-github\-actions\-updates was force-pushed to `fd786c6`
[python-discord/site] Checks Successful on PR: #837 Make dependabot check for github actions updates
GitHub Actions run 3920019526 succeeded.
GitHub Actions run 3920269316 succeeded.
Looking at the build preview it seems it gets a bit squished up on mobile

Other than that this seems great though. Any thoughts on how the behaviour on mobile could be improved?
@doublevcodes Are you still interested in working on this?
I think it's better to stick with one representation of these reactions.
For example, we can check if emoji == 'โ
' or 'โ'
Looks good overall, just some final touches.
adding whatever functionality you wish. Usually this is done by adding custom methods or properties, so that you don't need to
Maybe refer to the intents tag here for more information/details ?
0dc0a79 Bump pymdown-extensions from 9.9 to 9.9.1 - dependabot[bot]
f641ed3 Merge pull request #835 from python-discord/dep... - Xithrius
348f130 Bump gitpython from 3.1.29 to 3.1.30 - dependabot[bot]
555817d Merge pull request #836 from python-discord/dep... - Xithrius
f4ceb72 Merge branch 'main' into enable-dependabot-gith... - Xithrius
[python-discord/site] Checks Successful on PR: #837 Make dependabot check for github actions updates
GitHub Actions run 3921115958 succeeded.
GitHub Actions run 3921125188 succeeded.
It was discussed in [#community-meta](#community-meta message) if the !eval command should reply to the original command instead of sending a separate message with a ping. Thoughts on this? I'd be willing to PR if accepted.
I think we still need to decide if this is something we want, and how it should work, so i'll mark this as stalled until there's a consensus on the issue, or on the server/in a staff meeting.
Sounds good to me, I don't see why not.
This seems good to me, the only thing I would suggest is limiting the number of prompts to 3, as 5 seems like quite a lot (i'd still leave the detection for when it goes over the title limit, just in case there are 3 really long topics).
Let's see what others think first though, as I don't mind too much and some people may prefer more.
Played around with the command and noticed a few things:
- The command didn't seem to sync automatically, I had to run
.int eval `await bot.tree.sync()`for it to appear. - The first time I use the command, this seems to stay there forever:

For bookmarks run after that it disappears normally, although after refreshing the client the same thing happens again. - Running th...
f"A bookmark for [this message]({target_message.jump_url}) "
f"has been successfully sent to your DMs.\n"
"retrieve" makes it sound like there's more action required from the user. In reality, bookmarks are for storing things for later so there's probably no point in them checking now.
Our style guide specifies to put it on a new line https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/style-guide/#line-structure
I like it more that way too.
Good point, I've removed the call to action for the DMS
What would happen when the issue invokes the command in a non-whitelisted channel? I suppose it'd have to still send a new message that pings in bot-commands?
Also would the reply be ephemeral? We presumably don't want that, since a lot of the time the eval command is used to showcase something, meaning other people need to see the outcome.
And I believe @bj0key, the user who raised this, said they'd like to implement this fwiw @Robin5605/
GitHub Actions run 3923108346 succeeded.
- The command didn't seem to sync automatically, I had to run
.int eval `await bot.tree.sync()`for it to appear.
I think you might have missed the description, this relies on the newest release of pydis_core, where the tree is synced upon loading extensions. So that might be the reason.
- The first time I use the command, this seems to stay there forever:
...
I think you might have missed the description, this relies on the newest release of pydis_core, where the tree is synced upon loading extensions. So that might be the reason.
I noticed that but since the PR was merged I assumed poetry install would do fine. Is it just that the version used by this project needs to be bumped now?
Yeah this seems good. Sometimes the eval takes a while and returns a few messages after the original !eval, so it's unclear which eval resulted in the output, the reply would provide a clickable link.
This sounds like a nice idea, some thoughts:
Some events (e.g. Hanukkah) happen at different dates on different years. Is the idea that we pick a rough range that is fixed for these e.g. the whole of december? (I think that would probably be the best option)
flake8 is excluded in the snowflake trigger regex (but this can be changed)
I'm sure this is what you meant anyway, but to confirm, we should match words exactly using \b word boundary markers in the regex (how it already wor...
Slightly unsure, as we only allow english on the server, and this feature requires using another language, and I'm not sure if having a conversation through lancebot commands is ideal even if we want that.
Yeah, I wouldn't mind implementing this. I think it'd also be worth having !timeit also reply on response, for the sake of parity between the two evaluation-based commands.
GitHub Actions run 3927340468 succeeded.
GitHub Actions run 3927344852 failed.
GitHub Actions run 3927349911 succeeded.
[python-discord/sir-lancebot] Checks Failed on PR: #1186 Bump flake8-bugbear from 22.12.6 to 23.1.14
GitHub Actions run 3929227248 failed.
It seems the new B028 rule may be moved to become an off-by-default one (https://github.com/PyCQA/flake8-bugbear/pull/333). I think the current and suggested way are equally fine so i'd be happy to just wait to see if that happens.
Did just notice that we're using ignore instead of extend-ignore in our .flake8 anyway though, will open a PR to change that.
3962c73 Use extend-ignore instead of ignore in flak... - wookie184
[python-discord/site] New branch created: ignore\-to\-extend\-ignore
This prevents off-by-default rules from being overridden and turned on
GitHub Actions run 3933406965 succeeded.
[python-discord/site] Checks Successful on PR: #837 Make dependabot check for github actions updates
GitHub Actions run 3933449955 succeeded.
GitHub Actions run 3933465926 succeeded.
GitHub Actions run 3933465335 succeeded.
GitHub Actions run 3933466129 succeeded.
GitHub Actions run 3933465063 succeeded.
GitHub Actions run 3933470833 succeeded.
GitHub Actions run 3933465594 succeeded.
GitHub Actions run 3933471868 succeeded.
GitHub Actions run 3933482915 succeeded.
5f57e51 Ensure dev reqs are installed in CI - ChrisLovering
[site] Branch dependabot/github\_actions/actions/checkout\-3 was force-pushed to `dd4c14f`
GitHub Actions run 3933515149 succeeded.
GitHub Actions run 3933517477 failed.
GitHub Actions run 3933517589 succeeded.
GitHub Actions run 3933516522 succeeded.
GitHub Actions run 3933528838 succeeded.
GitHub Actions run 3933531457 succeeded.
GitHub Actions run 3933531496 failed.
GitHub Actions run 3933526992 succeeded.
GitHub Actions run 3933536762 succeeded.
GitHub Actions run 3933524467 succeeded.
GitHub Actions run 3933543321 succeeded.
GitHub Actions run 3933543481 failed.
GitHub Actions run 3933546020 succeeded.
101c7b4 Bump pyfakefs from 5.0.0 to 5.1.0 - dependabot[bot]
5c8676c Merge pull request #840 from python-discord/dep... - ChrisLovering
dd4c14f Bump actions/checkout from 2 to 3 - dependabot[bot]
7eed875 Merge pull request #842 from python-discord/dep... - ChrisLovering
929e8e7 Bump docker/setup-buildx-action from 1 to 2 - dependabot[bot]
ah, will leave this one for now then
GitHub Actions run 3933560243 failed.
GitHub Actions run 3933566515 succeeded.
GitHub Actions run 3933570997 succeeded.
[site] Branch dependabot/github\_actions/HassanAbouelela/actions\-setup\-python\_v1\.4\.0 was force-pushed to `3fb0d96`
GitHub Actions run 3933747280 failed.
GitHub Actions run 3933754895 failed.
GitHub Actions run 3933757970 succeeded.
GitHub Actions run 3933747280 succeeded.
[python-discord/site] New branch created: poetry\-groups
This updates the project to make use of poetry groups.
No dependencies, nor their versions, were changed in the pyproject.toml file, they were just moved around.
GitHub Actions run 3933850442 succeeded.
fd786c6 Make dependabot check for github actions updates - wookie184
f4ceb72 Merge branch 'main' into enable-dependabot-gith... - Xithrius
294d0ba Merge pull request #837 from python-discord/ena... - ChrisLovering
49b2e05 Bump Azure/k8s-deploy from 1 to 4 - dependabot[bot]
101c7b4 Bump pyfakefs from 5.0.0 to 5.1.0 - dependabot[bot]
[python-discord/site] branch deleted: ignore\-to\-extend\-ignore
Relevant Issues
Closes #1165
Description
This change adds the ability for Sir Lancebot to react to regex-based seasonal comment triggers outside of Halloween. Total reaction triggers increase from 8 to 19, with 13 of these in two months: Halloween (where the original 8 are kept) and Christmas.
A graphical list of reaction emojis, along with a demo screenshot during Christmas, is available in the linked issue. (Although the emoji rendering in Github differs from Discord)
An e...
GitHub Actions run 3934822550 succeeded.
GitHub Actions run 3936268906 failed.
GitHub Actions run 3936268569 succeeded.
GitHub Actions run 3936269258 succeeded.
GitHub Actions run 3936268209 succeeded.
Seems like they were using an outdated client version - I've tested around and seen the change now in stable everywhere (besides mobile).
Updated the disclaimer accordingly as well
GitHub Actions run 3940894911 failed.
GitHub Actions run 3942243667 succeeded.
GitHub Actions run 3942243262 succeeded.
GitHub Actions run 3942248462 failed.
GitHub Actions run 3942251406 succeeded.
GitHub Actions run 3942364431 succeeded.
GitHub Actions run 3942371369 succeeded.
GitHub Actions run 3942369825 failed.
"menorah": Trigger(r"\bc?hanukkah|menorah\b", ["\U0001F54E"]),
This is already run in on_message so shouldn't be necessary here
The resources page does not include the tutorial linked on the python.org site here
I think we should include it as a free tutorial source.
GitHub Actions run 3942651807 succeeded.
See wookie's comment. Once the new version is released, dependabot will open a new PR.
Nice, thanks for looking into that.
Could you just add a newline to the file to appease the linter?
Looks like there are some commits from other PRs that shouldn't be here
Thanks for pointing that out :) - has been done
GitHub Actions run 3942824569 succeeded.
GitHub Actions run 3942858487 succeeded.
GitHub Actions run 3942889675 succeeded.
Looks like there are some commits from other PRs that shouldn't be here
I think those are the results of the rebase ?
GitHub Actions run 3943280959 succeeded.
GitHub Actions run 3945860915 succeeded.
GitHub Actions run 3947575517 succeeded.
GitHub Actions run 3947729165 succeeded.
Connected!
GitHub Actions run 3948237271 succeeded.
I don't think this will work quite as intended with the word boundary, as e.g. the \b at the start is only part of reindeer, so while this prevents areindeer from matching, acaribou etc. would still match.
A fix for this would be to write \b(reindeer|caribou|buck|stag)\b, this also applies to a couple of the other regexes.
[python-discord/site] New branch created: improve\-actions\-workflows
[python-discord/site] Pull request opened: #854 Stop using deprecated set\-output command in actions
GitHub has deprecated the old format, so I've updated to use the new syntax:
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
GitHub Actions run 3952100317 succeeded.
Looks good. We should migrate to reusable flows soon though, see https://github.com/python-discord/black-knight/tree/main/.github/workflows for reference
GitHub Actions run 3954211564 succeeded.
I don't really mind either way too, for the record.
GitHub Actions run 3975902005 succeeded.
Looks good. We should migrate to reusable flows soon though, see https://github.com/python-discord/black-knight/tree/main/.github/workflows for reference
Ah, yeah, that does seems nice.
Also, I wonder if it would be worth having a repo of workflows we can reuse across the org. The build/deploy/status_embed workflows seem very similar wherever they're used. Not sure if the differences are enough that in practice it would be difficult though.
Great job, thanks! I have no more suggestions ๐
GitHub Actions run 3981436336 succeeded.
3686f60 add the roles channel to the config - shtlrs
5795140 add the AllSelfAssignableRolesView and its corr... - shtlrs
6ab9f9c add the logic for attaching the persistent view - shtlrs
b25ad80 add assignable_roles as a property to the Claim... - shtlrs
1d60403 add implementation of the button's callback tha... - shtlrs
Connected!
GitHub Actions run 3981468935 succeeded.
963d8f8 Make sure interaction_owner is an instance attr... - mbaruh
[python-discord/bot] New branch created: mbaruh\-patch\-1
If it's a class attribute then the interaction owner of all current views will be the last person to view the buttons.
Should the interaction_owner docstring still be kept around? Theoretically speaking one could still access it via role_button_view_instance.interaction_owner given that role_button_view_instance is an instance of RoleButtonView
GitHub Actions run 3981525333 succeeded.
[python-discord/bot] branch deleted: mbaruh\-patch\-1
The block_duplicate_invocations decorator prevents commands from running in a single channel if they were used previously recently with the same arguments. This can be applied to commands like resources or docs that are often used with responses to prevent being unnecessarily invoked multiple times.
It should also be able to replace the similar functionalit...
GitHub Actions run 3983490329 succeeded.
GitHub Actions run 3983495670 failed.
GitHub Actions run 3983503870 succeeded.
GitHub Actions run 3983513804 succeeded.
GitHub Actions run 3985259312 succeeded.
GitHub Actions run 3985267634 succeeded.
GitHub Actions run 3985279470 succeeded.
This is something we've discussed with the new help channel style and haven't come to a good solution for.
If discord removes interaction limitations with locked threads, it could be something we look into in future.
Closing this as we have not seen an occurrence of this with the new system.
Closing this as we have not seen an occurrence of this with the new system.
[python-discord/branding] New branch created: zzz\-icon
[python-discord/branding] branch deleted: lemoji/holding\-back\-tears
GitHub Actions run 3986001290 succeeded.
This icon will be used as part of a new dormant help channel embed (python-discord/bot#1295)

GitHub Actions run 3986009577 succeeded.
GitHub Actions run 3986016231 succeeded.
[python-discord/bot] New branch created: Update\-help\-post\-messages
Closes #1295
Preview:

GitHub Actions run 3986068310 succeeded.
[bot] Branch Update\-help\-post\-messages was force-pushed to `e852f8a`
GitHub Actions run 3986384566 succeeded.
[bot] Branch Update\-help\-post\-messages was force-pushed to `ebed332`
GitHub Actions run 3986413391 succeeded.
GitHub Actions run 3993171330 succeeded.
[python-discord/site] New branch created: official\-tutorial\-resource
7a2a9c3 yaml for python.org tutorial - Assile
Possible fix for #852
GitHub Actions run 3997384707 succeeded.
Some questions I have about my own code for this are:
- Is it OK to use an image directly from their website as the logo?
- Is it OK to use a trademarked logo for this?
GitHub Actions run 3998087724 succeeded.
Looks good to me and addresses all of the information that should be present.
Currently, Sir Lancebot cannot bookmark messages that are part of closed posts and according to @HassanAbouelela, this is due to the nature of discord.py ([discussion](#dev-contrib message))
Chris suggested to use a PartialMessage, and if the content is not available, we can just use its jump_url
7a2a9c3 yaml for python.org tutorial - Assile
6f449f6 changed over to a clearer logo - Assile
29cde7a Merge pull request #860 from python-discord/off... - wookie184
[python-discord/site] branch deleted: official\-tutorial\-resource
GitHub Actions run 4000000437 succeeded.
Is it OK to use a trademarked logo for this?
I think there are certain rules about how you can modify the logo, but since we're not doing that there should be no issue.
Im interested in implementing this
Im interested in implementing this
๐ I've assigned you
[python-discord/snekbox] New branch created: dep\-version\-update
7d0a7eb Update einspect version to 0.5 - ionite34
Bumps version of einspect from 0.4 to 0.5 for numerous bug fixes and features
Closes #2360
Would love to hear feedback on if the tag explanation is good or not as it is my first time contributing.
GitHub Actions run 4010678467 failed.
a8cb1ea fix typo in help channel opening embed - shenanigansd
[python-discord/bot] New branch created: shenanigansd\-patch\-1
GitHub Actions run 4011563029 succeeded.
[python-discord/snekbox] branch deleted: dep\-version\-update
This has caught me off before. However, there is a grammatically correct interpretation as-is, though it isn't the first one I come to naturally. It can be read as "If you got one" -> "if you got an error message" -> "if you got an error message when you ran the code"
Ah well we have #2386 so there probably is no point in merging this. It'll just create conflicts and the impact of the issue this solves is minimal.
It'd be more appropriate to call it a "post" than a "channel", but we can revisit that in the future if we want. I don't think it's that significant either way.
[python-discord/bot] branch deleted: Update\-help\-post\-messages
2794059 Add icons to help channel bot embeds - ChrisLovering
eb45d03 Add action text to help channel embeds - ChrisLovering
ebed332 Update closed help post embed text - ChrisLovering
6e8f127 Merge #2386 - simplify closed help post message - MarkKoz
2a3fa81 Merge branch 'main' into shenanigansd-patch-1 - mbaruh
Connected!
GitHub Actions run 4011682854 succeeded.
GitHub Actions run 4011681611 succeeded.
Connected!
GitHub Actions run 4011708324 succeeded.
GitHub Actions run 4016658514 failed.
print(a_new_list[1]) # This will error because it is empty
Is this part necessary?
To avoid these errors and unexpected results, it is required to assign the result of `sorted(...)` to a new variable and use `list.sort()` method in the original list. This way, the original list will be sorted and the new list will be created with the sorted elements.
On the other hand, using the function `sorted(...)` will return a new sorted list, leaving the original list unchanged. This means that if you expect the original list to be sorted, you will be disappointed with the result. For example, the following code will print 3 instead of 1:
GitHub Actions run 4023421427 succeeded.
GitHub Actions run 4028119479 failed.
Looks good to me, but I'm sure others with have things to say
GitHub Actions run 4029256445 failed.
cf8ce93 add token safety pin - Shom770
9ba4891 fixing image not appearing - Shom770
306a906 Update pydis_site/apps/content/resources/guides... - Shom770
eb86541 Migrate Setting Different Statuses to Set Your ... - iamericfletcher
63c67a1 Adding extra space at end of file in response t... - iamericfletcher
Thanks! There are a couple wording changes that I think would be beneficial for clarity here, but this is overall pretty good.
In programming, there are two types of operations: "out of place" operations create a new object, leaving the original object unchanged. "in place" operations modify the original object, without creating a new one. These operations return None explicitly.
This empty line can be removed, Discord's rendering auto-inserts one after code blocks.
I'm not sure how I like "it is required" here, as well as the focus on lists. My proposal here would something along the lines of:
To avoid these errors and unexpected results, you should either use an in-place operation (
sorted(...)) and assign it to a variable or use an out-of-place operation (list.sort()) without assignment.
The wording can obviously vary, but I'd prefer to see something general like that.
It may be a bit clearer to use a new name here.
For both or what exact list name
GitHub Actions run 4029368969 failed.
Seems good to me, I chose required because I couldn't really think of anything else in the moment.
GitHub Actions run 4029373109 failed.
GitHub Actions run 4029384596 failed.
I think variable names here should be more in-line with what the person was trying to do, so I would call them unsorted_list and sorted_list for the first example and list_to_sort in the second one.
I also think the way you show the unexpected behavior (with indexing) is overcomplicated, something like
print(a_new_list) # Outputs None. Where did the list go?
and
print(outofplace_list) # The list still isn't sorted. Why?
would make it easier to understand...
I have noticed that sir lancebot only allows us to add bookmarks, but not deleting them.
I figured that this could be useful for whatever reasons people would have w be that the DM channel is too cluttered, or they have bookmarked some message explaining Middlewares for example but then have read a better one which they'd like to replace with the old one.
This should be a pretty easy change considering we'd use add another button to the view we already send in the DMs.
Would you like...
This can already be done using the .bm delete command in DMs, although I wouldn't be against adding a button (possibly replacing the command).
This can already be done using the
.bm deletecommand in DMs, although I wouldn't be against adding a button (possibly replacing the command).
I guess it'll be more intuitive through a button, no need to search or even know about the command (Just like I didn't) because it'll be available '' out of box'' now.
And, if for some reason, the interaction fails, then the command will serve as a fallback.
This pull request implements the new filtering system as described in #1530.
For more information about the database and API changes made to support this, check out the site PR.
Warning This PR must be merged at the same time as the site PR.
closes #1530.
Overview
The main idea is that there is a list of filters each deciding whether they apply to the given content.
For example, there can be a filter that decides it will trigger when the content contains the string "lem...
This pull request applies the necessary changes to the database and API to support the new filtering system on the bot.
For more information on how the information in these tables will be used, check out the bot PR.
Warning This PR must be merged at the same time as the bot PR.
closes #479.
Scheme

- There is a unique constraint on combinations of name, list_...
GitHub Actions run 4032961064 succeeded.
@n0Oo0Oo0b Seems like a good idea, updated.
GitHub Actions run 4033099922 failed.
Connected!
GitHub Actions run 4033258388 succeeded.
Connected!
GitHub Actions run 4033265679 succeeded.
Connected!
GitHub Actions run 4033273848 succeeded.
Connected!
GitHub Actions run 4033281392 succeeded.
Connected!
GitHub Actions run 4033292413 succeeded.
GitHub Actions run 4033732218 succeeded.
GitHub Actions run 4034134520 succeeded.
Good idea, however, I am not sure the best way to do this considering it is not on our site.
GitHub Actions run 4034148168 failed.
I still think the list variable names could still be improved (not necessarily to what I suggested before) because having inplace/outofplace in the names suggest that it's a property of the lists, not the functions that you run.
I still think the list variable names could still be improved (not necessarily to what I suggested before) because having inplace/outofplace in the names suggest that it's a property of the lists, not the functions that you run.
What would be some better names, I mean they could be generically named a_list, and another_list.
I think unsorted_list/sorted_list for the first example and list_to_sort for the second will be fine, probably with a # Wrong: line at the top as well to show that it isn't the correct way to use the functions
I think
unsorted_list/sorted_listfor the first example andlist_to_sortfor the second will be fine, probably with a# Wrong:line at the top as well to show that it isn't the correct way to use the functions
๐, updated.
GitHub Actions run 4034520358 failed.
Looks good, approved.
Closes #1196
The change is pretty simple, and makes it more intuitive to delete a book mark start from the DMs.
I'll keep this in draft for now since the original issue hasn't been approved, but I thought i'd do it still since it's only 4 lines of code
GitHub Actions run 4036377703 succeeded.
I don't like using a button here. It limits the messages that can be deleted to only new bookmarks, that have the button view in them.
I think this would work better as an app command that has the same behaviour as bm delete (can only delete sir lances messages in your own DMs.)
Yeah a context command is better, but that's 2 more clicks for this to be achievable.
I think they can both co-live together, as a button jumps straight at you when you see it and I personally think it's a better option for the new bookmarks.
And for the old ones, we can still use either the text command or the context one, WDYT ?
Pretty much everything here seems to be completed now. We don't describe how to set snekbox up on host, but that's by design since docker is easier.
Tests are mentioned here: https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#running-tests, we don't have a general guide on site, although there's an issue for documenting testing on site here https://github.com/python-discord/site/issues/678
I agree with comments that this doesn't seem massively useful, so i'll close this.
Sentry Issue: BOT-3CR
HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In content: Must be 2000 or fewer in length.
File "discord/ext/tasks/__init__.py", line 239, in _loop
await self.coro(*args, **kwargs)
File "bot/exts/filters/filter_lists.py", line 308, in weekly_autoban_report_task
await self.send_weekly_autoban_report()
File "bot/exts/filters/filter_lis...
If there are too many filters to fit in a message we should probably split into multiple messages
Sentry Issue: SIR-LANCEBOT-91
AttributeError: __enter__
File "discord/ext/commands/core.py", line 229, in wrapped
ret = await coro(*args, **kwargs)
File "bot/exts/fun/snakes/_snakes_cog.py", line 601, in draw_command
with ctx.typing():
Unhandled command error: __enter__
There are a couple of occurences of with ctx.typing(): across the bot. These should all be changed to async with ctx.typing():
Changed in version 2.0: This no longer works with the with syntax, async with must be used instead.
I agree that this isn't worth doing
I don't think this is worth the added complexity. When going into a new post it seems you start scrolled at the top, so it's easier to see the context of the post even without a pinned message.
I agree that this feature seems unnecessary.
bd4adec De-Halloween-ify reaction Cog: move file, renam... - thurisatic
8e87506 Add non-Christmas holiday reaction capabilities - thurisatic
b1470e9 Perform small fixes, incl requested: sorting, t... - thurisatic
28a8c67 Isolate regex options to capture groups - thurisatic
057c60b Merge branch 'main' into holiday-react - wookie184
GitHub Actions run 4037604612 succeeded.
Connected!
GitHub Actions run 4037610033 succeeded.
The bot guide now mentions that site is required if the user isn't using docker https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/#with-more-things-running-locally
I can't identify anything else that needs to be done here so i'll close this. In general we do recommend using docker as it's much simpler though, so our documentation of how to set up bot on host isn't as good.
I don't see much of a reason for adding this at the moment, so i'll close this
7dde1fb Upload weekly autoban report to pastebin - ChrisLovering
[python-discord/bot] New branch created: use\-paste\-service\-for\-long\-autoban\-filters
This fixes #2391 and solves BOT-3CR which was caused by the weekly report being longer than 2000 characters.
Instead of posting the entire report to Discord, it is now upload to the pastebin, and uploaded as a text file to the channel, so people can use what they prefer.
7a2a9c3 yaml for python.org tutorial - Assile
6f449f6 changed over to a clearer logo - Assile
29cde7a Merge pull request #860 from python-discord/off... - wookie184
f704524 Merge branch 'main' into dependabot/pip/python-... - Xithrius
GitHub Actions run 4039541398 succeeded.
GitHub Actions run 4039567873 succeeded.
GitHub Actions run 4039744469 succeeded.
GitHub Actions run 4039744383 succeeded.
GitHub Actions run 4039763677 succeeded.
GitHub Actions run 4039763515 succeeded.
GitHub Actions run 4039886743 succeeded.
GitHub Actions run 4039888301 succeeded.
GitHub Actions run 4039914025 succeeded.
GitHub Actions run 4040317925 succeeded.
GitHub Actions run 4040326886 succeeded.
GitHub Actions run 4040343153 succeeded.
GitHub Actions run 4040743805 succeeded.
GitHub Actions run 4040750014 succeeded.
GitHub Actions run 4040759891 succeeded.
Hey, I'm closing this PR since it's had no activity and is outdated due to migrating to using a forum for the help system.
GitHub Actions run 4050971241 succeeded.
Please do not merge this as it is just an example.
GitHub Actions run 4054058887 failed.
Buttons are great for quick access, but there is the trade off with it taking up the a large amount of screen height, which is very prevalent on mobile devices.
SO while I'd agree having buttons for common actions, I just don't see deleting a bookmark to be common enough of an action to warrant a dedicated button with that trade off.
To help with discoverability, I'd also suggest ripping out the current bookmark delete command's functionality, and replacing it with a guide on how to u...
Buttons are great for quick access, but there is the trade off with it taking up the a large amount of screen height, which is very prevalent on mobile devices.
SO while I'd agree having buttons for common actions, I just don't see deleting a bookmark to be common enough of an action to warrant a dedicated button with that trade off.
Do we have statistics on the usage of this command ? To be able to determine how "common" it is ?
I understand your point regarding the previous boo...
sir-lance doesn't track command usage like Python bot does, so we can't get that info.
An educated guess is that it's very low, as it was only recently added and wasn't announced.
32fba6c Bump pre-commit from 3.0.1 to 3.0.2 (#1200) - dependabot[bot]
a471782 Bump flake8-docstrings from 1.6.0 to 1.7.0 (#1199) - dependabot[bot]
Connected!
GitHub Actions run 4056939563 succeeded.
GitHub Actions run 4056940709 succeeded.
GitHub Actions run 4056942644 succeeded.
GitHub Actions run 4056946155 succeeded.
GitHub Actions run 4056949887 succeeded.
Connected!
GitHub Actions run 4056961137 succeeded.
GitHub Actions run 4056964279 succeeded.
GitHub Actions run 4056983267 succeeded.
GitHub Actions run 4056999052 succeeded.
GitHub Actions run 4056995594 succeeded.
GitHub Actions run 4057044143 succeeded.
GitHub Actions run 4057050796 succeeded.
I've not looked at the bot side yet so this is just comments while I was understanding how things work. Looks really good though.
I'm not sure I see anything in the FilterList model that isn't here? Help text here seems fine though, fine to leave exact specification to the bot side.
Is there a reason this isn't just import pydis_site.apps.api.models since that's what's being accessed? Same for other migrations.
Or just a from import to get the names directly?
"null if permanent" is impossible given "null=False`
I see the difficulty since for FilterBase null means "not overwritten" so we can't really make use of it for other meanings. I guess you've already sorted this on the bot side so the help text just needs changing
- 404: if a Filter with the given `id` does not exist
- 404: if a FilterList with the given `id` does not exist
Caught red handed!
6b28013 Extend AoC commands to Feb - janine9vn
[python-discord/sir-robin] New branch created: janine9vn\-patch\-1
We keep Advent of Code around until February (plus a week or two into February) for the AoC Completionist role. I'd like have the core commands available during the 1-2 weeks into February we keep it around for people to fix and account linking or if they finish AoC at the very end of January.
I noticed the global leaderboard command tries to fetch the 2023 leaderboard instead of the 2022 one, but that's probably outside of the scope of this PR.
GitHub Actions run 4063125864 succeeded.
9c0346d Bump pre-commit hook versions - ChrisLovering
[python-discord/sir-robin] branch deleted: janine9vn\-patch\-1
Connected!
Alright then sir, I trust your knowledge with Sir Lance
We'll make it a context command & write a guide on how to do it.
Just to be clear, we're talking about a guide in site right ?
GitHub Actions run 4064890482 succeeded.
Migrating !tag to a slash command. Allowing for autocomplete, and a subcommand /tag help or /tags listing all tags. THat list could also be ephemeral, so the user can see all the tags. Also, if possible, make the main command /tag, and the param would be autocomplete.
GitHub Actions run 4066200543 failed.
PR Author
Workflow Run
Source Branch
shenanigansd:patch-1
GitHub Actions run 4066200543 failed.
PR Author
Workflow Run
Source Branch
shenanigansd:patch-1
GitHub Actions run 4066200543 failed.
PR Author
Workflow Run
Source Branch
shenanigansd:patch-1
GitHub Actions run 4066303966 failed.
PR Author
Workflow Run
Source Branch
shenanigansd:patch-1
GitHub Actions run 4066345868 succeeded.
GitHub Actions run 4066372552 failed.
GitHub Actions run 4066414999 failed.
GitHub Actions run 4066444371 failed.
GitHub Actions run 4066487117 failed.
GitHub Actions run 4066520822 failed.
GitHub Actions run 4066592759 succeeded.
GitHub Actions run 4066614698 succeeded.
GitHub Actions run 4066645675 succeeded.
GitHub Actions run 4066814612 succeeded.
PR Author
Workflow Run
Source Branch
shenanigansd-patch-1
Don't see any point in adding anymore. The link provides enough detail. Looks good
Relevant Issues
None. I asked in python-general whether I should make this an issue first, but godlygeek and zig suggested its size (+3 characters) makes that unnecessary.
Description
I accidentally told Sir Lancebot that Earth Day was in February, meaning that it is currently (during February) reacting on earth. This PR moves it to April, where Earth Day resides on the 22nd.
Did you:
- [x] Join the Python Discord Community?
- [x] Read all...
GitHub Actions run 4070737936 succeeded.
GitHub Actions run 4073312343 succeeded.
Connected!
GitHub Actions run 4075135718 succeeded.
@dawnofmidnight availabe to rereview?
def get_field_value(data: dict, field_name: str) -> Any:
"""Get the value directly from the key, or from the filter list if it's missing or is None."""
if data.get(field_name) is None:
return data[field_name]
return getattr(data["filter_list"], field_name)
This should be checked to be None, to avoid a value being falsy or even False. Take the example of a filter with the value enabled=False. in this case, the filter_list's enabled field...
wouldn't we want is not None here? So if it's not None return it, else return the filter list default
Maybe, lol. You are correct. I've amended the suggestion. Thankyou.
[python-discord/snekbox] New branch created: deployment\-update
e6da590 Add --upgrade flag to deployment pip install - ionite34
Since pip install doesn't actually update patch versions with ~= version specifiers, we may not be getting the newest security / patch releases for our deployment venv.
This adds the --upgrade flag
d260c0c Update .pre-commit-config.yaml - ionite34
[python-discord/snekbox] branch deleted: deployment\-update
Description
Rather than using the usernames, the command should first try nicknames, then fall back to usernames if necessary. Alternatively, according to Nicky, "Love is deep. Love is about the true you". The .love command will use the id of the user rather than the nickname.
Reasoning
This would allow users to change their nicknames in order to get a score of 100% compatibility.
Proposed Implementation
Change https://github.com/python-discord/sir-lancebot/blob/main/bot/e...
Since it's not that relevant to the actual aim of the guide I think it's fine to leave it as is personally.
Also seems like the hanging command problem isn't there for me either any more๐
I believe this is a duplicate of #2362
While I agree most of our commands could work as slash commands, I do not believe one mega-issue is the best approach for migrating.
Instead I'd suggest raiding raising an issue on a per-command basis, describing the benefit of migrating it to a slash command, and a proposed implementation. Each issue can then be separately approved by a core dev when appropriate.
I believe this is a duplicate of #2362
I suggested splitting out the issue in this way, see my recent comment in the issue for context.
and a subcommand /tag help or /tags listing all tags. That list could also be ephemeral, so the user can see all the tags.
We could make the tag name argument optional, so if not given it lists all the tags in a paginated embed, like it does currently.
Also, if possible, make the main command /tag, and the param would be autocomplete.
I a...
Currently, the only Python implementation supported is the default CPython. Additional support should be added for executing Python code in other Python implementations like PyPy, RustPython, IronPython, etc. To support this, these implementations would simply have to be installed in the NSJail environment and alternate functions would have to be provided in the NsJail class for executing with alternate implementations.
Updated original comment with the new "Tags" section which has been scoped as part of this project
Under the "Partners" role in the server roles page, there is some text about a #partners channel that does not exist anymore.

Thanks! We should also remove the comment about being part of Discord partners, since that's no longer applicable.
GitHub Actions run 4100526740 succeeded.
The channel still exists, but itโs always been private. The note below can be updated though.
GitHub Actions run 4102453020 succeeded.
Maybe you're thinking of #partner-discussion, @HassanAbouelela? #partners, which used to list all our partnered communities, does not exist anymore.
I do agree with this issue, but I'm not sure exactly what this should be changed to. We're discussing it among the @python-discord/admins, so let's hold off on this until then.
This looks mostly good to me, apart from a small fix and the proposed reโwording in my previous review.
sorted_list = unsorted_list.sort() # This will be None
Yup, you're correct, I was confused.
GitHub Actions run 4109914873 succeeded.
Code looks good to me, will test after a few hours.
There's a typo in the docstring of send_weekly_autoban_report method, btw.
If chanel is not specified, it is sent to #mod-meta.
line 316.
GitHub Actions run 4111617902 failed.
GitHub Actions run 4111676961 succeeded.
Very preliminary few notes
Why are we hashing memory identity of the instance here? Shouldn't the instance id be used instead?
Why does the type hint for other not match the guard condition for NotImplemented here?
Why does these have to be hashable in any case? I'm not sure how it makes sense for something to not be comparable (no __eq__) but to be hashable.
We should probably avoid accessing __args__ and __origin__ directly and use typing.get_args and typing.get_origin instead, since these apis seem to almost always change between versions, #type-hinting message.
Checking if get_origin(type_) in (typing.Union, types.UnionType): seems it could work here?
GitHub Actions run 4118207591 succeeded.
GitHub Actions run 4119905388 failed.
GitHub Actions run 4120804996 failed.
GitHub Actions run 4120952963 succeeded.
GitHub Actions run 4122917620 succeeded.
Description
A user asked in PyGen if it would be possible to enable the .latex command in the help threads. After discussing it with Stelercus, we were wondering if it would be possible to either enable the .latex command everywhere except PyGen, or at least in the help threads.
Reasoning
It can be helpful to nicely typeset some maths when asking and answering questions.
Proposed Implementation
The .latex command here https://github.com/python-discord/sir-lancebot/blob/m...
GitHub Actions run 4125692802 failed.
GitHub Actions run 4126560020 succeeded.
22e6d8c Add MOTD and sudo lecture roles (#76) - ChrisLovering
[python-discord/infra] branch deleted: Add\-sudo\-lecture
[python-discord/infra] branch deleted: unify\-inventory
Connected!
Connected!
GitHub Actions run 4128253235 succeeded.
Can I make a page on using cogs/extensions?
GitHub Actions run 4132120615 succeeded.
GitHub Actions run 4136642361 succeeded.
GitHub Actions run 4136653862 succeeded.
GitHub Actions run 4136666838 succeeded.
GitHub Actions run 4136940198 succeeded.
GitHub Actions run 4136939882 succeeded.
GitHub Actions run 4136941232 succeeded.
Connected!
GitHub Actions run 4136956374 succeeded.
GitHub Actions run 4136961871 succeeded.
GitHub Actions run 4136963553 succeeded.
GitHub Actions run 4136974627 succeeded.
Connected!
GitHub Actions run 4136978566 succeeded.
GitHub Actions run 4137212152 succeeded.
GitHub Actions run 4137341983 succeeded.
Connected!
GitHub Actions run 4137358423 succeeded.
GitHub Actions run 4137364340 succeeded.
GitHub Actions run 4137367424 succeeded.
GitHub Actions run 4137419887 succeeded.
Connected!
Connected!
Connected!
Connected!
Connected!
Connected!
Connected!
[python-discord/bot] branch deleted: master
GitHub Actions run 4138960315 succeeded.
Connected!
GitHub Actions run 4139025773 succeeded.
GitHub Actions run 4139109846 succeeded.
Connected!
GitHub Actions run 4139125023 succeeded.
GitHub Actions run 4139562321 failed.
GitHub Actions run 4139582926 failed.
GitHub Actions run 4139597331 succeeded.
[metricity] Branch stable\-d\.py\-and\-python\-3\.10 was force-pushed to `36038f1`
GitHub Actions run 4139625278 succeeded.
GitHub Actions run 4139633920 succeeded.
GitHub Actions run 4139633946 succeeded.
GitHub Actions run 4139633722 succeeded.
GitHub Actions run 4139633969 succeeded.
[metricity] Branch stable\-d\.py\-and\-python\-3\.10 was force-pushed to `39ece5a`
2870959 Use latest bot-core version and bump all deps - ChrisLovering
3c24f33 Bump to Python 3.10 and mark version as breaking - ChrisLovering
e3cc00e Update botcore module name throughout repo - ChrisLovering
39ece5a BotBase.load_extensions no longer needs to be r... - ChrisLovering
29104f2 Merge pull request #27 from python-discord/stab... - ChrisLovering
[python-discord/metricity] branch deleted: stable\-d\.py\-and\-python\-3\.10
[python-discord/sir-robin] branch deleted: bump\-dpy
Connected!
On this page of the website, it is showing that Code Jam 2022 is underway while its finished.

The events calendar on this page is outdated.

The modification should be to atleast update the PyWeek info and change the year to 2023. For the code jam, I think it should be changed to just text and remove the redirecting link?
4efc61e Add local_testing utilizing vagrant and virtu... - GDWR
[python-discord/infra] New branch created: GDWR/local\-testing
A very high quality PR overall!
if len(file) > char_max:
names.append(file[:char_max] + "...")
break
Considering we don't currently allow users to send the extensions in TXT_LIKE_FILES I'm not sure it should be here.
394f0b4 add snippet about world writable ansible dir on... - shenanigansd
Out of stall now; should be done sooner than later.
93efefe Remove admonition from /events/ that says CJ9 i... - ichard26
[python-discord/site] New branch created: ichard26\-patch\-1
GitHub Actions run 4149718420 succeeded.
[python-discord/site] New comment on issue #872: Events page on website showing outdated information
Good timing! I independently discovered this issue and brought it up in a staff channel a few days ago, but nothing happened. So I filed a PR to simply remove this call-out. Thanks for reporting ๐ฆ
GitHub Actions run 4149962211 succeeded.
The modification should be to atleast update the PyWeek info and change the year to 2023. For the code jam, I think it should be changed to just text and remove the redirecting link?
@ichard26 Can I make a pr according to this or it is being internally discussed?
GitHub Actions run 4151184349 failed.
This code has gone through so many iteration I have no clue why this was written. 4eecda9
Good spot, should be 0. 4eecda9
@Ibrahim2750mi I assigned it to you. Summer code jam and Pixels dates are TBD, and the rest of the links should be removed. Thanks!
This is something I wasn't very sure about. I didn't want to change the signature of the overridden method. Is changing the type hint considered common?
I tried to make this correct beyond my current usage of it. For example, if I use AtomicList objects as dict keys, I don't want to assume what else will be put in that dict.
Because I'm using the class' instances as dict keys https://github.com/python-discord/bot/blob/dc0ce05ff1406954f5006366b2899c4bde1967ce/bot/exts/filtering/filtering.py#L912
There's no logic to it being hashable but not comparable beyond not needing to compare.
I don't mind returning hash(("AtomicList", self.id)) or something of that sort, though I'm not sure how it's preferable.
- removed links for all the events
- changed the pyweek 33 and 34 to 35 and 36
- changed summer code jam and pixels month to be TBD
#873
GitHub Actions run 4151458976 succeeded.
GitHub Actions run 4151790256 succeeded.
+ Fix list item numbers in README.md
+ Add Vagrant + Virtualbox instructions for testing ansible deployments
Tested on Linux by @GDWR
Tested on Windows by @shenanigansd
Not tested on MacOS
b13025a Update bot/exts/moderation/infraction/_utils.py
- swfarnsworth
3686f60 add the roles channel to the config - shtlrs
5795140 add the AllSelfAssignableRolesView and its corr... - shtlrs
6ab9f9c add the logic for attaching the persistent view - shtlrs
b25ad80 add assignable_roles as a property to the Claim... - shtlrs
1d60403 add implementation of the button's callback tha... - shtlrs
[python-discord/bot] New comment on pull request #2374: Make the infraction message mention @ModMail
For whatever this is worth: I just tested this in my test server by warning a dummy account that hasn't viewed the main guild in a long time, and the ModMail bot was not in that account's user cache until I clicked through to the guild.
GitHub Actions run 4153733289 succeeded.
13eb7c4 Made EvalJob and FileAttachment dataclasses frozen - ionite34
51b719e Merge branch 'snekbox-files' of https://github.... - ionite34
GitHub Actions run 4155506691 was cancelled.
GitHub Actions run 4155508237 was cancelled.
Since it releases GIL I guess performance may be better? Just used since we have it. Could use re either way, no big preference.
GitHub Actions run 4155513323 succeeded.
Changed in 70d3d3c
if files_error := result.files_error_message:
msg += f"\n{files_error}"
The file error message may include the name of the files, which may trigger the filter at this point.
GitHub Actions run 4155526521 succeeded.
GitHub Actions run 4155541636 succeeded.
There's no file error message being sent here though. A message is sent for the content for of stdout triggering some filter, and then you exit.
We should leave these as all 1's. This way the markdown renderer does the hard work for us, and if we need to add a number in the middle we don't need to update all the other numbers too.
@mbaruh If you want to require the same class, use typing.Self in the parent and child classes instead of ActionPing.
Also, it's still a good idea to check for isinstance(other, Ping) because the right operand might implement __ror__...
Actually, why is this a dunder method and not just a normal method? Dunder methods have quite a lot of hariballish semantics to them (like NotImplemented).
import openai
import discord
from discord.ext import commands
Initialize OpenAI API key
openai.api_key = "YOUR_OPENAI_API_KEY"
Initialize Discord client
client = commands.Bot(command_prefix='!')
Define a command for the bot to generate an image
@client.command()
async def generate_image(ctx, *, prompt: str):
# Use OpenAI's GPT-3 API to generate an image based on the prompt
response = openai.Image.create(
prompt=prompt,
n=1,
size="1024x1024"
)
...
The config-default.yml might be confusing at first and would more than likely turn away new contributers(it almost did it for me). While the config is a essentialy part of the discord bot, I think it can be easily automated for people running it on there own servers/local dev.
I am thinking first to implement a auto-config from the server temeplate. Then Auto Creating channels in the proper categories as well as adding emotes and roles If needed.
Why are some of the generics here (like list) not parameterized? This is a chonky object, I'd at least want to know what's in it
is None -> `== Infraction.NONE
@wookie184 Hello, I can do this.
I see a lot of people join the server and go to #python-general and not #how-to-get-help and instead of just asking the question they ask people if they are familiar with xyz module or can anyone help me?
I propose we add a auto-reply to redirect the person to either ask the question directly or to read the #how-to-get-help. (ghost ping them inside #how-to-get-help) then delete there original message on general and the bot message.
[python-discord/bot] New branch created: reusable\-status\-embed\-workflow
The purpose of this PR is to make the status_embed workflow reusable.
Problem
We currently have the same status_embed workflow file scattered & duplicated across our bot, site, sir-robin, pixels, king-arthur and sir-lancebot repositories.
A solution is to make the workflow reusable, and port it over as part of the organization's workflows, from where it can be accessed & invoked by all of our repos.
Note
- This is currently just a test, and the workflow file w...
1511b08 update extension of referenced reusable workflow - shtlrs
c0d3079 remove trailing whitespace - shtlrs
0ff66e7 restore the flake8 command - shtlrs
7737e68 rename status embed to reusable_status_embed - shtlrs
3ed6c4c Revert "rename status embed to reusable_status_... - shtlrs
3b42bc3 try a different concurreny group - shtlrs
GitHub Actions run 4165297872 succeeded.
@wookie184 Hello, I can do this.
๐ I've assigned you
I don't know we should necessarily do this. A lot of our actions have a very linear flow of steps that depend on each other:
Install dependencies -> setup requirements and environment -> do something
In theory it doesn't matter if one step fails and causes the next to fail instead of be skipped, but it doesn't seem appropriate for some of the more critical actions such as deployments and uploading images.
I believe you can force a certain step to always run without forcing the entire workfl...
can I be assigned this?
The problem is that we don't know which step would fail, and since the second job needs the first one, it should always run otherwise the second might not run? (Not sure of this tbh, I'll test it)
The continue-on-error cannot be part of the job that calls the reusable workflow as well, that's why I put it there
I just tested it, the call_status_embed doesn't trigger without having continue-on-error set to true.
It also won't when i move that key to the reusable workflow's file
b5b4b87 make flake8 a multitline cmd - shtlrs
GitHub Actions run 4167880720 failed.
GitHub Actions run 4167900213 succeeded.
