#dev-log

1 messages · Page 75 of 1

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/site] New branch created: update/sir\-lancebot\-env\-vars
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
#
Akarys42
[python-discord/sir-lancebot] branch deleted: pipenv\-to\-poetry
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
jb3
[python-discord/site] branch deleted: update/sir\-lancebot\-env\-vars
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: fix\-dockerfile
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] branch deleted: reword\-contrib\-guides
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: nomination\-archive\-automation
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: vcokltfre/feat/bookmark\-reply
oak estuaryBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Akarys42
[python-discord/bot] New branch created: incident\-emojis
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: incident\-emojis
oak estuaryBOT
odd spireBOT
regal archBOT
#
jb3
[python-discord/site] New branch created: mod\-policy\-redirect
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[sir-lancebot:main] 63 new commits

4c3312a Add 30 math questions and 30 science questions ... - Objectivitix
93331ac Merge pull request #1 from Objectivitix/Objecti... - Objectivitix
113a20a Implement the math and science questions, added... - Objectivitix
9e6004f fix NameError, add some forgotten imports - Objectivitix
fc80051 Merge pull request #2 from Objectivitix/Objecti... - Objectivitix

night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
ToxicKidz
[python-discord/sir-lancebot] New branch created: fix/wrong\-source\-message
#
ToxicKidz
[python-discord/sir-lancebot] Pull request opened: #740 fix: Send the right help message for \`\.src help\`

Description

Currently, when you do .source help, it sends the description of bot.command.Command, instead of the actual help command. This is because of this line:

https://github.com/python-discord/bot/blob/67934bd582ae5a6fac459c02ba3fa6c17d085d1a/bot/exts/info/source.py#L112

Since our help command is just a regular command, I removed the check to see if the source item is a help command.

Screensh...

#
decorator-factory
[python-discord/bot] New comment on issue #1557: New "\!gethelp" tag to allow the community to quickly respond to a commonly asked question\.

Do you think it is the humanity of us sending it over the bot? If so, your argument is ridiculous.

I don't see how it's ridiculous. People abused !ask mostly because it's very quick and easy to invoke the message.

If a person is lost, you can just say something along the lines of: "Hey, check out #how-to-get-help and claim a help channel." or "You can look at the guide to our channels here: https://pythondiscord.com/pages/guides/pydis-guides/help-channel-guide/"

odd spireBOT
regal archBOT
#
RohanJnr
[python-discord/bot] Issue opened: #1591 Tag: Ternary Operator

Description

A new tag that explains the following:

  • Ternary operator syntax.
  • How is it different from a regular if-else statement.
  • Examples.

A different approach

Since we don't already have a tag for if-else, we could have 1 tag for both if-else and ternary operator, stating their differences and syntax.

#
ToxicKidz
[python-discord/sir-lancebot] New branch created: fix/quiz\-keyerror
#
jb3
[python-discord/site] branch deleted: mod\-policy\-redirect
odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/sir-lancebot] Pull request opened: #741 fix: Handle a KeyError when using \.quiz stop

Relevant Issues

Closes #739

<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->

Description

This PR handles a KeyError in a race condition for .quiz stop. Just a try/except.

Did you:

  • [x] Join the Python Discord Community?
  • [x] Read all the comments in this template?
  • [x] Ensure there is an issue open, or link relevant discord discussions?
  • [x] Read the [contributing guidelines]...
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
#
vcokltfre
[python-discord/sir-lancebot] Issue opened: #742 Make storage of static data consistent

Description

Currently, the vast majority of static data stored in lance is in JSON format, but a few files are YAML or plain text. I propose that this all be turned into one format (JSON, and while I'm there make the JSON formatting consistent across all JSON files) so all static data is stored consistently.

Reasoning

This is a QoL improvement to make the bot more consistent, akin to the spring cleanup but without the huge amount of effort reviewing that took, instead this woul...

#
doublevcodes
[python-discord/sir-lancebot] Issue opened: #743 Issue/PR hyperlink embed upgrade

Description

What this issue solves

This issue aims to solve two things:

  • The format in which issues and pull requests are displayed in the embed that the bot automatically sends.
  • The order in which the issues and pull requests are sent. This wasn't part of the original issue but I figure if this is correctly implemented then this could be fixed too.

Current implementation

Upon mentioning an issue or pull requests in a message such as sir-lancebot#69 or `bot#420...

regal archBOT
#
vcokltfre
[python-discord/metricity] Pull request opened: #4 chore: remove unneeded env variables and command processing

Since the option to opt-in or out of metric collection was removed there are no longer any commands in Metricity, this means the following changes can be made (and have been made in this PR):

  • removing bot_commands_channel config option, it's now redundant
  • removing command_prefix config option, there are no commands, prefix now set to ""
  • adding help_command=None to the constructor, else you just get this:
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Objectivitix
[python-discord/bot] Issue opened: #1592 Tag: dont\-ask\-to\-ask

Description

This will be a tag that briefly explains why "asking if one could ask a question"/"asking if there's anyone who can answer a question" or any other variants decreases one's chance of getting an answer quickly.

Motivation

Currently there is a website that explains this but it is too long and considered dismissive (or sometimes used in a passive-aggressive way), because 1) people would have to navigate between the site and server and 2) the wo...

#
HassanAbouelela
[python-discord/bot] New comment on issue #1592: Tag: dont\-ask\-to\-ask

Duplicate of #1111. That issue also explained why the original ask tag failed, and from what I gather, this suggestion is no different from the original tag or the website. The third reason the website was blocked, the reason you didn't mention, is that people would use it in a dismissive way, and no amount of content changes can fix that. We've discussed this hundreds of times, but every time we come to the same conclusion that something like this just won't work.

#
ChrisLovering
[python-discord/bot] New comment on issue #1592: Tag: dont\-ask\-to\-ask

I'm extremely against this type of tag. Users who this tag would be used for are usually newer users, and if one of their first interactions with our server is our members sending this tag to them with no other context could be very off putting.

I would much prefer our members to just write out a sentence asking them what their question is, as that would be far more welcoming.

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] branch deleted: admin\_endpoint
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/snekbox] New branch created: feat/custom\-eval\-arguments
#
ChrisLovering
[python-discord/site] New branch created: poetry\-and\-python\-3\.9
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: flake\-8\-isn't\-a\-task
#
HassanAbouelela
[python-discord/forms-backend] New branch created: auth\-form
#
HassanAbouelela
[python-discord/bot] branch deleted: flake\-8\-isn't\-a\-task
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] Pull request opened: #90 Empty Authorization Form

The first thing a developer might want to do when setting up the backend is to add a form. Adding a form requires you have your authorization cookie. Getting your authorization cookie requires clicking the discord OAuth button at the bottom of a form. This creates a sort of chicken-or-egg situation.

This PR aims to resolve that by returning a fake form when the backend contains no forms, and the server is running with the PRODUCTION flag set to false.

A better alternative would be to ...

odd spireBOT
odd spireBOT
regal archBOT
#
vcokltfre
[python-discord/bot] New branch created: vcokltfre/tag/async\-await
odd spireBOT
regal archBOT
#
jb3
[python-discord/bot] New comment on issue #1111: Write a new \!ask tag

None of the proposals put forward after the removal of the !ask tag have met the criteria specified in this issue.

We currently document good asking practices in the help channels passively through the available channel embed, which summarises good asking points and also links through to our longer form guide on asking good questions.

<img width="487" alt="Screenshot 2021-05-17 at 19 48 12" src="https://user-i...

regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/bot] New comment on issue #1316: Command to teach how to add to Path

I'm fine with this tag being created, provided the implementation meets the following three criteria:

  1. It's short. Tags are meant to be a quick way to convey information.
  2. It only explains what PATH is, and makes no attempt to explain how to modify it. Reasoning for that above.
  3. Links to an external guide on how to do it. Preferablly, this guide will be up to date, from a reputable site, and have useful resources such as pictures.
#
jb3
[python-discord/site] branch deleted: poetry\-and\-python\-3\.9
#
jb3
[site:main] 8 new commits

e9f6501 Replace pipenv with poetry - ChrisLovering
9c4867d Convert docker file to use 3.9 and poetry - ChrisLovering
b0308d9 Bump lint flow to 3.9 and use poetry - ChrisLovering
e978443 update CODEOWNERS to track poetry files - ChrisLovering
706658b Fix pre-commit, since flake8 isn't a task - ChrisLovering

odd spireBOT
regal archBOT
#
Numerlor
[python-discord/bot] Issue opened: #1595 Cleaning up config\-default\.yml and constants\.py

The default config is rather long (over 500 lines currently) which can be rather dauting for new users as it's not clear which values have to be set for the bot to run. I'm not sure how smartconfig would affect this but it looks like it's still a bit before it can be used within the bots for configuration.

As the first thign all the unused values from both the default config and constants should be removed, for example Channels.reddit or Free.

The formatting of the file is a bit a...

regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] New branch created: remove\-superfluous\-dependencies
#
ChrisLovering
[python-discord/sir-lancebot] Pull request opened: #744 Remove superfluous dependencies

Description

  1. Change AoC helpers to use arrow over pytz
  2. remove bs4 and pytz
  3. update arrow to 1.1.0

pytz is only used in one file in the whole bot, so we can remove the dependency by changing AoC to use arrow.
I chose to use America/Chicago arbitrarily, for no other reason that being the first tz google returned that was in EST.

Beautifulsoup4 is no longer used, it used to be used in AoC code a while ago.

I also updated arrow to 1.1.0 while I was here as the break...

odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/sir-lancebot] Pull request review submitted: #744 Remove superfluous dependencies

I get this error when I run .aoc countdown

05/17/21 18:41:12 - bot.exts.evergreen.error_handler ERROR: Unhandled command error: tzinfo argument must be None or of a tzinfo subclass, not type 'str'
Traceback (most recent call last):
  File "python3.9\lib\site-packages\discord\ext\commands\core.py", line 85, in wrapped 
    ret = await coro(*args, **kwargs)
  File "~\sir-lancebot\bot\exts\christmas\advent_of_code\_cog.py", line 103, in aoc_countdown
    datetime_now = datetime.no...
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Shivansh-007
[python-discord/sir-lancebot] Pull request opened: #745 Add logic for preserving unloaded Cogs\.

This PR makes a redis cache in bot.bot to store all the unloaded extensions, on every unload and load this cache would be edited in the desired manner, either removing it from it or adding it in the cache.

Now, coming to not loading the unloaded commands, on bot startup, just after redis gets connected, the bot would run bot.get_unloaded_extensions and get all the unloaded extensions from the cache. This extensions would be skipped and not loaded, the skipping message would be logged. ...

odd spireBOT
regal archBOT
#
Kronifer
[python-discord/bot] Issue opened: #1597 Delete Discord Server List Server Links

We currently have a ban on advertisement, but I've noticed a few people getting around this by listing their server on disboard and then sending that link instead. I propose we do something similar to how we block discord invites to block these links. (Honestly this might be overkill, but I'm pretty sure advertisement isn't allowed). If given the option, I'd be fine with implementing this.

regal archBOT
#
Akarys42
[python-discord/bot] New comment on issue #1599: Add issue templates to the repo

I don’t think we reached a consensus on this, although here are my thoughts:

I’m not sure if locking ourselves in with a template issue is a good idea. I think having this freedom of structuring it as we want makes us more efficient and we can assume most of our users know how to write a good issue. It feels also quite bureaucratic.

It isn’t mentioned in the issue but a contrario I’d say copying the PR template over would be a good idea. You don’t need much freeway when writing PRs and ...

#
Shivansh-007
[python-discord/sir-lancebot] New comment on pull request #699: Cowsay

i am talking about message character length, not codeblock breaking, here is a simple code:

.int e

from cowsay import char_names, get_output_string

s = list(char_names)
for element in ["dragon", "trex", "stegosaurus", "turkey", "ghostbusters", "turtle"]:s.remove(element)

t = f'{"helloworld"*100}'[:150]

for char in s:
  msgbody = get_output_string(char.lower(), t)
  try:
     await ctx.send(f"\`\`\`\n{msgbody}\n\`\`\`")
  except:
    await ctx.send(f"{char} failed")
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: fix/quiz\-keyerror
regal archBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: fix/wrong\-source\-message
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/site] New branch created: faq\-update
odd spireBOT
regal archBOT
#
jb3
[python-discord/site] branch deleted: faq\-update
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/sir-lancebot] Issue opened: #746 Escape square brackets in reddit post bodies

On mobile (Android, not sure about iOS), an opening bracket without a matching closing bracket causes the Markdown renderer to interpret everything in between as a link title. The mobile renderer ostensibly uses the earliest opening bracket it finds while the desktop renderer uses the latest. It renders as a link because later on, there is link Markdown which is used for the following post's title. This is what that looks like on mobile:

![bild](https://user-images.githubusercontent.com/15...

regal archBOT
regal archBOT
#
jb3
[sir-lancebot:main] 6 new commits

1b21117 Change AoC helpers to use arrow over pytz - ChrisLovering
36738b6 Remove bs4 and pytz update arrow to 1.1.0 - ChrisLovering
38e7ca7 Update references using EST const to use arrow - ChrisLovering
b55df6c Use an actual EST timezone - ChrisLovering
c9114d0 Merge branch 'main' into remove-superfluous-dep... - jb3

#
jb3
[python-discord/sir-lancebot] branch deleted: remove\-superfluous\-dependencies
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Shivansh-007
[python-discord/quackstack] New comment on issue #12: Flesh out the readme

it is, but we're waiting on more things to be ready to do this.

For example, the issue mentions PyPI installation and API usage, things we should probably bring to a more production ready state before we start writing lots of documentation.

i think quackstack is already in a good stable state with a good amount of features, so we can now continue with the other work and not deffer them.

odd spireBOT
regal archBOT
regal archBOT
#
Shivansh-007
[python-discord/quackstack] Pull request opened: #45 \(details\): Rework /details endpoint

This commit does two major things, it adds a query parameter called type which takes in a string object of the detail they are requesting i.e. either man-duck or ducky as of now. If it is None, then it would send all the details of all types, otherwise it would send the details only of the requested type.

This commit also adds details for man-duck generation and accordingly alters the man-duck generation class

Requesting ?type=ducky

![Screenshot from 2021-05-19 19-40-18](https://...

regal archBOT
#
vcokltfre
[python-discord/quackstack] New branch created: vcokltfre/chore/uvicorn\-woohoo
#
ToxicKidz
[python-discord/quackstack] branch deleted: vcokltfre/chore/uvicorn\-woohoo
#
janine9vn
[python-discord/bot] Issue opened: #1601 Code Jam Channels Automatic Creation

Code Jam preparation is in full swing. To reduce the burden on our organizing team, I'd like a command that will automate almost all of the team channels and role creation. Please ping me here or in server if there are any questions about this implementation. This is fairly high priority as I would like it working by June 14th.

The Proposal

A single command !codejam create that will create the channels and roles needed for the code jam.
The csv will be structured like so:
| Team Na...

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
ToxicKidz
[python-discord/quackstack] New branch created: upgrade/python3\.9
#
Numerlor
[python-discord/bot] New comment on issue #1545: Create tag groups

It would also be nice if we could call the tag name without the group name,
example: Both !dpy intents and !intents are valid commands and the bot responds with the same embed.

As I understand it the issue's main purpose is to avoid having all of the tags as direct commands to avoid cluttering and prevent accidental invocations. If both methods are allowed the only real thing it'd bring is the ability to list a certain group and the grouping in the source

#
RohanJnr
[python-discord/bot] New comment on issue #1559: Create a delete and reroll command for OT names

hello @swfarnsworth , can you update the issue description to match the updated implementation?

Updated implementation

Instead of deleting the off topic name after re-rolling, we are gonna mark that name as inactive in the backend (database).
These inactive names will not be considered for future channel names. We are storing these to avoid similar names to be added via the add command as the command checks existence of similar names.

regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: feat/custom\-eval\-arguments
#
MarkKoz
[snekbox:main] 9 new commits

d2ce374 feat: Allow custom arguments to be passed to eval - ToxicKidz
6e98935 chore: Improve documentation for arguments in /... - ToxicKidz
4dfd47a chore: Add tests for py_args and ns_jail args - ToxicKidz
16c00ae chore: Fix some typos - ToxicKidz
3058f37 Add a step to lint-test to show pre-commit logs - ToxicKidz

regal archBOT
#
Objectivitix
[python-discord/sir-lancebot] Issue opened: #747 Question missing a possible correct answer in \`trivia\_quiz\.py\`

Description

The following math question with ID of 228 should have one more possible correct answer in the database.

Steps to Reproduce

  1. input the command .quiz math 30.
  2. wait for it to reach that question (randomly generated, might take a while)
  3. input the answer "centisecond".

It won't recognize it as a correct answer.

Expected Behaviour

It should recognize it as a correct answer.

Actual Behaviour

It doesn't.
![image](https://user-images.githubusercont...

#
vcokltfre
[python-discord/quackstack] branch deleted: upgrade/python3\.9
odd spireBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
ToxicKidz
[python-discord/bot] New branch created: feat/timeit\-command
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
janine9vn
[python-discord/bot] Issue opened: #1603 Small Event Management System

Context

As our server grows we're looking to have more ways for people to engage on the server besides the help channels. One of the ways we do this is with more events. Not all events need to be as large as the code jams though. Some just need a quick announcement, a way for people to subscribe to said announcement, and then a ping when the mini-event starts. I'm hoping to have a system with Python that helps with this.

The Proposal

We previously had a PR to introduce a User Events...

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Kronifer
[python-discord/sir-lancebot] Issue opened: #749 Add a \.reverse command

Description

Add a .reverse command that reverses a submitted string.

Reasoning

Idk, as a joke I guess, could go with fun.py.

Proposed Implementation

Basically, something like this ```py
@commands.command()
async def reverse(self, ctx: commands.Context, *, text: str = "reverse time"):
text = text[::-1]
await ctx.send(f"> {text}")



## Would you like to implement this yourself?


- [x] I'd like to implement this feature myself
- [ ] Anyone can ...
regal archBOT
regal archBOT
regal archBOT
#
ToxicKidz
[python-discord/quackstack] New branch created: feat/details\-endpoint
#
vcokltfre
[python-discord/quackstack] New branch created: vcokltfre/feat/typing
#
Numerlor
[python-discord/bot] Pull request opened: #1606 Improve doc symbol name conflict resolution

Instead of only using the package name as a fallback to resolve the conflicts, it's now used as the first path with the group renames only used for conflicts within packages, this should make it clearer what the symbol may be about when it's shown in the similar names footer.
This also makes the names a bit less dependent on the order the inventories were fetched in.

In addition to using the package name, conflicts within a package where both of the symbols have their group in `FORCE_PREF...

#
vcokltfre
[python-discord/quackstack] New branch created: vcokltfre/chore/reload\-task
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
vcokltfre
[python-discord/quackstack] branch deleted: vcokltfre/chore/reload\-task
regal archBOT
#
ToxicKidz
[python-discord/quackstack] branch deleted: vcokltfre/feat/typing
regal archBOT
#
vcokltfre
[python-discord/bot] New branch created: vcokltfre/chore/modmail\-tag\-update
#
ChrisLovering
[python-discord/bot] branch deleted: vcokltfre/chore/modmail\-tag\-update
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
Numerlor
[python-discord/bot] New comment on pull request #1602: Add the timeit command

I don't think the current output redirecting approach is a good way to do it, as it heavily influences the timing of code that has its runtime on the lower end.
Somewhat hacky to persist the state across runs, but something like this works as the setup code and doesn't affect code that doesn't interact with stdout.

import atexit
import sys
from collections import deque

if not hasattr(sys, "_setup_finished"):
    class Writer(deque):
        def __init__(self):
            sup...
regal archBOT
#
dawnofmidnight
[python-discord/site] Issue opened: #509 Move command line instructions above PyCharm in guides

On various guides on the site such as the one on Installing Project Dependencies, you find the command line instructions at the very bottom of the page, below the (rather space consuming) guides for PyCharm.

On pages filled with images, it can be easy to miss the command line instructions for very important steps. Plus, it's likely that more people will use the command line instructions, si...

regal archBOT
#
swfarnsworth
[python-discord/bot] New branch created: swfarnsworth/one\_help\_channel
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
JagTheFriend
[python-discord/sir-lancebot] Pull request opened: #752 Better ping command

Updated ping command

Made a better ping command.

Before:

After:

Important:

The bot will not reply to message(or to the sender of .ping command)
because I have removed that feature but I can add it back if requested

  • [x] Join the Python Discord Community?
  • [x] Read all the comments in this template?
  • [ ] Ensure there is an issue open, or link relevant discord discussions?
  • [x] Read the [contributing guidelines]...
odd spireBOT
regal archBOT
#
bast0006
[python-discord/bot] Pull request opened: #1609 Fix infraction rescheduler breaking with more than 100 in flight reactions

Make sure to only fetch infractions to reschedule by filtering by type and permanent status. We don't reschedule permanents as they will never be automatically expired, so they're a waste and clog to filter out manually. There is a PR for site to add the requisite filters (types and permanent).
We also only reschedule the soonest-expiring infractions, waiting until we've processed all of them before fetching the next batch by ordering them by expiration time.

This requires python-di...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Objectivitix
[python-discord/sir-lancebot] Issue opened: #753 Add "cs" and "python" categories to the \`\.quiz\` command

Description

This is an extension of PR #726. It was planned to also add these two categories in that PR, but we didn't get enough quiz entries on "cs" and "python" and thus it couldn't be implemented. But now, thanks to lak, we got 30 questions each, which is enough!

Reasoning

I believe this would be beneficial to our community because they're programming, computer science and Python questions. These questions are very related to our server.

Proposed Implementation

Since all...

regal archBOT
regal archBOT
#
ToxicKidz
[python-discord/site] New branch created: update/docs\-pipenv\-to\-poetry
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
NovaTohsaka
[python-discord/sir-lancebot] Issue opened: #754 ascii command

Description

A command to convert the text args (passed to the command by a member) into an ascii art.

Reasoning

Well, just a fun command! People may find it fun using.

Proposed Implementation

Implementation will require an installation of third-party lib pyfiglet.

Would you like to implement this yourself?

  • [x] I'd like to implement this feature myself
  • [ ] Anyone can implement this feature
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
JagTheFriend
[python-discord/sir-lancebot] New comment on pull request #752: Better ping command

Hi, is there any context for this? Perhaps a conversation on discord or an issue on the repository?

I just made the ping command of the bot more stylish
but to be honest, there isn't really a need for it.

As a side note, please don’t use the CI as your only linting. Run the lint locally, and maybe setup precommit to do it for you.

I used the "format document" option you get within Vscode to format the file

regal archBOT
#
lemonsaurus
[python-discord/site] New branch created: lemon/smarter\-resources/tag\_filtering
#
NovaTohsaka
[python-discord/sir-lancebot] Issue opened: #755 new command: evergreen\_images

Description

A command which fetches (evergreen) forest images using unsplash API like earth_photos.

Reasoning

I and (many of us) find greenery "relaxing", thus a command for providing such photos will be good.

Proposed Implementation

Implementation will (mostly) be same as earth_photos, frontend will have all the appropriate details like it as well.

Would you like to implement this yourself?

  • [x] I'd like to implement this feature myself
  • [ ] Anyon...
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/site] New branch created: auth\-with\-github\-api
odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/bot] New branch created: fix/float\-tag\-typo
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: fix/float\-tag\-typo
oak estuaryBOT
odd spireBOT
oak estuaryBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Akarys42
[python-discord/site] branch deleted: auth\-with\-github\-api
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
gustavwilliam
[python-discord/bot] New branch created: higher\-duck\-limit
#
mbaruh
[python-discord/bot] branch deleted: higher\-duck\-limit
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New comment on pull request #1608: Enforce one help channel per user

Have played around with it a bit and it seems to work fine. My only worry is that removing the timed cooldown creates a potential for trolls to quickly open and close their channel to mess with the system, I think that was one of the reasons for having a cooldown. On the other hand, being able to simplify the system is nice, and the amount of harm somebody could cause is likely pretty low and we would notice/deal with it pretty quickly.

To clarify, that would still involve limiting the hel...

night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
swfarnsworth
[python-discord/bot] New comment on pull request #1608: Enforce one help channel per user

@wookie184 I don't think that concern was raised during the meeting where we green-lit this change.

I'm hesitant to have the "channel is closed and five minutes" cooldown as it re-introduces some of the complexity that this seeks to eliminate. I have also at times closed help sessions that have been overrun by trolls and instructed the claimant to open a new session, and I wouldn't want to lose the option to do that without penalizing the claimant.

#
Objectivitix
[python-discord/sir-lancebot] Pull request opened: #756 Add "cs" and "python" categories to the \`\.quiz\` command \(and more in desc\)

Relevant Issues

Closes issue #753 and #747 , extends PR #726

Description

It was planned to incorporate these two categories in the same PR, however I didn't get enough quiz entries for them. But now, thanks to lak, I have enough! So in total, 60 new questions are added. I also changed the starting embed a bit to make it look nicer, made the variation tolerance stricter, and added some missing answers specified in 747 to the math & science questions.

Did you:

  • [x] Join the [...
regal archBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: update\-pixels\-regex
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/bot] branch deleted: update\-pixels\-regex
oak estuaryBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New review comment on pull request #1570: Add a persistent scheduler

Hopefully it's not too much of an issue
The logger name will be really long since __name__ is full qualified and two fully qualified names are being concatenated.

A way to around it is to add an optional namespace argument to the scheduler init, although I'm not sure if that's desirable.

I'd be okay with it if it was really necessary, but I don't think it's important to keep the names the same. In fact, it may be a bad idea to use __name__ for the cache, since it will cause cac...

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] New branch created: block\-some\-status\-dog\-codes
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
Den4200
[python-discord/sir-lancebot] branch deleted: block\-some\-status\-dog\-codes
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
#
ToxicKidz
[python-discord/sir-lancebot] Pull request review submitted: #756 Add "cs" and "python" categories to the \`\.quiz\` command \(and more in desc\)

Hey, you should test before you push, because I get this error:

Traceback (most recent call last):
   File "/usr/local/lib/python3.9/site-packages/discord/ext/commands/bot.py", line 618, in _load_from_module_spec
     setup(self)
   File "/bot/bot/exts/evergreen/trivia_quiz.py", line 593, in setup
     bot.add_cog(TriviaQuiz(bot))
   File "/bot/bot/exts/evergreen/trivia_quiz.py", line 202, in __init__
     self.questions = self.load_questions()
   File "/bot/bot/exts/evergreen/...
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
Akarys42
[python-discord/bot] New branch created: reviews\-revert\-order
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: reviews\-revert\-order
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Numerlor
[python-discord/bot] New comment on pull request #1539: Move static content site commands to tags

I don't think we should wait for tag grouping before moving along with this. It wouldn't change anything about the current functionality, we can group them later on when we merge the other PR.

The PR is stalled because the group PR should also introduce the ability to pass in embed arguments through the tag file to be used as titles, after the issue is approved. I suppose it could also be done in a PR separate from the grouping functionality but I don't think it'd be worth complicating i...

#
Akarys42
[python-discord/site] branch deleted: update/docs\-pipenv\-to\-poetry
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New review comment on pull request #1570: Add a persistent scheduler

Originally, I meant that the user of the scheduler would call reschedule_all_tasks and keep track of the task itself, if needed. The responsibility of ensuring rescheduling is finished before attempting to delete something would be delegated to the user of the class.

However, on second thought, having to manage the task externally sounds inconvenient. I'd feel fine with passing an event loop to the scheduler so it can schedule task. As for waiting for the guild to be available, that coul...

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
bast0006
[python-discord/site] New comment on pull request #510: Add new infraction filters for the infraction rescheduler

From an initial glance, should we have to worry about both the type and types parameters being used together?

I'm not sure, but I don't think so. At the moment, doing so behaves mostly as someone would expect. If you specify both type and types, and type is in types, then it will return only type results. If you specify a type that is not in your specified types you will get no results.

We could add an error condition for mixing of parameters like that, but if we do...

regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
kosayoda
[python-discord/sir-lancebot] Pull request review submitted: #745 Add logic for preserving unloaded Cogs\.

From a cursory look at the PR:

I don't think this is a suitable way to store unloaded cogs in the cache. Currently, you manually serialize cog names by joining them with a pipe character, then setting it to a single key unloaded in the cache.

This is redundant when we have a cache to work with: Simply set the cog name as the cache key. That way, there is no need to even deserialize the cache into a dictionary on every time you want to modify the cache: Simply set and get the cog names...

night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
ashen phoenix
#

I will now obliterate this issue

#

That's almost otn worthy

#

But I can't find a good one

regal archBOT
#
Xithrius
[bot:main] 5 new commits

734e4ef Prefer using the package name as a prefix when ... - Numerlor
b6ccd03 Prioritize symbols depending on their group's p... - Numerlor
f00fe17 Add the doc group to FORCE_PREFIX_GROUPS - Numerlor
9d5567d Merge branch 'main' into doc-force-prefix-priority - Xithrius
58b1d27 Merge pull request #1606 from Numerlor/doc-forc... - Xithrius

odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New comment on pull request #1570: Add a persistent scheduler

What do you think about the last point in the original post? It will allow for future flexibility, but it's not currently needed and complicates the class a bit. Then again, if we do need it, then adding it later will be a headache that will invalidate the caches.

I'm not sure, but I have a feeling this would be better saved for another PR, especially since this one is pretty close to being done already. It doesn't sound like a terrible idea - I did store JSON once in redis for s...

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
OculusMode
[python-discord/sir-lancebot] Pull request opened: #758 Added RPS command

Closes #613

Description

Added Rock paper scissors command which allows players to play it with bot.

Format

.rps

where option_name can be any of r, p, s, rock, scissor, paper in any case.

Did you:

regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/bot] branch deleted: swfarnsworth/one\_help\_channel
oak estuaryBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
OculusMode
[python-discord/sir-lancebot] New comment on pull request #758: Added RPS command

@Shivansh-007 Thanks a lot for reviewing I have made changes you requested and resolved all the flake8 errors the PR faced.
Just a small review of changes done:

  • Merged Game class and cog class
  • Added Doc strings in all the methods
  • Capitalised static variables
  • Added missing types
  • Removed dead code
  • Changed @commands.groups to @commands.command
  • Changed variable name to appropriate meaningful name(i.e. arg => move)
  • Checked linting on local machine and tested bot in my se...