#dev-log
1 messages ยท Page 77 of 1
[python-discord/site] branch deleted: jb3\-cj8\-patches
This PR adds some additional changes required before we can make the code jam 8 announcement.
Summary of changes:
- Updated Import Dates to reflect that the announcement is going out today
- Changed main NavBar to have one Events tab that should bring you to the main event page
- Updated wording about joining the Discord server
- Correctly spelled GitHub, YouTube
- Added in DigitalOcean as a Sponsor
- Clarified and updated the rules
- Ensured consistency for "Approved Frameworks"
- A...
The chosen technology/tech stack for this year is <strong>Text User Interfaces</strong> (TUIs).
Missing the link in the anchor here
83f1be2 Update pydis_site/templates/events/pages/code-j... - jb3
I thought I checked it last night, but isn't this adding a double space now?
No, this removes a double space.
<img width="133" alt="Screenshot 2021-06-15 at 16 05 34" src="https://user-images.githubusercontent.com/20439493/122077301-852d0800-cdf3-11eb-83d2-0425e139a5d4.png">
<a href="https://www.digitalocean.com/" target="_blank">
@kosayoda Sorry for the late, late response, but this PR wasn't going anywhere. I'm going to look into alternatives to matplotlib altogether, and open up new issues/PRs when I get there.
I hope that fixes #1640.
Hi, @ChrisLovering. I made a PR regarding the issue #1642
Hi :wave:
I noticed that background color appears behind the error box in the error_page.css stylesheet in the top left and right corner
Before the change:

After the change:

I hope that simple change helps :sweat_smile:
[python-discord/bot] New branch created: vcokltfre/tag/for\-else
This PR adds a tag showing how you can use for/else, as approved by kutie [here](#the-pydis-mass-layoffs-lounge message)
Here's how it looks in Discord:

Connected!
Connected!
Connected!
[python-discord/site] New branch created: cj8\-frameworks\-update
- Python Prompt Toolkit is added as an approved framework.
- Unicurses and nurses are added as additional frameworks you can use with curses, although it's a use at your own risk and no promises are made for functionality or stability.
Tested it locally, looks good to me.
[python-discord/site] branch deleted: cj8\-frameworks\-update
All looks good to me, maybe consider changing Unicurses to UniCurses to match the pypi project, but this is such a tiny detail.
[python-discord/site] New branch created: jb3/cf\-cache\-purge
I want to serve site from the edge, since the content there is primarily static. This PR ensures that cached content is cleared upon deploy so that we can benefit from high edge cache TTLs while still having new content deploy immediately.
Joe is the best programmer in the world
Is there a smarter way to clear invalidated cache items rather than purging the whole thing? Not sure if that even make sense with the way they store cache, but I don't know how they do that.
Is there a smarter way to clear invalidated cache items rather than purging the whole thing? Not sure if that even make sense with the way they store cache, but I don't know how they do that.
@MarkKoz If you pay Cloudflare, you can pass a list of wildcards to invalidate. We could pass a list of individual file URLs but it's unnecessary complexity. It's better to just have pages have a slightly higher initial load time (which they already do โ considering we currently serve django pages w...
It's better to just have pages have a slightly higher initial load time
Better in terms of what, just reduced complexity?
Anyway a max of 1.2s for a few unlucky people sounds acceptable.
This integer should be replaced with RedirectOutput.delete_delay with from bot.constants import RedirectOutput
I think this message should be simplified.
f"{message.author.mention} {constants.Emojis.cross_mark} "
"To receive updates on help channels you're active in, unblock your DMs."
To make the output a bit more visually interesting, let's add some prints for each iteration.
break
print(f"{number} is odd.")
F-strings.
print(f"Found an even number: {number}")
Let's demo the actual functionality here
numbers = [1, 3, 5, 7, 9, 11]
This PR is in high priority since this feature is getting used relatively often, as far as I can see.
@FaresAhmedb remember to lint before you push by running poetry run task lint.
Better in terms of what, just reduced complexity?
Yeah, that's all. The process for conditional invalidation would be along the lines of getting a diff of files changed, mapping that to URLs and then pushing that to Cloudflare, it's possible but is another component to maintain. We'd need to build static assets, reverse all the resource and guide URLs and so on, which is possible but I'm not sure it's worth it.
Unblock may sound like you have the Python bot blocked. "please enable your DMs" or something of the sort may be a bit clearer.
Agreed. I have updated the suggestion.
from bot.constants import Channels, RedirectOutput
When committing from a suggestion, please change the message to have a relevant description besides "Updating <file here>".
Context: b9456b4047b658b868c5ea5a965c0610ce7d9736
@Xithrius Sorry for the mistake. Can I use --amend?
In already public commits, that isn't a good idea. I'll squash merge this PR so there's only one commit in the end.
Thank you for this contribution!
Connected!
Mentioning something about hobby programmers would be nice here, as there are many of those.
Relevant Issues
Closes #1598.
Description
Adds the dunder methods tag described in that issue.
Sorry for the second-fourth commits, I forgot about good commit messages lol, so here they are
- Add text to the tag
- Add an example codeblock
- Refine the codeblock
fb4d167 Proposed alternative "available help channel" m... - swfarnsworth
141ff51 Modified the proposed message after discussion ... - swfarnsworth
211b228 Merge branch 'main' of https://github.com/pytho... - swfarnsworth
1e8715b Merge branch 'main' into swfarnsworth/help_chan... - Akarys42
6b413d3 Merge pull request #1638 from python-discord/sw... - Akarys42
[python-discord/bot] branch deleted: swfarnsworth/help\_channel\_message
Connected!
thought it'd be cool to have token detection added to the !paste command . So if someone pastes a token , and clicks save .. before it is saved it reads the whole text looking for the word "token" ... and then checks with the user before confirming save .
[python-discord/bot] New comment on issue #1645: \!paste command \- token detection and notification
The service we use for https://paste.pydis.com isn't one PyDis made themselves. It's simply a hosted version of Hastebin, so implementing this could be rather expensive to do.
[python-discord/bot] New comment on issue #1645: \!paste command \- token detection and notification
continued conversation here ..
#dev-contrib message
heard there will be a new issue created for "adding workers to add token filters"
will post the issue link here and close this issue
[python-discord/bot] New comment on issue #1645: \!paste command \- token detection and notification
Yeah, we'll open an issue over on python-discord/workers :smile:
Ignoring exception in on_command_error
Traceback (most recent call last):
File "/opt/virtualenvs/python3/lib/python3.8/site-packages/discord/client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "main.py", line 26, in on_command_error
await delete(ctx)
File "/opt/virtualenvs/python3/lib/python3.8/site-packages/discord/ext/commands/core.py", line 374, in call
return await self.callback(*args, **kwargs)
File "main.py", line 313, in delete
awai...
I think you may be in the wrong repository.
This is for the bot within the Python Discord server.
Abstract
We had instances of users inserting bot tokens and bad words inside their paste. We should proxy POSTs to the Hastebin and return a 400 if any of those is found.
Specification
We should put a worker on paste.pythondiscord.com/documents that will search the payload for a user token regex or any of a selected list of bad words. If any is found, 400 should be returned with a JSON message payload.
Connected!
48df82c Increase max size of permission number - ChrisLovering
[python-discord/site] New branch created: add\-support\-for\-d\.py\-1\.7
With the addition of new permissions in Discord, the max permission number now exceeds that which can be stored in an int.
This needs to be merged before we can bump the bot up to 1.7 and Discord.py add support for these new permissions.
See this table in the Discord API docs to see that the new max number 1 << 36
https://discord.com/developers/docs/topics/permissions#permissions-bitwise-permission-flags
Doc item doc_item.symbol_id='environment-variables' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
45dbf06 Update discord.py 1.7.3 - ChrisLovering
488c36d Update LinePaginator with new Paginator param a... - ChrisLovering
d1ba19a Don't voice verify ping users who join a stage ... - ChrisLovering
[python-discord/bot] New branch created: voice\-verify\-pings
We recently had our first large stage channel event. This led to a lot of people joining a voice channel for the first time. This normally means that they get pinged to tell them how to verify for voice.
We do not want this to happen for stage channels, as the user's don't get perms to speak there anyway.
This PR ignores users who haven't been pinged yet, when trying to ping them the voice verification warning. When they join a normal voice channel for the first time, they will be pinge...
Nitro now provides the ability to submit messages 4k characters long. When antispam in the bot attempts deletion of messages this size, it uploads them to bot/deleted_messages, which then fails with 400 Bad Request because an individual message doesn't validate.
I'll make a PR for this shortly
Is the MaxValueValidator necessary? It doesn't feel like it adds much value while adding a maintenance cost. The current max value in discord.py is also 33 bit so I'm not even sure if that or 36 would fit more
Bear in mind thread permissions are coming at some point, so with a constraint on bit length another PR like this will need to be made to upgrade to a d.py version that supports those permissions
Is the MaxValueValidator necessary? It doesn't feel like it adds much value while adding a maintenance cost. The current max value in discord.py is also 33 bit so I'm not even sure if that or 36 would fit more
Hmm yea, if we went with d.py, then we'd need to bump this with a migration each time. If we went with 36, it gives us some time, but it won't be long until Discord adds new permissions.
I tend to agree that we could remove this validator entirely.
Approving, but bear in mind above comments for the future
Bear in mind thread permissions are coming at some point, so with a constraint on bit length another PR like this will need to be made to upgrade to a d.py version that supports those permissions
the 36 bit val should take in account the new thread perms (unless some new ones are added after), but then it won't be a correct check for current d.py. I'd say the validator should just be removed
log.trace("User joined a stage channel. Ignore.")
converted to draft as I am going to remove the max value validator later this evening.
3b11890 Correct spelling error in voice_gate trace log - ChrisLovering
Ah yes. I forgot to run a coverage run. Still really confused how this didn't raise a coverage error before? I didn't delete any tests so coverage shouldn't have gone down?
[site] Branch add\-support\-for\-d\.py\-1\.7 was force-pushed to `c40ed8c`
Approving, but bear in mind above comments for the future
Thanks for taking the time to write this out, it's great. A chunk of these are just suggestions that could make it better rather than required changes, if you can't fit them feel free to just say so/let another reviewer look at it.
While these are MD files these are actually just output into discord embeds. While most markdown syntax (including links!) works in embeds, ### for titles doesn't. Other tags, like decorators.md and async-await.md use **Title** so if we intend to have a title (which I think works great here) we should use the same syntax/formatting as otherwise it ends up as literally ### Title and looks a little out of place
Consider doing f"Foo({repr(self.value)})", if that doesn't make things too complicated
Perhaps you could word the inheritance portion a little differently? All methods are inherited, dunders are just special because they come with defaults. The way this is phrased suggests that the inheritance is special.
For the curious: You can use dir(object) to see what's available by default and thus almost universally available.
return f"Foo({self.value!r})" # way to recreate this object
TIL. Yes, that's better. Thank you!
d4daa9c Filters: up character limit to 4,200 - Akarys42
[python-discord/bot] New branch created: filters/4200\-chars\-limit
Since Discord Nitro now unlock a 4,000 character limit, some of our code broke, including our filters which limit you to only post up to 3,000 character in 10s. This limit has been upped to 4,200.
I added 200 characters so if a full length message is sent with another small comment it wouldn't trigger the filter. I can think of some past instances where that would have happened.
Connected!
would it be interesting to refactor like this?
not_applicable = any(
member.bot, # is a bot
member.voice is None, # not in a voice channel
isinstance(after.channel, discord.StageChannel) # user is muted in a StageChannel
)
if not_applicable:
return
Or do you need the log.trace for each case?
Following a discussion in mod channels, we've decided it would be more useful for the seen emoji on reviews to only be used when a user has been reviewed/looked over by a mod, rather than as soon as they have seen the post. This PR changes the wording to make that change
[python-discord/snekbox] New branch created: env\-deps/add\-tzdata
Snekbox is lacking an IANA timezone database, this first-party tzdata package will provide them. It can be tested by running the following script:
import zoneinfo
if len(zoneinfo.available_timezones()) == 0:
print("The environment doesn't have a valid IANA database.")
[python-discord/snekbox] branch deleted: env\-deps/add\-tzdata
Thanks for your contributions!
[python-discord/forms-frontend] branch deleted: codemirror
19c50be Only fetch message counts if in mod channel - wookie184
[python-discord/bot] New branch created: wookie184\-user\-command\-fix
Changes to only fetch message counts for the user command if in a mod channel, as they are not displayed otherwise, and so an unnecessary api call.
This was done as the !user command can currently be quite slow even when ran in non mod channels. This would probably mean we can close #1635
Connected!
Connected!
bfcb4cb (WIP) - HassanAbouelela
[forms-frontend] Branch display\-test\-failures was force-pushed to `0b7ce9c`
Sentry Issue: BOT-147
Forbidden: 403 Forbidden (error code: 50007): Cannot send messages to this user
File "bot/exts/help_channels/_cog.py", line 586, in notify_session_participants
await message.author.send(embed=embed)
File "discord/abc.py", line 935, in send
data = await state.http.send_message(channel.id, content, tts=tts, embed=embed,
File "discord/http.py", line 241, in re...
9a0ccf9 Use the correct constant name for deleteing aft... - ChrisLovering
[python-discord/bot] New branch created: correct\-constant\-name
Connected!
Why does this PR increase the discord.py version?
I'm not sure why calling the init function of the parent class seems better than doing it manually. Additionally this should have been a separate PR.
Stage channels were introduced in 1.7 (reference).
As just discussed in our call, we will move from Pipenv to Poetry for dependency management.
Set up the Alembic project for migrating the database, and create an initial migration to bootstrap the database based on the current API database schema.
To allow for a clean move, look for a way to forward requests that do not match any defined endpoint to another API.
In our case, this will be used to forward requests that do not have an implementation on the new API to the old Django-based API, and incrementally port features over.
Create a simple workflow to lint and test the project.
Create GitHub Action to perform automatic deployment of the main branch on push. This will need coordination with Joe.
This issue should be done after the forwarding setup is considered stable, so this is not something that should be worked on at the moment.
We want to migrate the off-topic names endpoint to FastApi, as it should be a fairly straightforward migration that serves as a proof of concept. It will also help us get used to the toolset that we're going to use.
We recently added a command, !helpdm on, to the bot that allows users to enable notifications for the help channel of others that they are active in. However, the command is currently only mentioned in a #changelog entry, which scrolls out of view rather quickly.
@MarkKoz suggested that we should add it to the help channel guide, which is a good idea.
yep, i forgot about that, i will use the double asterisks
yep, i will add that in the commit i'm editing right now!
@Objectivitix When creating a commit message, please make sure that if a person was to look back into git history they'd be able to get a hint of what occurred in a change/addition/deletion.
For more information on how to create a great commit message, see this tutorial.
Looks good, just two last things
I'm not sure on this. The NOTE seems a bit extra. Giving it a moments thought I think we shouldn't include this--one of the goals with tags is as a famous engineering adage goes:
Perfection is achieved, not when there is nothing more to add, but when there is nothing left to take away
The goal here is to explain dunders correctly, succinctly, and with the goal of educating the reader. I think this note could probably be dropped to make this easier to read and digits without losing ...
Sorry, I'm going to be a little nitpicky here. I think this can be worded a little better. At the moment, the first sentence is quiet long, with a lot of asides in it. We could probably get away with something much shorter. I don't think we need to really explain inheritance here. The first sentence attempts to do that a little, but I think we can get away with just stating that there are defaults that come from object.
d2112a8 Update navbar hamburger/dropdown anchors to spans - jb3
[python-discord/site] New branch created: jb3/navbar\-spans
There is no href on these items, so they should not be anchors.
@Xithrius thanks for telling me this. From now on, I'm putting all the changes in the extended description.
@Xithrius thanks for telling me this. From now on, I'm putting all the changes in the extended description.
I think this should only occur for commits that absolutely need more of an explanation. The message should be short enough to where it's not truncated by GitHub, and give a very quick insight to the change.
Connected!
Fair, enough my bad, I seem to have overlooked that part in the message.
This is a breaking change in d.py 1.7 so needs to be part of this PR.
Calling the parent constructor was just for simplicity.
I found the separate trace logs for each case very useful during dev testing. I'm hesitant to make this change for that reason.
Sure, but wouldn't it be cleaner to call the functions manually? Since that's the way it was done before
Thanks! The coverage is due to a branch never being ran, see my message below.
This except block is causing the 100% coverage to fail because it is never run. Can you please obliterate it?
This same change was made when sir-lancebot was upgraded to 1.17
Do we want to exclude tests from coverage? Knowing if some test or its portion didn't run can be important, and both are also measured on the bot repo. For cases where it isn't feasible to do we can ignore the part that isn't covered
We definitely don't need a separate PR for every small change, the feature needed a version bump of a dependency so that was done.
Calling the super class' init feels better to me as in case the underlying init changes we won't be storing the attributes in a possibly incompatible way
d4daa9c Filters: up character limit to 4,200 - Akarys42
bd1ee16 Merge pull request #1649 from python-discord/fi... - ChrisLovering
4bd92ca Change seen emoji to reviewed emoji - wookie184
19c50be Only fetch message counts if in mod channel - wookie184
46b05e1 Merge pull request #1651 from python-discord/wo... - Xithrius
[python-discord/site] branch deleted: add\-support\-for\-d\.py\-1\.7
[python-discord/site] New branch created: jb3/remove\-rlbot
Removes RLBot from the Communities list on the resources page.
45dbf06 Update discord.py 1.7.3 - ChrisLovering
488c36d Update LinePaginator with new Paginator param a... - ChrisLovering
d1ba19a Don't voice verify ping users who join a stage ... - ChrisLovering
3b11890 Correct spelling error in voice_gate trace log - ChrisLovering
b6279c7 Merge branch 'main' into voice-verify-pings - Xithrius
[python-discord/bot] branch deleted: voice\-verify\-pings
Connected!
[python-discord/site] New branch created: cj8\-qualifier\-release
This PR updates the website to include information for the Qualifier Release today. It adds the following:
- Adds a page for the GitHub Bootcamp information. Right now it's fairly basic. It will be more updated in a later PR.
- Adds Tabnine as a sponsor
- Adds a prize section with a quick blurb for each of the sponsors
- Adds information about the qualifier itself and the signup link
c40ed8c Use BigInt for permissions field and remove max... - ChrisLovering
7bf685d Merge pull request #534 from python-discord/add... - ChrisLovering
115220e Remove RLBot from Communities list - jb3
eef7f1f Merge pull request #540 from python-discord/jb3... - Akarys42
01ca1d7 Merge branch 'main' into cj8-qualifier-release - jb3
This session will run for an hour. It will be focused on using git and GitHub in a team environment.
1-year JetBrain licenses to the members of a winning team.</p>
$250 in DigitalOcean credits to the members of a winning team.</p>
1-year Pro Licenses to Tabnine to the members of a winning team.</p>
[python-discord/forms-backend] Checks Successful on PR: #97 Bump python-dotenv from 0.17.0 to 0.18.0
GitHub Actions run 958503752 succeeded.
We've decided to go through with the feature.
Some things to keep in mind for this:
- Handling staff restricted commands in the new file structure. @Numerlor's seems like a good solution.
- Discoverability. Tags under groups should remain as discoverable as possible. One approach is that when
!tagsis called, to list all groups and their tags, similarly to how extensions and their commands are currently listed under!help. Furthermore,!tags <group_name>can list only the tags unde...
This means messages with both @everyone and a watched term will now ping the mod team instead of hiding beneath the everyone ping's log message which does not ping
We've decided to go through with the feature.
@Numerlor if you're still interested I can assign you.
sure, I can start working on it in a bit
https://imgur.com/a/tYsTD7S
this suggestion is for adding documentation to the tooltip
suggesting to have a tooltip on which time format is accepted when setting a reminder .
started with doing!remind new 12 api and it didn't accept the format. (i was guessing what this would do, because otherwise i would check the code on the github to find the formats) , and then i assumed that a h format could be used and it worked .
note:
to quickly get more information in tooltips while...
57bc7ba Prevent one from using the !echo command for ... - swfarnsworth
[python-discord/bot] New branch created: swfarnsworth/echo\_permissions
Changes the !echo command so that if someone attempts to echo to a channel in which they don't have send message permissions, the message is not relayed. Though that has never happened in the history of Python Discord.
await ctx.send("You don't have permission to speak in that channel.")
What sort of escapism is this? ๐ฆ ๐ฆ๐ฆ
0c87e5a Update bot/exts/utils/bot.py
- swfarnsworth
Connected!
54e1dce Replace pipenv by poetry - SebastiaanZ
[python-discord/api] New branch created: sebastiaan/6\-migrate\-to\-poetry
I've replaced pipenv by poetry, as discussed in #6. This also means that the Dockerfile had to be rewritten to support poetry as the dependency manager instead of pipenv.
In addition, since I wasn't happy with how well poetry and pre-commit worked together when it came to running "local" commands, I've temporarily removed pre-commit from our stack. We can reevaluate this later on.
This PR closes #6.
Considering this being an ongoing issue, it's unnecessary to block this bugfix until the new system. This is now open to implementing a fix for by anyone interested in doing so.
[python-discord/site] New branch created: bootcamp\-update
Adds time and more information about the GitHub bootcamp.
I was told to approve this, I'm slightly intoxicated but it looks good. :trollface:
You'll learn how to set-up a GitHub repository with other contributors, how to make branches, make commits, pull down changes, and then make pull requests.
Typo
The instructional parts of this Bootcamp will be recorded. We'll also be providing a cheatsheet/reference guide to all attendees that will
In the first sentence with strong, bootcamp is capital.
We'll have helpers available to help you set up git and integrated with your editor of choice.
Typo
All looks good. No typos as far as I can tell.
[python-discord/site] branch deleted: bootcamp\-update
Yeah let's go with the other option, let's just ignore the particular test we don't want to be included.
Anything else I should improve :eyes: ?
As discussed in dev-core, we should add a pragma no cover here, so this specific branch is ignored, but we keep coverage on other tests
I'm going to rebuild the migration since now migrations have broken with the latest other branch merge to main
[python-discord/site] branch deleted: jb3/navbar\-spans
a8a0322 Adjust Message model to support new nitro messa... - bast0006
05730b3 Add pragma: no cover to mark failure condition ... - bast0006
88f91ab Merge remote-tracking branch 'upstream/main' in... - bast0006
6c9c7f0 Recreate nitro message length migration and nam... - bast0006
abccdba Merge branch 'main' into fix-nitro-message - Akarys42
GitHub Actions run 968625616 succeeded.
GitHub Actions run 968625699 succeeded.
GitHub Actions run 968632366 succeeded.
Relevant Issues
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->
Closes #769
Description
I modified the code to use the proper API endpoint, and changed the HTTP parameters accordingly.
Also, I removed a reference to an environment variable, that was unnecessary. ( That parameter (TMDB_TOKEN) wasn't even present in the list on the Wiki)...
[python-discord/sir-lancebot] New review comment on pull request #583: Add the stackoverflow command
The IDs here will need to be added eventually, yes?
[python-discord/sir-lancebot] New review comment on pull request #583: Add the stackoverflow command
Eventually, yes, it would be needed.
[python-discord/sir-lancebot] New review comment on pull request #583: Add the stackoverflow command
So these are the icons that you would need added to the server as emojis?
[python-discord/sir-lancebot] New review comment on pull request #583: Add the stackoverflow command
Yup, since for the rest of the icons I am using the reddit icons.
Looks good to me, one comment
We might as well use the constant in constants.py here.
Sure, I'll make a commit shortly.
Connected!
[python-discord/site] New branch created: branch
[python-discord/site] branch deleted: branch
@doublevcodes will you be able to finish this up?
a56e02b Create issue template for resource suggestions. - swfarnsworth
4893ab9 Merge branch 'main' into swfarnsworth/resource_... - ChrisLovering
efa3c45 Merge pull request #524 from python-discord/swf... - ChrisLovering
1fd6c9b Added Cloudflare to Sponsors list (#527) - Kronifer
b8132bd Code Jam 8 - Initial PR for Announcement (#528) - janine9vn
[python-discord/site] branch deleted: move\-config\-docs\-to\-contributing\-guide
fc652a7 Move configuration section to site contributing... - jchristgit
172f839 Kill docs/ directory. - jchristgit
ef59812 Address review comments. - jchristgit
cc59fc7 Merge branch 'main' into move-config-docs-to-co... - kosayoda
5b64d1f Merge pull request #523 from python-discord/mov... - kosayoda
@doublevcodes will you be able to finish this up?
I will try do it ASAP but I wouldn't say I'll manage in any less than 2 weeks because IRL stuff
@doublevcodes will you be able to finish this up?
I will try do it ASAP but I wouldn't say I'll manage in any less than 2 weeks because IRL stuff
Would you still like to complete it yourself, or would you like to have another human take this task?
[python-discord/site] branch deleted: master
[python-discord/site] New branch created: lemon/summer\_code\_jam\_21/front\_page\_banner
- Move existing code jam assets into a folder.
- Add some new Summer Code Jam banners.
- Put Summer Code Jam '21 banner on the front page.
[python-discord/site] branch deleted: lemon/summer\_code\_jam\_21/front\_page\_banner
f23f1ce Docker-compose: upgrade to postgres 13 - Akarys42
[python-discord/site] New branch created: docker\-compose/postgres\-13
We have been using postgres 13 in production for a long time, but we never actually updated the docker-compose environment.
A draft of the announcement can be found [here](#dev-core message).
[python-discord/site] New branch created: lemon/summer\_code\_jam\_21/front\_page\_banner\_fixes
- Update banners to have vertically centered text.
- Hyperlink code jam banner to code jam page.
[python-discord/site] Checks Successful on PR: #546 lemon/summer code jam 21/front page banner fixes
GitHub Actions run 976301042 succeeded.
[python-discord/site] branch deleted: lemon/summer\_code\_jam\_21/front\_page\_banner\_fixes
9eb774d Docker-compose: upgrade to postgres 13 - Akarys42
[python-discord/bot] New branch created: docker\-compose/postgres\-13
[python-discord/bot] branch deleted: docker\-compose/postgres\-13
Connected!
f7f73f2 Add alias for voice_verify command - wookie184
[python-discord/bot] New branch created: wookie184\-voiceverify\-alias
Added voice-verify alias as the channel and tag are hyphenated so it may be confusing as it is currently.
Thanks for the double quotes ๐
[python-discord/bot] branch deleted: wookie184\-voiceverify\-alias
Connected!
The override file can be used to customize compose locally; makes the gitignore consistent with bot's which had the file added in https://github.com/python-discord/bot/pull/1584
a71d37b Allowed text format warning to have multiple fo... - Xithrius
4edecf6 Added .txt file extension to antimalware test. - Xithrius
d510a6a Added subtests for .txt, .json, and .csv ... - Xithrius
aeaef8f Added new formats to unittest docstrings. - Xithrius
a305d39 Renamed blocked_extension_str to blocked_extens... - Xithrius
Looks good! Signing off for DevOps.
[python-discord/bot] branch deleted: xdist
Connected!
[python-discord/bot] New branch created: move\-cov\-config\-to\-toml
How does a test not run? How does that make it past review?
https://nedbatchelder.com/blog/201908/dont_omit_tests_from_coverage.html
I had a horror story in which pytest-twisted was not installed so most tests were passing without actually scheduling any coroutines or running any assertions
this pytest-xdist is now outdated I upgraded it here https://github.com/python-discord/bot/pull/1659
Looks good to me, consider this approved. FWIW I prefer the second implementation proposal
Hi! I would like to give it a try! ๐
[python-discord/site] New branch created: bootcamp\-update
[python-discord/site] branch deleted: bootcamp\-update
Doc item doc_item.symbol_id='environment-variables' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
16b31f5 chore: Merge the .issue command into the github... - ToxicKidz
[python-discord/sir-lancebot] New branch created: merge\-github\-issues
Relevant Issues
Closes #711
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->
Description
I moved the code in issues.py to githubinfo.py, and the .issue command was moved into the .github group. This means that you can invoke the command like .github issue 1, but it uses root_aliases so you can still do .issue 1. The automatic linking of issues was also moved into this cog.
Did you:
- [x] Join the [**Python D...
fc30c54 Ensure mods cannot be watched - ChrisLovering
[python-discord/bot] New branch created: don't\-allow\-mods\-to\-be\-watched
This meant admins could also be watched, meaning their messages in any channel would be relayed to the BB channel.
Connected!
[python-discord/bot] New branch created: move\-cov\-config\-to\-toml
[python-discord/bot] branch deleted: move\-cov\-config\-to\-toml
Connected!
317a06f Remove the pixels token detector - ChrisLovering
[python-discord/bot] New branch created: remove\-pixels\-token\-detection
The event is over now, so it doesn't need to exist anymore :)
Sentry Issue: BOT-14F
DNSError: (4, 'Domain name not found')
File "aiohttp/resolver.py", line 91, in resolve
resp = await self._resolver.gethostbyname(host, family)
OSError: Domain name not found
File "aiohttp/connector.py", line 999, in _create_direct_connection
hosts = await asyncio.shield(host_resolved)
File "aiohttp/connector.py", line 865, in _resolve_host
addrs = awai...
c o m p l e t e l y a n n i h i l a t e d
[python-discord/bot] branch deleted: remove\-pixels\-token\-detection
Connected!
In the Conversation Etiquette in Our Off-Topic Channels page, it says that there are "three off-topic channels". The current count seems to be about 22. I would suggest "dozens of off-topic channels" or "many off-topic channels".
Are you talking about topical channels? Off-topic channels are the ones generally under the discussion category (currently under the code jam), in a category itself called OFF-TOPIC / GENERAL. There are currently only three of them.
Are you thinking of our help channels? We definitely have 3 off topic channels

Ooops. My bad. I had seen the "topical chat/help" channels, and so I then translated the "help channels" in my head to be "off-topic help channels". Nothing to see here. Move along. Closing.
Introduces tag groups described in #1545.
When matching a tag with a group, the group is matched with no minimum amount of chars (unlike a tag that has to have 3 chars of its name) to allow a short invocation name to be used and possibly prevent some conflicts with tag names where multiple tags would be suggested instead of a s tag being sent.
Additionally tags now support metadata through a frontmatter block like in [meta files in branding](https://raw.githubusercontent.com/python-disc...
As it is, listing the top level group alone isn't possible because it lacks a group to look it up with. Would some kind of a string (None, nogroup or something like that) be worth using here to list them alone instead of only being accessible through the whole list
This is done in the help command

which links to strftime.org but I do think updating the help command to instead show a few examples would be nice.
[python-discord/api] New branch created: add\-ci
PostgreSQL is already set up and configured for later, and simply needs to be
commented out to start.
The workflow YAML is mostly taken from the site, and should thus be equally as
fast due to dependency caching and other involved magic.
Yeah I don't know why I did that, removed in b8d58e0
Yeah, I tried to be smart and tripped on my own fit. Untangled in 76e62b3
When an infraction is dealt, the user is DMed an embed with infraction information.
We'd like to have a command that takes an infraction ID, and sends that embed again. A possible syntax can be !infraction resend
Various behaviors might cause the bot to mute a user.
When a moderator deals a mute, it should disable a pre-existing bot mute.
Superseded by #1665
6f45d68 Bump embed limit to 4096 characters - Akarys42
[python-discord/bot] New branch created: new\-discord\-features
- [X] Bump embed limit to 4096 characters
- [ ] Use localized timestamps
Currently when the !server command is used, we get an embed like the one below. Since a stage channel has been added recently, the roles/channels listing is now uneven.
I suggest that we combine the project leads and domain leads role into a singular role called "leads".

This is the part of the code based this role list is currently generated, as a hint for whoever picks this up https://github.com/python-discord/bot/blob/6b6a291625ce01517df29dc0d0f6170cbe8f3044/bot/exts/info/information.py#L44-L53
231f6dc Tests: remove stale patch of time_since - Akarys42
This is now ready for review! That said since iOS doesn't have support for timestamps yet, I'm leaving this PR in a draft state for the time being.
Should we consider having the integer 4096 as a constant?
Please approve my PR @scragly
Unfortunately, your PR does not meet any of our standards. As such, it's been closed.
Please don't create unnecessary PRs like this. If you have a genuine interest in contributing to our projects, ensure you follow our contributing guidelines, read the appropriate resources to know our processes, and follow these processes correctly the whole way through.
If you're interested in actually contributing properly, please reference our [Contributing] page on our website, read the resources ...
Thanks for your instructions and will surely abide by the instructions.
Connected!
Should we consider having the integer
4096as a constant?
The thing is some modules needs 4096, some 4093, 4090 or even 4000. I think it would have been worth it from the start, now I doubt discord will ever change it again. I don't believe it is worth changing and retesting everything to use 4096, minus some magic numbers in some cases.
Agreed. These numbers will remain where they are.
cb058d8 Merge branch 'main' of https://github.com/pytho... - swfarnsworth
8cb4d72 Add final line break for style compliance. - swfarnsworth
4633910 Add resource Data Science from Scratch. - swfarnsworth
45128f4 Add resource regex101. - swfarnsworth
[python-discord/site] New branch created: swfarnsworth/smarter\-resources/new\-resources
Connected!
Hey! I'd like to grab this issue! :)
Go for it! If you have any question, feel free to ask in #dev-contrib!
Connected!
Connected!
1c67bc3 chore: Add the codejam create command - ToxicKidz
[python-discord/bot] New branch created: feat/code\-jam\-channels\-automation
Relevant Issues
Closes #1601
Description
This admin only command takes a CSV file or a link to one. This CSV file has three rows: 'Team Name', 'Team Member Discord ID', and 'Team Leader'.
The Team Name will be the name of the team's channel, the Member ID tells which user belongs to this team, and team leader, which is either Y/N, tells if this user is the team leader. It will create text channels for each team and make a team leaders chat channel as well. It will ping the Even...
GitHub Actions run 1002395230 failed.
[bot] Branch feat/code\-jam\-channels\-automation was force-pushed to `7d1ee89`
GitHub Actions run 1002401673 failed.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
We're not planning on giving mods access.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
Do we really want to continue here? ( @janine9vn )
In general, I wonder if it's better to validate the CSV before doing anything with it:
- Make sure all ID's are valid
- Make sure there are only Y or N values for leadership
- Make sure there is only one leader
- Make sure all team names are valid channel names.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
This line is no longer up to date.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
In the future, probably. I imagine this command would be rolled in with out overall code jam management system. In this specific case for this code jam, I'm operating under the assumption that the CSV is valid to begin with.
9ae03f3 Adds Documentation For Running A Single Test - HassanAbouelela
[python-discord/bot] New branch created: running\-single\-tests
Adds a portion to the testing README explaining how and when to run an individual test file when working with tests.
Additionally adds a table of contents as the document has become quite long.
I used nocov instead of the regular version in the documentation as the options seemed to interfere with the selection (using cov and cov-report leads to pytest not using the passed path, and instead collecting all the tests).
These changes do not cover how to further specify tests or test sui...
GitHub Actions run 1002616683 succeeded.
All hyperlinks work, I see no errors in the English language.
Very well.
089ef62 Adds Documentation For Running A Single Test (#... - HassanAbouelela
[python-discord/bot] branch deleted: running\-single\-tests
Connected!
GitHub Actions run 1002626501 succeeded.
GitHub Actions run 1002648481 succeeded.
37f8637 chore: Remove the moderators role from the team... - ToxicKidz
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
I guess this changed from the original issue, fixed in https://github.com/python-discord/bot/pull/1668/commits/37f8637955350c6c5b437d66807f42d8f26adfcd.
GitHub Actions run 1002658511 succeeded.
GitHub Actions run 1002769332 succeeded.
Maybe some filter arguments? That could be a good feature.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
I think a higher log level would be useful here, or maybe even send a message to the channel saying that it contains the following invalid members and then it lists down all of them.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
I think it would be good if we keep track of members who could not be found in a list, and then send them in an embed at the end. This will allow the person running them to easily know who needs to be manually contacted.
Looks good, just one comment for future improvement :)
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
But also, the fact that J9 said below that she's operating on the assumption that this csv is valid for now, I don't think we need this for this PR.
7d1ee89 chore: Add the codejam create command - ToxicKidz
2a5a15f chore: Update tests for the new codejam create ... - ToxicKidz
37f8637 chore: Remove the moderators role from the team... - ToxicKidz
1905f7c chore: Change the Code Jam Team Leader role's... - ToxicKidz
646b840 Merge branch 'main' into feat/code-jam-channels... - ToxicKidz
[python-discord/bot] branch deleted: feat/code\-jam\-channels\-automation
Connected!
GitHub Actions run 1003637245 succeeded.
[python-discord/bot] New review comment on pull request #1668: chore: Add the codejam create command
Alright sounds good ๐๐พ
This disables filter_zalgo, filter_invites, filter_domains, watch_regex, watch_rich_embeds, and filter_everyone_ping in the code jam team channels (the main one we wanted to disable was filter_domains but it's easiest to do all at once, just wanted to list them to be clear of the changes).
Code jam team channels are based on the category name Code Jam used by the jams cog, so we shouldn't put any public channels under a category with that name. This code will likely be replac...
GitHub Actions run 1003950321 succeeded.
GitHub Actions run 1004008801 succeeded.
GitHub Actions run 1004049075 succeeded.
Connected!
GitHub Actions run 1004132276 succeeded.
Sentry Issue: BOT-14P
AttributeError: 'DMChannel' object has no attribute 'category'
File "discord/client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "bot/exts/filters/filtering.py", line 166, in on_message
await self._filter_message(msg)
File "bot/exts/filters/filtering.py", line 287, in _filter_message
category = msg.channel.category
Unhandled exception ...
Sentry Issue: BOT-14Q
AttributeError: 'DMChannel' object has no attribute 'category'
File "discord/client.py", line 343, in _run_event
await coro(*args, **kwargs)
File "bot/exts/filters/filtering.py", line 183, in on_message_edit
await self._filter_message(after, delta)
File "bot/exts/filters/filtering.py", line 287, in _filter_message
category = msg.channel.category
Unhand...
3996c2f Use getattr with a default, to protect against ... - ChrisLovering
[python-discord/bot] New branch created: fix\-category\-check\-in\-dm\-channels
GitHub Actions run 1004177702 succeeded.
[python-discord/bot] branch deleted: fix\-category\-check\-in\-dm\-channels
Connected!
GitHub Actions run 1004375394 succeeded.
GitHub Actions run 1004422380 succeeded.
[python-discord/bot] branch deleted: max\-mentions\-8
412719d Disable everyone ping filter in jam channels - mbaruh
[python-discord/bot] New branch created: mbaruh\-patch\-1
GitHub Actions run 1004629878 failed.
GitHub Actions run 1004644598 failed.
GitHub Actions run 1004651024 succeeded.
Connected!
GitHub Actions run 1004676288 succeeded.
ce35566 Allow commands in code jam team channels - wookie184
[python-discord/sir-lancebot] New branch created: wookie184\-codejam\-permissions
Relevant Issues
It is mandatory to link to an issue that has been approved by a Core Developer, indicated by an "approved" label.
Issues can be skipped with explicit core dev approval, but you have to link the discussion.
Uh, well, actually technically I didn't do that, sorry ๐ข
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->
Description
I added permissions for members to run sir-lancebot commands in their codejam team channels...
[python-discord/sir-lancebot] Checks Successful on PR: #779 Allow commands in code jam team channels
GitHub Actions run 1005102872 succeeded.
[python-discord/sir-lancebot] branch deleted: wookie184\-codejam\-permissions
Connected!
GitHub Actions run 1005116040 succeeded.
GitHub Actions run 1005290919 succeeded.
Connected!
GitHub Actions run 1005769728 succeeded.
[python-discord/sir-lancebot] New branch created: remove\-hacktoberfest\-whitelist
Sentry Issue: SIR-LANCEBOT-48
Channel "hacktoberfest_2020" with ID 760857070781071431 missing
198d22c Remove hacktoberfest channel overrides - ChrisLovering
Relevant Issues
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->
Closes #780
Description
The hacktoberfest channel has now been removed, so these overrides can also be removed.
The commands may still be useful in future, and can be used in lancebot's playground.
Did you:
- [ ] Join the Python Discord Community?
- [ ] Read all the comments in this template?
- [ ] Ensure there is an issue ope...
GitHub Actions run 1005793774 succeeded.
Everything looks good to me!
@vcokltfre Are you still working on this? I'm assuming that this hasn't been worked on because I can't seem to find any PRs related to this...
f249ca3 Removes Unnecessary Docker Packages - HassanAbouelela
[python-discord/metricity] New branch created: drop\-deps
[python-discord/sir-lancebot] branch deleted: remove\-hacktoberfest\-whitelist
Connected!
GitHub Actions run 1005987867 succeeded.
Removes the GCC & git installation steps from the docker file. They were added end of last year, and my best guess is they were used to install packages while wheels were not available. They are no longer needed.
To address similar problems in the future, I've locked python version more tightly (now supporting 3.8,3.9 instead of >3.8), so we don't end up in a similar situation when 3.10 releases. I've locked the docker version as well.
I also bumped the program version.
[python-discord/metricity] branch deleted: drop\-deps
Description
Implement a game inspired by the classic card game, Set, using pydis ducks.
After running the command, the bot posts an embed with an image of the board consisting of 12 cards. Players race to claim valid sets of 3 cards. Correct answers earn them a point, incorrect answers deduct a point. After a time limit the game ends, and the scores are displayed in the initial embed along with any valid sets that went unclaimed. Alternatively, a p...
Description
An Avatar modifier that will return a gif of your Avatar, but turns into a rickroll
Reasoning
Just a fun trick when someone asks for your profile and if you have nitro and want it. Also, it's just a fun command lmao.
Proposed Implementation
Don't have it yet. I can't seem to find the person who's pfp looks like this lmao (or any recording). I can make one though if needed. Don't want to do unnecessary work if this gets rejected.
Would you like to implement th...
I love this idea, let's do this! Even as a proof of concept your board looks rather nice.
This looks great! I wonder how much help quackstack could be for this, would be worth reaching out to @vcokltfre
To generate those ducks randomly, quackstack ๐ฆ would be great. If you want to keep the ducks similar you could randomly choose details (/details/{type}) and provide it to the API, allowing it to choose the others.
Alec was working on a package version of quackstack, there is a WIP PR open for that, not sure how long it would take ๐คท๐พโโ๏ธ
There are 3^4 = 81 cards, so I pre-generated all of them into a single PNG. Assembling a board is then just a matter of slicing and pasting the cards you want. I considered generating them on the fly, but I think it's just simpler this way.
Removes all dependencies licensed under the GPL license, and replaces them where possible.
- Fuzzywuzzy -> Rapidfuzz (MIT)
- AIOPing -> Removed
All other packages were double-checked by liccheck, so hopefully, they shouldn't present a problem.
GitHub Actions run 1008480650 failed.
Running an update ended up introducing a new linting rule (N818). I'll try to figure out the best way to handle it.
I decided to just kaizen in the renames.
GitHub Actions run 1008534515 succeeded.
Description
In love calculator when love is found 0% it does not show "There's no real connection between you two" like in json file.
Steps to Reproduce
Found it while getting love as 0%
Expected Behaviour
It should be "There's no real connection between you two" as per json(I did try to check if it was sarcasm for 0% but that's not the case)
Actual Behaviour
![i...
Ah yeah, thanks for pointing that out.
e6fed40 Prevents Blocking In Ping Command - HassanAbouelela
GitHub Actions run 1008806980 succeeded.
While this truly is a bug, I think it is quite fun and it is a known hidden fact of our bot, I would actually vote to not fix it.
haha yeah, @Akarys42 I actually thought that this may be a sarcastic intended feature too, but looking at code, it's indeed bug.
But +1 to not fixing the bug lol(or maybe fixing the bug and changing the value of 0% as in json for current funny output.)
Relevant Issues
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->
Closes #784
Description
I removed the -1 from the quote index calculation at This means the command [p]love ([p] refers to Lancebot's prefix) will now return the correct quote for love_percent = 0 edge-case, and scores in the range [96, 100] will now correctly use the 100 quote from bot/resources/valentines/love_matches.json instead of the 95 quote.
Did y...
Converted to draft as the issue isn't approved yet
[python-discord/snekbox] New branch created: graingert\-patch\-1
[python-discord/snekbox] branch deleted: graingert\-patch\-1
GitHub Actions run 1009438750 succeeded.
Connected!
0cec14f Create The Code Style Guide under Code Jam pages - ks129
20ae1ec Add The Code Style Guide to useful information ... - ks129
fac0a2b Add The Code Style Guide to CJ 8 sidebar - ks129
Created CJ page The Code Style Guide that explains the importance of code style and gives basic Python code style rules. Added page to useful information and CJ 8 sidebar.
Please note that I didn't actually run the site, because I currently can't.
GitHub Actions run 1012487861 failed.
GitHub Actions run 1012513575 succeeded.
Seems good to me.
I haven't run the site either, as I currently have no access to Docker or Python.
The multiline sentence start and end quotes should be at different lines than the content.
What are the item tags for?
The codeblocks also currently contain trailing whitespace
Just like block comments, inline comments also have to start with <code>#</code> followed by a single space.
Currently all the block code tags include leading and trailing newlines and are indented because of the HTML structure, dedenting and removing all the non code characters in the pre tag works but I'm not sure if that's the proper solution

The comment and docstring paragraphs feel a bit too dense to me:

There were some other places where the wording didn't quite fit but I didn't come up with something that felt m...
1521fc5 Fallback to the old API for undefined routes. - jchristgit
[python-discord/api] New branch created: fallback\-to\-old\-api
Shouldn't we check for the result of default_process here anymore, in case it returns an empty string? Or does it not matter as much anymore, since rapidfuzz won't issue a warning like fuzzywuzzy?
Looks good. I left a small comment, but feel free to ignore it completely.
I remember (sort of) bringing this up when I originally added this:
https://github.com/python-discord/bot/pull/53#discussion_r182782953
This adds a new endpoint handler using the path converter in order to match
requests directed to any subtree. As long as this endpoint is the last to be
included in the router, all requests not matching defined routes under the
/api endpoint will be forwarded to the old API.
Two new settings are added for this. One is the endpoint of the old API to
forward to, which may change in the future due to the pending subdomain removal
pull request. The other is the API token to use for th...
That was basically my reasoning. The original comment said that was done due to the fuzzywuzzy warning, which doesn't exist in rapidfuzz, so it'll safely return an empty list.
Yes, that's a great idea ๐
fc652a7 Move configuration section to site contributing... - jchristgit
172f839 Kill docs/ directory. - jchristgit
a56e02b Create issue template for resource suggestions. - swfarnsworth
4893ab9 Merge branch 'main' into swfarnsworth/resource_... - ChrisLovering
efa3c45 Merge pull request #524 from python-discord/swf... - ChrisLovering
GitHub Actions run 1012782685 succeeded.
The purpose of this PR is for us to avoid manually searching the permissions of team channels, and dealing with user-based overwrites. This also allows us to quickly tell what team a participant belong to.
Added:
- An info embed with team the member is in. The team is decided by finding in which channel the member has overwrites.
- Command to move a member from one team to another by changing the permissions of the appropriate team channels.
- A command to end the code jam and delete ...
GitHub Actions run 1012944273 succeeded.
GitHub Actions run 1012960624 succeeded.
When !help test is used, the following error occurs:

bot_1 | 2021-07-08 20:54:11 | bot.exts.backend.error_handler | ERROR | Error executing command invoked by Xithrius#1318: \help test
bot_1 | Traceback (most recent call last):
bot_1 | File "/usr/local/lib/python3.9/site-packages/discord/ext/commands/core.py", line 85, in wrapped
bot_1 |...
GitHub Actions run 1013048003 succeeded.
GitHub Actions run 1013056855 succeeded.
GitHub Actions run 1013067299 succeeded.
GitHub Actions run 1013072932 succeeded.
bdc492f Escape HTML in The Code Style Guide - ks129
I missed this when I converted from MD to HTML. Escaped now.
GitHub Actions run 1013085138 succeeded.
GitHub Actions run 1013126405 succeeded.
20962cb Split comments paragraphs to even smaller parag... - ks129
Closes #1667
After I proposed a solution on Discord, I made a possible implementation, I tried to make it readable by creating a helper function that might be useful later on if you're planning to other groups. Although I'm not fully sure about it.
GitHub Actions run 1013139984 succeeded.
@Numerlor Code blocks should look okay now and also I split some of the comment section paragraphs.
GitHub Actions run 1013142590 failed.
GitHub Actions run 1013178525 succeeded.
GitHub Actions run 1013217756 succeeded.
Connected!
GitHub Actions run 1013228634 succeeded.
@MarkKoz I think the current duplicated functions should remain where they are for now. If we do decide to move these functions somewhere else, it should be within a different PR focused on refactoring.
The functions themselves in my own attempt of refactoring ended up taking a bit too many variables as arguments. That may have just been me though, and it was quite a while since said last attempt. Although at this point in time it seems to be still the same case.
[python-discord/site] New branch created: cj8\-info\-update
Updates the Important Dates section of the website with the most recent changes.
It also adds Textual as a conditionally approved framework participants can use.
GitHub Actions run 1013359162 succeeded.
GitHub Actions run 1013375604 succeeded.
Let's make the url just code-style-guide and drop the "the"
Overall, love the content of this and I like the examples included. Thank you for doing this!
I've made some adjustments to the wording written so far. Almost all of it is small grammar changes to help the sentences flow a bit better.
I am going to approve this even though I have some comments and suggestions. The suggestions I made aren't critical and I don't want me being unavailable at work or in flight to block merging this.
I'm a bit concerned about the readability of this. Testing locally, I like this format better. Makes it less prone to eyes skipping over large chunks of dense text.
<li>
<h4>Block comments</h4>
<p>Probably the most common comment type. They should be indented to the same level as the code they describe.
Can we make this an iframe and embed it?
Neither urgent nor important, so feel free to skip this suggestion if it takes more than 30 seconds.
5bbd29c Properly Handle Fuzzy Matching Help - HassanAbouelela
Hey @Xithrius thanks for catching that. Seems it only affects help matching. I've pushed a fix in 5bbd29c.
Doc item doc_item.symbol_id='urwid.Widget' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
Doc item doc_item.symbol_id='urwid.MainLoop' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
GitHub Actions run 1013519941 succeeded.
Interesting implementation of proxying request the old api. Two small comments and rest looks good. I am not really sure about the tests if there is a better way to do the mocking.
You can assign this to me, I can start working on this once #7 is mentioned.
[python-discord/api] New comment on pull request #14: Fallback to the old API for undefined routes\.
Could we possibly add the python-discord/site GHCR package to docker-compose.yml?
Tested, things now work. Very nice.
[python-discord/bot] branch deleted: drop\-gpl\-deps
GitHub Actions run 1014140320 succeeded.
Connected!
GitHub Actions run 1014147317 succeeded.
The same information is at the line before this line.
GitHub Actions run 1014657273 failed.
GitHub Actions run 1014660804 succeeded.
GitHub Actions run 1014666727 succeeded.
GitHub Actions run 1015060514 succeeded.
GitHub Actions run 1015068872 succeeded.
GitHub Actions run 1015074611 succeeded.
GitHub Actions run 1015109486 succeeded.
Seems to be running, the content looks good, thank you!
[python-discord/site] branch deleted: the\-code\-style\-guide
0cec14f Create The Code Style Guide under Code Jam pages - ks129
20ae1ec Add The Code Style Guide to useful information ... - ks129
fac0a2b Add The Code Style Guide to CJ 8 sidebar - ks129
25a5057 Remove trailing whitespace from The Code Style ... - ks129
23b1e46 Add missing slash at b tag ending - ks129
[python-discord/site] New branch created: cj/minor\-adjustments
- Switch the right menu of each code jam page to ongoing

- Use the ongoing banner on the main page

- Use the right banner on the front page

GitHub Actions run 1015725106 succeeded.