[python-discord/site] branch deleted: jb3/resources/discord.py-masterclass
#dev-log
1 messages ยท Page 26 of 1
GitHub Actions run 10517123859 succeeded.
You know what I have to say, G.O.A.T
MY HERO!!!!!!
GitHub Actions run 10518125034 succeeded.
GitHub Actions run 10520237545 succeeded.
GitHub Actions run 10523978888 failed.
GitHub Actions run 10525447328 succeeded.
Last I heard, Bella can't wait for this feature either.
@ChrisLovering Please fix conflicts and I'll merge
2d23167 fix(deps): update dependency sentry-sdk to v2.1... - renovate[bot]
[python-discord/king-arthur] branch deleted: renovate/sentry-sdk-2.x
Why do we use squash merge in this repository?
I think we should apply similar strategy that we apply to the infra repository.
[python-discord/king-arthur] Checks Failed on PR: #244 chore(deps): update dependency ruff to v0.6.2
GitHub Actions run 10530271936 failed.
GitHub Actions run 10530277580 succeeded.
GitHub Actions run 10530277668 succeeded.
GitHub Actions run 10530277776 succeeded.
GitHub Actions run 10530283102 succeeded.
GitHub Actions run 10530785991 succeeded.
GitHub Actions run 10530786410 succeeded.
Connected!
GitHub Actions run 10530770581 succeeded.
Relevant Issues
add emoji
Description
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 and agree to the contributing guidelines?
GitHub Actions run 10535834196 succeeded.
GitHub Actions run 10537440431 succeeded.
GitHub Actions run 10537458255 succeeded.
GitHub Actions run 10537484646 succeeded.
Yeah that works, I just want to avoid args and kwargs generally but yeah
Why the hate against args and kwargs?
I think that we should use my solution but I'm biased. @ChrisLovering what do you think?
Yeah that works, I just want to avoid args and kwargs generally but yeah
Why the hate against args and kwargs?
I think that we should use my solution but I'm biased. @ChrisLovering what do you think?
Because the signature doesn't tell you anything about the function anymore, and the api loses meaning. The only way to find out is to read inside the wrapped method
Because the signature doesn't tell you anything about the function anymore, and the api loses meaning. The only way to find out is to read inside the wrapped method
But we don't implement the function ourselves, we just copy the
signature. So we cannot be an authorative source on what the meaning of
the API is supposed ot be, because as soon as upstream changes it, it's
going to break.
I would rather have a single function that loses meaning (and you can
document it as "same method as ...
Hmmmm, not sure about that.
You're exposing a new public interface for your clients, so the maintenance is part of the job. What you're trying to do, wrapping it, adding or removing stuff doesn't matter. I don't know, it just doesn't sit well with me for some reason.
Bella, The Hague,
apparently the LinePaginator is defined in the exact same module.
Why don't we just update the existing class to do this ?
[python-discord/king-arthur] New branch created: add-email-variables
dc53e37 Document e-mail variables - jchristgit
Initial work for #262.
GitHub Actions run 10549261951 succeeded.
:white_check_mark: Successfully fast-forwarded commits from python-discord:add-email-variables onto main
[python-discord/king-arthur] branch deleted: add-email-variables
GitHub Actions run 10550731377 succeeded.
GitHub Actions run 10558503187 failed.
GitHub Actions run 10558797953 succeeded.
@jchristgit How about we do it like this
from functools import partial
class LinePaginator:
@classmethod
def idk_what_to_call_this_but_it_returns_a_new_paginator_with_emojis(cls, emojis):
paginator = type(cls.__name__, cls.__bases__, dict(cls.__dict__))
paginator.paginate = partial(paginator.paginate, emojis=emojis)
return paginator
I think this could work
Amendment type
Content Update
Content type
Other
Please provide the location of the content
https://github.com/python-discord/bot/blob/main/bot/constants.py#L766
What amendment do you think should be made to this content, and why?
Have seen people in the Discord try to use the bot, then get greeted with error messages such as:
Please don't do that.
You have to stop.
In the future, don't do that.
You blew it.
You're bad at computers.
They're pretty fu...
Transferred to bot as the project where the error messages are most prevalent, we likely should implement this on all projects but bot is the project that users, particularly new users, interact with the most and as such is the most important project for this issue to be implemented on.
What feature are you proposing?
Send a DM to members when the bot detects a certain keyword in the incoming messages.
What is your justification for the proposed feature?
This was something that was suggested before, but I was against it previously for the following reasons:
- It is easy to abuse this system to stalk/follow people. It's not something we should enable.
- It is possible to create alerts which fire on way too many messages (for instance just watching the letter `e...
Transferred issue to bot pending implementation.
We likely don't need to worry about ratelimits, abandoned accounts get purged by Discord after a period of time and as mentioned our ratelimits are high enough for us not to worry.
We should not overengineer this and instead should add the ability to observe the feature and address ratelimits down the line should they become a problem (which is highly unlikely)
This idea has been bounced around for a while now. A few people have mentioned they'd like it and there's not really been a reason against it (to my knowledge). For the concern about confusion, a description of the functionality can be shown before the user hits run.
!raw [#mailing-lists message](/guild/267624335836053506/channel/704372456592506880/) returns something containing:
See the full release notes for all of the details:
https://python-
oracledb.readthedocs.io/en/latest/release\_notes.html#oracledb-2-4-1-august-2024
Note the backslash escape.
Discord interprets that as (backslash is turned into slash... for some reason), which is a broken link
Yes, that could probably work. But as a casual reader I now need to remember what type did and why it did what it does.
I have no idea what we are doing here, because it's defined in the same module.
Isn't this more straightforward?
def paginate_with_default_emojis(*args, **kwargs):
kwargs.setdefault("pagination_emojis", ...)
return LinePaginator.paginate(*args, **kwargs)
GitHub Actions run 10579425899 succeeded.
Connected!
GitHub Actions run 10593845560 succeeded.
[python-discord/snekbox] New branch created: bump-versions
This bumps Pytohn version within each container to 3.12.5 and bumps eval-deps to latest.
import spacy
import requests
from bs4 import BeautifulSoup
Load NLP model
nlp = spacy.load('en_core_web_sm')
Example job description URL
job_url = 'https://example.com/job-posting'
Scrape job description
response = requests.get(job_url)
soup = BeautifulSoup(response.text, 'html.parser')
job_description = soup.get_text()
Analyze job description
doc = nlp(job_description)
skills = [ent.text for ent in doc.ents if ent.label_ == 'SKILL']
Generate a cover letter based on the job des...
Thanks for opening this issue.
However, the content doesn't seem to put things that need to be addressed under the spotlight.
If you think there are things that need improvement with our bot, then i invite you to open a new issue with more details.
You can refer to the currently open issues for inspiration.
GitHub Actions run 10618565688 succeeded.
Doc item doc_item.symbol_id='discord.Sticker.pack_id' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
GitHub Actions run 10627528803 succeeded.
d0e85bd Bump pip-licenses from 4.5.1 to 5.0.0 (#1583) - dependabot[bot]
Connected!
GitHub Actions run 10629585640 succeeded.
Connected!
GitHub Actions run 10629618792 failed.
GitHub Actions run 10629616224 succeeded.
GitHub Actions run 10629598621 succeeded.
GitHub Actions run 10629616969 succeeded.
GitHub Actions run 10629623596 succeeded.
c8a5e24 Bump pip-licenses from 4.5.1 to 5.0.0 (#3134) - dependabot[bot]
5a7f150 Bump pre-commit from 3.7.1 to 3.8.0 (#1587) - dependabot[bot]
GitHub Actions run 10629639183 succeeded.
GitHub Actions run 10629644168 succeeded.
GitHub Actions run 10629643960 failed.
Connected!
Connected!
GitHub Actions run 10629651295 succeeded.
GitHub Actions run 10629653154 succeeded.
GitHub Actions run 10629680856 succeeded.
30e0d09 Bump sentry-sdk from 2.10.0 to 2.13.0 (#3146) - dependabot[bot]
Connected!
GitHub Actions run 10629708762 failed.
Connected!
GitHub Actions run 10629709713 succeeded.
GitHub Actions run 10629699498 succeeded.
GitHub Actions run 10629739143 succeeded.
Connected!
GitHub Actions run 10629746906 was cancelled.
GitHub Actions run 10629784972 failed.
Connected!
GitHub Actions run 10629770768 succeeded.
GitHub Actions run 10630801063 succeeded.
GitHub Actions run 10630810759 succeeded.
GitHub Actions run 10632511033 succeeded.
GitHub Actions run 10632516512 succeeded.
GitHub Actions run 10632522806 succeeded.
[python-discord/site] New branch created: remove-discrim-from-bot-logs-view
57616e0 Remove disciminator from bot logs view - jchristgit
When a user does not have a discriminator, do not display it anymore.
Behaviour for users with discriminators (for historic infractions is
unchanged).
GitHub Actions run 10635386435 succeeded.
GitHub Actions run 10635441230 failed.
GitHub Actions run 10635464365 succeeded.
[site] Branch remove-discrim-from-bot-logs-view was force-pushed to `24a7854`
GitHub Actions run 10635485343 succeeded.
GitHub Actions run 10636176225 succeeded.
GitHub Actions run 10636558586 succeeded.
Hello @jb3, what's the status with this PR?
i forgor ๐
GitHub Actions run 10647563005 succeeded.
GitHub Actions run 10647563676 succeeded.
Connected!
GitHub Actions run 10647570724 succeeded.
GitHub Actions run 10647583120 succeeded.
Connected!
GitHub Actions run 10647573737 succeeded.
94a07c3 Bump rapidfuzz from 3.9.4 to 3.9.6 (#1598) - dependabot[bot]
Connected!
GitHub Actions run 10647604415 succeeded.
Connected!
GitHub Actions run 10647617393 succeeded.
Connected!
GitHub Actions run 10647619901 succeeded.
Connected!
GitHub Actions run 10647620216 was cancelled.
Connected!
GitHub Actions run 10647626644 succeeded.
Connected!
GitHub Actions run 10647632639 was cancelled.
GitHub Actions run 10647649410 failed.
Connected!
GitHub Actions run 10647643772 succeeded.
GitHub Actions run 10653005266 succeeded.
Do we still want this?
At present I can't think of commands we would run through this, except perhaps
fail2ban. But so far that has worked, plus the affected team members have surely
by now switched to password managers and no longer run into those issues.
Fail2ban is the only thing that comes to mind right now also.
Potentially fetching info like things from "doveadm who" could be nice but isn't
a big deal.
I think it's fine to keep open but not prioritize implementation, it doesn't
hurt us.
We don't, it appears as the default because it was the last used action and is
the one that's enabled. Nothing here is different to the infra repository.
There is a policy file for ff-bot on main so everything is already setup to be
able to /merge in the same way you would on infra.
PRs #243, #257, #261 and #264 were all merged using the fast-forward bot.
I've disabled all methods but squash merging now because:
- Merge commits are evil and obviously we have a policy of not using them on...
GitHub Actions run 10655257567 succeeded.
GitHub Actions run 10661549358 succeeded.
GitHub Actions run 10661552588 succeeded.
GitHub Actions run 10661557514 succeeded.
GitHub Actions run 10667132565 succeeded.
GitHub Actions run 10667138233 succeeded.
GitHub Actions run 10667146946 succeeded.
GitHub Actions run 10667155369 succeeded.
GitHub Actions run 10667163998 succeeded.
8a1c461 Fix incorrect order of namespace/deployment arg... - jchristgit
[python-discord/king-arthur] New branch created: fix-bug
GitHub Actions run 10670667137 succeeded.
[python-discord/bot] Pull request opened: #3165 General refactor of bot.py, log.py and pagination.py
There are a few small changes that I made thought these files.
For the log file:
Broken down the setup function to be more understandable and readable.
Improved name of the _set_trace_loggers function and it now uses a single underscore, which is more conventional for internal use.
Added an early return in the _set_trace_loggers function. Now a early return is called if level_filter is false, making it more readable.
This goes for all the files, added consistent spacing and ind...
We appreciate your effort into this PR, but there are some issues with the changes you've proposed here:
Broken down the setup function to be more understandable and readable.
I don't see the benefit of this, especially considering the docstring of setup() already indicates that it's meant to "set up loggers". If you find readability to be an issue in the existing code (IMO it's fine as it is), labeling each section of the function with a comment would be more reasonable.
...
Thanks for the changes, couple of small comments.
I thought ruff would complain once you have execution paths that do not return none when you have at least one that does. surprising.
We can make use of the StartupError exception here and raise it instead of a bare exception.
You don't really need the variable if it's only being used to pass it as argument to the pagination_emojis param.
I guess you could leave the comment.
[python-discord/bot] Checks Failed on PR: #3165 General refactor of bot.py, log.py and pagination.py
GitHub Actions run 10677813452 failed.
GitHub Actions run 10681123360 succeeded.
All paths in that function return None, just implicitly (one from a bare return statement, one from having no explicit return statement)
All paths in that function return None, just implicitly
Yes that i know, it's just that i thought ruff would either impose that all returns be implicit, and when you have at leat one explicit, the rest need to become as such as well
[python-discord/forms-backend] Checks Successful on PR: #311 Bump cryptography from 42.0.8 to 43.0.1
GitHub Actions run 10692872209 succeeded.
GitHub Actions run 10692924089 succeeded.
GitHub Actions run 10693898682 succeeded.
GitHub Actions run 10693888582 succeeded.
GitHub Actions run 10695527203 succeeded.
efb6956 Fix incorrect order of namespace/deployment arg... - jchristgit
[python-discord/king-arthur] branch deleted: fix-bug
GitHub Actions run 10702882748 succeeded.
GitHub Actions run 10727474532 succeeded.
GitHub Actions run 10727474156 succeeded.
GitHub Actions run 10736481926 failed.
GitHub Actions run 10738045945 succeeded.
None of those changes affect us
c514110 fix(deps): update dependency pydantic to v2.9.0... - renovate[bot]
[python-discord/king-arthur] branch deleted: renovate/pydantic-2.x
674bce5 chore(deps): update dependency ruff to v0.6.4 (... - renovate[bot]
[python-discord/king-arthur] branch deleted: renovate/ruff-0.x
GitHub Actions run 10738407600 was cancelled.
[python-discord/king-arthur] Checks Failed on PR: #266 chore(deps): update dependency ruff to v0.6.4
GitHub Actions run 10738414194 failed.
GitHub Actions run 10738416868 was cancelled.
GitHub Actions run 10738419966 succeeded.
GitHub Actions run 10738463416 succeeded.
GitHub Actions run 10759913672 succeeded.
GitHub Actions run 10772806555 succeeded.
GitHub Actions run 10777072385 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.13.0
GitHub Actions run 10777416942 succeeded.
GitHub Actions run 10790977425 succeeded.
[python-discord/metricity] New branch created: jb3/features/content-hash
[python-discord/metricity] Pull request opened: #187 Store a MD5 hash of content for stored messages
Storing an MD5 hash of content allows us to perform tasks related to the content of identical messages without compromising our promise of privacy and not storing actual user generated content.
With this field, we can:
- Look for instances of same-channel spam
- Look for instances of cross-channel spam
- Factor these into when we make a voice verification decision (i.e. if there is reason to believe a user has spammed, hold off from verifying until a Moderator has reviewed the case)
...
Just one comment on top of the ruff required changes
You'll need to either define a default value here, or add a migration so that existing rows meet the NOT NULL constraint added by setting nullable=False here
GitHub Actions run 10796323346 succeeded.
1741980 fix(deps): update dependency pydantic-settings ... - renovate[bot]
[python-discord/king-arthur] branch deleted: renovate/pydantic-settings-2.x
[python-discord/metricity] branch deleted: jb3/features/content-hash
GitHub Actions run 10796406317 succeeded.
GitHub Actions run 10796408051 succeeded.
GitHub Actions run 10796411025 succeeded.
[python-discord/metricity] New branch created: chris/2.8.0
99093a9 Bump pydis-core & sqlalchemy to latest - ChrisLovering
6cfafbb Bump dev dependencies to latest - ChrisLovering
823b59f Disable ruff preview mode - ChrisLovering
544f8d3 Change default ruff output format to concise - ChrisLovering
86fa572 Disable poetry packing mode since this is not a... - ChrisLovering
99093a9 Bump pydis-core & sqlalchemy to latest - ChrisLovering
6cfafbb Bump dev dependencies to latest - ChrisLovering
823b59f Disable ruff preview mode - ChrisLovering
544f8d3 Change default ruff output format to concise - ChrisLovering
86fa572 Disable poetry packing mode since this is not a... - ChrisLovering
[python-discord/metricity] branch deleted: chris/2.8.0
[python-discord/metricity] New tag created: v2.8.0
pin this issue and flag p1
GitHub Actions run 10808138326 succeeded.
GitHub Actions run 10809583512 succeeded.
deefdac Disable ruff preview mode - ChrisLovering
de24020 Bump dev dependencies to latest - ChrisLovering
5500c5e Ignore symbols starting with __ in the linkcode... - ChrisLovering
0bd32af Update Discord invite regex to also match backs... - ChrisLovering
41c7469 Bump version to v11.4.0 - ChrisLovering
[python-discord/bot-core] New branch created: chris/11.4.0
deefdac Disable ruff preview mode - ChrisLovering
de24020 Bump dev dependencies to latest - ChrisLovering
5500c5e Ignore symbols starting with __ in the linkcode... - ChrisLovering
0bd32af Update Discord invite regex to also match backs... - ChrisLovering
b892c86 Remove an unneeded noqa - ChrisLovering
[python-discord/bot-core] branch deleted: chris/11.4.0
[python-discord/bot-core] New tag created: v11.4.0
c6d2f0e Deploying to docs from @ python-discord/bot-cor... - ChrisLovering
Connected!
GitHub Actions run 10817623962 succeeded.
GitHub Actions run 10817599518 succeeded.
GitHub Actions run 10819168784 succeeded.
GitHub Actions run 10829275544 succeeded.
Connected!
GitHub Actions run 10830316323 succeeded.
GitHub Actions run 10830449631 succeeded.
GitHub Actions run 10830463992 succeeded.
Connected!
GitHub Actions run 10831725117 succeeded.
GitHub Actions run 10831764874 succeeded.
Connected!
GitHub Actions run 10831812552 succeeded.
GitHub Actions run 10831817425 succeeded.
GitHub Actions run 10849059841 succeeded.
GitHub Actions run 10849072561 succeeded.
weird, seems fine on my phone, and on desktop
which is odd, since we have the same phone
It's time to change those sunglasses
GitHub Actions run 10852496629 succeeded.
According to the markdown spec, lines separated by a single newline should be joined with a space.
Discord embeds doesn't seem to respect this, as demonstrated by the !traceback tag:
We could resolve this by formatting paragraphs on a single line, but I think joi...
Relevant Issues
https://discordapp.com/channels/267624335836053506/635950537262759947/1284217021634187341
Description
This is a small change, the cleaned content was not used in the uwu command.
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 and agree to the [contributing guidelines](https://p...
GitHub Actions run 10854552568 succeeded.
GitHub Actions run 10854593252 succeeded.
Connected!
GitHub Actions run 10854780949 succeeded.
6ad510d fix(deps): update dependency pydantic to v2.9.1 - renovate[bot]
b359a72 fix(deps): update dependency pydantic-settings ... - renovate[bot]
GitHub Actions run 10861395042 was cancelled.
GitHub Actions run 10861396934 failed.
GitHub Actions run 10861400282 succeeded.
GitHub Actions run 10861402798 succeeded.
GitHub Actions run 10861403035 was cancelled.
GitHub Actions run 10861409582 succeeded.
GitHub Actions run 10861410104 succeeded.
GitHub Actions run 10861410844 succeeded.
GitHub Actions run 10861410741 succeeded.
4bff7ae chore(deps): update dependency ruff to v0.6.5 - renovate[bot]
:white_check_mark: Successfully fast-forwarded commits from python-discord:renovate/ruff-0.x onto main
[python-discord/king-arthur] branch deleted: renovate/ruff-0.x
GitHub Actions run 10863123005 succeeded.
GitHub Actions run 10863126600 succeeded.
GitHub Actions run 10863129910 succeeded.
GitHub Actions run 10863685791 succeeded.
GitHub Actions run 10866983219 succeeded.
There are three ways a help channel can be closed:
- A user closes it either via running
!closeor by using the discord right-click context menu - It gets closed by our bot after 1 hour of inactivity
- A moderator closes it via
!close
The Pitch
I think it would be helpful to have different closing messages based on the method of how it was closed. The two main messages I think would be useful are:
- A user closes it: Send a short and sweet message that says the user (or a mode...
GitHub Actions run 10868743408 succeeded.
GitHub Actions run 10868743547 succeeded.
GitHub Actions run 10868744880 succeeded.
GitHub Actions run 10881993176 failed.
GitHub Actions run 10908167746 succeeded.
GitHub Actions run 10928027477 succeeded.
GitHub Actions run 10953456391 succeeded.
c982f11 Limit message cache to 100 messages - ChrisLovering
[python-discord/king-arthur] New branch created: limit-message-cache
GitHub Actions run 10956484342 succeeded.
89f5f6a Limit message cache to 100 messages (#279) - ChrisLovering
[python-discord/king-arthur] branch deleted: limit-message-cache
GitHub Actions run 10957281353 succeeded.
GitHub Actions run 10975209831 succeeded.
GitHub Actions run 10983491174 succeeded.
GitHub Actions run 10989947407 succeeded.
GitHub Actions run 10992016849 failed.
GitHub Actions run 10994626603 succeeded.
GitHub Actions run 11011908248 succeeded.
GitHub Actions run 11011915192 succeeded.
GitHub Actions run 11052979742 succeeded.
Doc item doc_item.symbol_id='grammar-token-python-grammar-starred_item' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
GitHub Actions run 11068782896 failed.
GitHub Actions run 11075677285 succeeded.
GitHub Actions run 11078734513 succeeded.
GitHub Actions run 11087525468 succeeded.
Doc item doc_item.symbol_id='class.__bases__' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
Connected!
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.15.0
GitHub Actions run 11128474923 succeeded.
GitHub Actions run 11131608670 succeeded.
GitHub Actions run 11141819005 succeeded.
GitHub Actions run 11142119506 succeeded.
GitHub Actions run 11143971965 succeeded.
[python-discord/snekbox] New branch created: py-3.13-rc
fdef8fd Move pyenv clone to builder-py-base image - ChrisLovering
91dd475 Add Python 3.12.0rc3 support - ChrisLovering
3e76dfa Bump eval deps to latest - ChrisLovering
26d30a4 Remove obsolete version specifier in docker com... - ChrisLovering
4d05016 Update version specifier tags in eval-deps - ChrisLovering
This adds support for Python 3.13 to snekbox.
Actually calling this isn't possible until https://github.com/python-discord/snekbox/pull/196 is merged, which I will be looking into now.
8952cac Update nsjail_args and py_args to store_const i... - ChrisLovering
Did you intended on leaving this in?
Do you have a sample of what the help output is after this change?
Yea, the reason I added the default to the help directly is because it didn't get added automatically.
โฏ python3 -m snekbox --help
usage: snekbox [-h] code [nsjail_args ...] [--- py_args ...]
positional arguments:
code the Python code to evaluate
nsjail_args override configured NsJail options (default: [])
py_args arguments to pass to the Python process (default: ['-c'])
options:
-h, --help show this help message and exit
Had to do something slightly idfferent so that hte default cna be used both by calling python3 directly, and when calling the API https://github.com/python-discord/snekbox/pull/196/commits/66c6d27335a5d1fc23bd0889d299cb79f3418219
changed this func quite a bit since then, so this is no longer relevant
Now ready for (re)review
3e3dfa0 Update invalid binary paths test to cover new e... - ChrisLovering
76ba512 Add a note about multi-verison support to the R... - ChrisLovering
Connected!
GitHub Actions run 11170370851 succeeded.
[python-discord/bot] New branch created: Snekbox-multi-version
995877a Support both Python 3.12 and 3.13 in eval - ChrisLovering
This is blocked until both snekbox PRs are merged.
https://github.com/python-discord/snekbox/pull/196
https://github.com/python-discord/snekbox/pull/217
GitHub Actions run 11170593494 failed.
GitHub Actions run 11170645280 succeeded.
Thanks - that still looks helpful
fdef8fd Move pyenv clone to builder-py-base image - ChrisLovering
91dd475 Add Python 3.12.0rc3 support - ChrisLovering
3e76dfa Bump eval deps to latest - ChrisLovering
26d30a4 Remove obsolete version specifier in docker com... - ChrisLovering
4d05016 Update version specifier tags in eval-deps - ChrisLovering
[python-discord/snekbox] branch deleted: py-3.13-rc
GitHub Actions run 11175081866 succeeded.
GitHub Actions run 11178388383 succeeded.
GitHub Actions run 11182636936 succeeded.
GitHub Actions run 11183727038 succeeded.
By default, the binary that runs within nsjail is the binary specified by `DEFAULT_BINARY_PATH` at the top of [`nsjail.py`]. This can be overridden by specifying `binary_path` in the request body of calls to `POST /eval` or by setting the `binary_path` kwarg if calling `NSJail.python3()` directly.
("/abc/def", "test non-existent files are not run", "binary_path does not exist"),
("/snekbin", "test directories are not run", "binary_path is not a file"),
Maybe "executable_path" is a better name than "binary_path"? I'm also okay with the latter though.
"test non-executable files are not run",
GitHub Actions run 11191596696 succeeded.
7afaffb Correct spelling errors - ChrisLovering
Yeah I agree, makes it clearer what is supported here, as it's not limitted to just binaries. https://github.com/python-discord/snekbox/pull/196/commits/edcaed6ad90f71ff61f6ceb7194de611ceeec592
GitHub Actions run 11194756300 succeeded.
GitHub Actions run 11195899021 succeeded.
ed81df4 Allow specifying a binary path when calling NsJ... - ChrisLovering
6cc4dba Add tests for multi-version support - ChrisLovering
c24785d Allow setting a binary_path when calling /eval - ChrisLovering
0c9b234 Also split on hyphens in build python script - ChrisLovering
d9b8ab3 Add additional tests to ensure invalid binary p... - ChrisLovering
[python-discord/snekbox] branch deleted: multi-version-api
GitHub Actions run 11196277739 succeeded.
Nit: this could use string formatting instead of if/else.
Have not tested locally, but the code looks good.
Yeah, not sure why I made it so complicated. https://github.com/python-discord/bot/compare/81efc67838f86cb8fb74f66a75b3e84f27d194d6..60e29b1a15a2f026a71114fed7bb2f2f93b2548d
This also showed a test I overlooked, which I also corrected
GitHub Actions run 11201235775 succeeded.
GitHub Actions run 11213577788 failed.
Connected!
GitHub Actions run 11219953497 succeeded.
1b9d7d8 Ensure Python versions ending in t are isntalle... - ChrisLovering
[python-discord/bot] New branch created: support-python-3.13-no-gil
GitHub Actions run 11222800987 succeeded.
GitHub Actions run 11222892435 succeeded.
GitHub Actions run 11226990927 failed.
GitHub Actions run 11233057432 failed.
GitHub Actions run 11235765652 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.15.0
GitHub Actions run 11236756939 succeeded.
GitHub Actions run 11236844113 succeeded.
GitHub Actions run 11242816954 succeeded.
GitHub Actions run 11246827652 succeeded.
GitHub Actions run 11248606852 succeeded.
GitHub Actions run 11253545390 succeeded.
Closes #2808
This PR aims to improve the resiliency of the Branding manager cog. It is a rework of #2869, which I've closed and re-implemented in this branch in adherence with the reviewer's request to use the tenacity lib.
Context
The Branding manager automatically discovers events once per day or when instructed to. During each such discovery, it makes multiple HTTP requests to the GitHub API. First, it fe...
GitHub Actions run 11260451365 succeeded.
GitHub Actions run 11260577359 succeeded.
For testing purposes, you can do this. :trollface:
async with self.bot.http_session.get(...) as response:
response.status = 500
_raise_for_status(response)
Relevant discussion: #python-discussion message
Summary of discussion: PEP 760 has been withdrawn, but at the time of discussion, !pep 760 still returns "Draft" for status.
Possible causes suggested:
- Caching is faulty (by @eivl, @thurisatic)
GitHub Actions run 11286794196 succeeded.
Nit: use py_version instead of 1
Nit: can we make this more readable by creating the dict on a separate line and saving it in a variable?
I take it there's (currently) no way to only exclude nogil builds?
Haven't tested this but the code looks great.
unfortunatly not right now.
https://discuss.python.org/t/environment-marker-for-free-threading/60007/5
I'll force push the commit to link to this discourse thread
9bf5123 Bump 3.12 to 3.12.7 from 3.12.5 - ChrisLovering
d7f57ee Add test to ensure free threaded versions have ... - ChrisLovering
aecc6f4 Also install 3.13t - ChrisLovering
f7ce899 Ensure Python versions ending in t are installe... - ChrisLovering
979e51a Don't install packages that aren't compatible w... - ChrisLovering
[python-discord/snekbox] branch deleted: 3.13t
[python-discord/snekbox] New branch created: 3.13-stable
771b9cd Bump up to stable versions of Python 3.13 and 3... - ChrisLovering
3e363a8 Bump coverallsapp/github-action in the ci-depen... - dependabot[bot]
bc88836 Rename kwarg in eval button to be clearer - ChrisLovering
fbfb299 Dynamically build eval switcher buttons based o... - ChrisLovering
516c986 Add support for 3.13t in eval - ChrisLovering
29a1310 Special case Python versions ending in t for ev... - ChrisLovering
b29ef1e Ensure eval version buttons ascend in release date - ChrisLovering
[python-discord/bot] branch deleted: support-python-3.13-no-gil
Connected!
GitHub Actions run 11305546064 succeeded.
2fc04c6 Bump up to stable versions of Python 3.13 and 3... - ChrisLovering
[python-discord/snekbox] branch deleted: 3.13-stable
[python-discord/bot] New branch created: remove-invite-filter-bypass
dd8c198 Drop newlines and url unquote messages before p... - ChrisLovering
GitHub Actions run 11317126032 succeeded.
GitHub Actions run 11344549794 succeeded.
GitHub Actions run 11352201460 succeeded.
GitHub Actions run 11352203867 succeeded.
GitHub Actions run 11352895474 succeeded.
GitHub Actions run 11352919194 succeeded.
GitHub Actions run 11359471033 succeeded.
GitHub Actions run 11378867990 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.17.0
GitHub Actions run 11382879205 succeeded.
GitHub Actions run 11382983708 succeeded.
GitHub Actions run 11383519129 succeeded.
GitHub Actions run 11384567752 succeeded.
GitHub Actions run 11390916140 succeeded.
GitHub Actions run 11392484303 failed.
[python-discord/bot-core] New branch created: invite-regex
90cb27b Drop / from invite regex fragements - ChrisLovering
3ee615e Improve readanility of invite regex by indentin... - ChrisLovering
23434f0 Add optional 'app' capture to invite regex OR g... - ChrisLovering
21caddb Move discord(app) capture to its own optional g... - ChrisLovering
bafaf69 Add an optional capture for /invite to invite r... - ChrisLovering
Here is the regex in one line for testing with regex101 or something
(https?:\/\/)?(www\.)?(discord(app)?)?(([.,]|dot)gg|([.,]|dot)com|([.,]|dot)me|([.,]|dot)li|([.,]|dot)io)((\/|slash|\\)(invite))?(\/|slash|\\)(?P\S+)
GitHub Actions run 11401761356 failed.
GitHub Actions run 11503105485 succeeded.
GitHub Actions run 11503922849 succeeded.
GitHub Actions run 11512460581 succeeded.
GitHub Actions run 11516289791 failed.
GitHub Actions run 11518412954 succeeded.
The user command supports discord username input, I'd like to see more commands support this such as
infraction
note
Maybe watch or other low risk commands.
This would help mobile moderation efforts.
Similar to how we can reply to a message to add a bookmark (Sir Lancebot's bookmark command) I'd like to be able to infraction append by replying to the bot's message that gets sent whenever we issue an infraction in mod channels.
This would help when appending to infractions issued by others (where infraction append last would not apply) so we can be less careful about typing the exact infraction ID.
I agree with having it in #message-change-log at minimum.
I still like having the current output we get in #mods, ambivalent about #mod-log
GitHub Actions run 11531114384 succeeded.
Relevant Issues
Description
Did you:
- [ ] Join the Python Discord Community?
- [ ] Read all the comments in this template?
- [ ] Ensure there is an issue open, or link relevant discord discussions?
- [ ] Read and agree to the contributing guidelines?
Commandes pour crรฉe un bot tรฉlรฉgrame avec des options avancรฉes
[python-discord/bot] branch deleted: fix-2500
[python-discord/bot] branch deleted: tag-suggestion-template
[python-discord/bot] branch deleted: fix-2492
[python-discord/bot] branch deleted: add-reminder-snoozing
[python-discord/bot] branch deleted: update-pep-cog
[python-discord/bot] branch deleted: helpdm
[python-discord/bot] branch deleted: fix-dm-sent-feature
[python-discord/bot] branch deleted: mbaruh/persistent_scheduler
[python-discord/bot] branch deleted: ui

GitHub Actions run 11633260388 succeeded.
Description
Add a .typst command, working similarly to .latex but for rendering Typst.
Reasoning
The bot's .latex command has been notoriously unstable in the past, but nowadays is mostly working. However, it's pretty hard to consistently write correct LaTeX snippets on the first try, and the error logs are not especially helpful because LaTeX logs rarely are. It would be nice to have a simpler alternative for quickly writing some simple math, and Typst would fill that niche...
GitHub Actions run 11652483985 succeeded.
git add remote isn't valid, and most likely was a typo.
GitHub Actions run 11653155210 succeeded.
GitHub Actions run 11662721794 failed.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.17.0
GitHub Actions run 11665841105 succeeded.
GitHub Actions run 11666785492 succeeded.
GitHub Actions run 11678308438 succeeded.
GitHub Actions run 11682128887 succeeded.
GitHub Actions run 11683904129 succeeded.
GitHub Actions run 11709075927 succeeded.
I received msg You're bad at computers
But it is completely not true ๐ i am paranoid spam evader
Keeping DMs opened is too much inviting for spammers. Especially at such large server like Python.
New suggestion to fix
"You're bad at computers.",
"Are you trying to kill me?",
"Fat chance.",
Just remove this and it should be okay imo
Reviewed msgs that looked hurtful and not desirable And offered to replace with some alternative joke options
addressing https://github.com/python-discord/bot/issues/3152
GitHub Actions run 11714239492 succeeded.
nit: only one that doesn't start with a capital letter
"Error 418. I am a teapot.",
LGTM, quick win to improve friendliness
GitHub Actions run 11714445478 succeeded.
GitHub Actions run 11714540308 succeeded.
GitHub Actions run 11723532462 succeeded.
Looks good. I can merge when the suggested change I made is resolved.
ERROR_REPLIES doesn't actually seem to be used anywhere, so those changes wont do anything. sir-lancebot has a copy of the same replies and it does use them, so it would be more useful to change them there.
We should move the lists into bot-core so we don...
"I would love to, but unfortunately... no.",
GitHub Actions run 11741814879 succeeded.
Have you considered using io.BytesIO? Or does the library not support it?
It looks like it does support not writing an output file (just need to not pass output). I'll change the implementation to use that. It will come with the problem that a maliciously constructed input can return an arbitrarily big bytes object, but I can probably fix this by setting a an rlimit on memory usage for the worker process.
If you're worried about a maliciously large output, that's probably a good idea regardless of the method
Closing this as we haven't reached a consensus within the core dev team.
#bot-commands message
Notice how the invoking message is !timeit but the response is one you would get from !eval
Cause:
- Run
!timeitcommand with an error (I don't remember the details, but I think it was something to do with codeblock fencing) - Fix the error by editing the message
- Rerun with the bot reaction
This seems like an edge case because the re-run immediately after that worked perfectly fine
Relevant Issues
Closes #1623.
Description
- The overall structure is based on the Latex cog, with about the same features (caching, support for code blocks, etc)
- Refactored some utility functions (such as the codeblock regex) out of the Latex cog to reuse them here.
- Initially I was going to use the
typstPyPI package, but while it works, it had a weird issue where (only in the bot's container and not on my computer) it needed ~1GB RAM to render the simplest things. I failed...
GitHub Actions run 11759086123 succeeded.
GitHub Actions run 11759581500 succeeded.
Relevant Issues
Smilar PR is already opened about it over there https://github.com/python-discord/bot/pull/3194
Description
Reviewed msgs that looked hurtful and not desirable And offered to replace with some alternative joke options
addressing https://github.com/python-discord/bot/issues/3152
Relevant discussion: #community-meta message
Did you:
- [x] Join the [Python Discord Community]...
@wookie184 merged suggestions an opened duplicate in lancerbot https://github.com/python-discord/sir-lancebot/pull/1627
GitHub Actions run 11759627380 succeeded.
GitHub Actions run 11776989033 failed.
GitHub Actions run 11827281188 succeeded.
Discord does not like periods (.) in aliased URLs, so this section of markdown is not rendered. Bare URLs, though, get made into links, and it looks fine.
One of the features new to Python 3.13 is the use of colored traceback output. These colors greatly aid in locating smaller bugs and are overall quite handy, but they are not always enabled, including for the bot's output message. This changes that.
Care was taken to achieve a significant degree of backwards compatibility. Long tracebacks, whose full forms are uploaded to pinnwand, go through a regular expression that removes ANSI escape codes. Pinnwand does not support ANSI, so if this ...
This PR is the snekbox side of changes made to allow bot's !eval to support colored output. The first of the two commits for bot's side allows ANSI codes to be rendered when given via print or such, but Python 3.13 does not produce these colors in tracebacks without an environment variable. Per documentation, there are two such envars - PYTHON_COLORS and FORCE_COLOR. Testing has shown FORCE_COLOR to have no effect ...
GitHub Actions run 11845643278 succeeded.
GitHub Actions run 11845652303 succeeded.
Connected!
GitHub Actions run 11846012133 succeeded.
GitHub Actions run 11856246766 succeeded.
GitHub Actions run 11878288817 succeeded.
GitHub Actions run 11878742477 succeeded.
GitHub Actions run 11891548638 failed.
GitHub Actions run 11901458585 succeeded.
GitHub Actions run 11901532254 succeeded.
GitHub Actions run 11901536341 succeeded.
GitHub Actions run 11913225364 succeeded.
GitHub Actions run 11933913604 succeeded.
GitHub Actions run 11943564333 succeeded.
GitHub Actions run 11957484885 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.18.0
GitHub Actions run 11957841753 succeeded.
GitHub Actions run 11965014161 succeeded.
GitHub Actions run 11972465199 failed.
GitHub Actions run 11973047886 succeeded.
GitHub Actions run 11973071256 succeeded.
GitHub Actions run 11974572444 succeeded.
e0d6f92 Drop / from invite regex fragements - ChrisLovering
8528c7f Improve readability of invite regex by indentin... - ChrisLovering
9385e91 Add optional 'app' capture to invite regex OR g... - ChrisLovering
4df8e90 Move discord(app) capture to its own optional g... - ChrisLovering
5911e10 Remove repetition from the invite regex - ChrisLovering
[python-discord/bot-core] branch deleted: invite-regex
[python-discord/bot-core] New tag created: v11.5.0
4497a82 Deploying to docs from @ python-discord/bot-cor... - ChrisLovering
GitHub Actions run 11989599515 succeeded.
f2a028d Sanitize MSVC++ build tools link for Discord (#... - thurisatic
aebc88a fix: Replace unfriendly error messages (#3194) - dd84ai
161d928 Drop newlines and url unquote messages before p... - ChrisLovering
[python-discord/bot] branch deleted: remove-invite-filter-bypass
GitHub Actions run 11989609204 was cancelled.
GitHub Actions run 11989608534 was cancelled.
Connected!
GitHub Actions run 11989609973 succeeded.
GitHub Actions run 11989747509 failed.
Tag looks great, lets merge this. If we want to add a gif we can do so in a later PR.
[python-discord/bot] branch deleted: feat/tag-list-iterate-and-remove-gotcha
447ddfb fix: replace unfriendly error msgs (#1627) - dd84ai
GitHub Actions run 11989773475 failed.
Connected!
GitHub Actions run 11989775802 succeeded.
Connected!
GitHub Actions run 11989791432 succeeded.
GitHub Actions run 11989819009 was cancelled.
Once copyright free images have been found, we can look into opening this back up and implementing this change.
Connected!
I'm not entirely sure we want to add & maintain this, given that running it via docker compose is much simpler, and easier to explain to new developers.
GitHub Actions run 11989829235 succeeded.
[python-discord/sir-lancebot] New review comment on pull request #1626: Typst command implementation
This isn't going to work in our infrastructure. We use read-only file systems.
I am also very uneasy with the fact we're downloading and executing a binary from the internet like this into the bot's pod.
Is there a typst API similar to what we use for latex? Either that, or we can look into having a typst deployment in our infra, where it won't have access to the bot's secrets.
22867d7 Fix typo in git remote add in the guide (#1437) - RundownRhino
5bd25bb Remove discriminator from bot logs view (#1424) - jchristgit
[python-discord/site] branch deleted: remove-discrim-from-bot-logs-view
GitHub Actions run 11989880959 was cancelled.
GitHub Actions run 11989882569 succeeded.
4d4f816 Bump coverallsapp/github-action from 2.3.0 to 2... - dependabot[bot]
GitHub Actions run 11989957581 was cancelled.
GitHub Actions run 11989960464 succeeded.
Connected!
[python-discord/bot-core] branch deleted: update-regex
Relevant Issues
Description
Did you:
- [ ] Join the Python Discord Community?
- [ ] Read all the comments in this template?
- [ ] Ensure there is an issue open, or link relevant discord discussions?
- [ ] Read and agree to the contributing guidelines?
[python-discord/bot-core] New branch created: fix-invite-regex
GitHub Actions run 11990396649 succeeded.
Connected!
GitHub Actions run 11990478465 succeeded.
21a32de Update filter regex to not be overly strict - ChrisLovering
3b7f09e Drop io and li from invite regex - ChrisLovering
93ce65f Escape backslashes in invite regex - ChrisLovering
f801992 Ensure we don't match to non-invite looking links - ChrisLovering
7268e93 Update invite regex test cases - ChrisLovering
[python-discord/bot-core] branch deleted: fix-invite-regex
[python-discord/bot-core] New tag created: v11.5.1
Connected!
GitHub Actions run 11990677887 succeeded.
Follow up to https://github.com/python-discord/sir-lancebot/pull/1596 , the wrong function was called.
Fixes SIR-LANCEBOT-CF
Here is proof it works:
Did you:
- [ ] Join the Python Discord Community?
- [ ] Read all the comments in this template?
- [ ] Ensure there is an issue open, or link relevant discord discussions?
- [ ] Read and agree to the [contri...
GitHub Actions run 11991143504 was cancelled.
GitHub Actions run 11991149183 succeeded.
GitHub Actions run 11992095250 succeeded.
GitHub Actions run 12003579390 succeeded.
GitHub Actions run 12008587089 succeeded.
GitHub Actions run 12010091116 succeeded.
GitHub Actions run 12010098381 succeeded.
GitHub Actions run 12036276066 succeeded.
GitHub Actions run 12044380023 succeeded.
GitHub Actions run 12048736173 succeeded.
GitHub Actions run 12050442385 succeeded.
8027ba6 Don't ignore newlines when filtering invites - mbaruh
[python-discord/bot] New branch created: fix/newlines-invites
Fixes a bug where in the following message:
discord.gg/python
discord.gg/djs
The newlines would be ignored, and the bot would end up checking the invite discord.gg/pythondiscord and therefore blocking the message.
GitHub Actions run 12054975769 succeeded.
16441a1 Don't ignore newlines when filtering invites (#... - mbaruh
[python-discord/bot] branch deleted: fix/newlines-invites
Connected!
GitHub Actions run 12056732563 succeeded.
GitHub Actions run 12062818230 succeeded.
GitHub Actions run 12064347135 succeeded.
GitHub Actions run 12068985210 succeeded.
GitHub Actions run 12072532392 succeeded.
GitHub Actions run 12078951720 succeeded.
GitHub Actions run 12079718753 succeeded.
GitHub Actions run 12079720864 succeeded.
GitHub Actions run 12080966375 succeeded.
GitHub Actions run 12083482553 succeeded.
Connected!
Connected!
GitHub Actions run 12097802060 succeeded.
Connected!
GitHub Actions run 12097897735 succeeded.
Connected!
GitHub Actions run 12097938527 succeeded.
GitHub Actions run 12111213043 succeeded.
GitHub Actions run 12120399579 succeeded.
GitHub Actions run 12131139625 succeeded.
Connected!
GitHub Actions run 12139772775 succeeded.
GitHub Actions run 12139809245 succeeded.
LGTM
Though all these criticisms are still valid on the paid tiers, aren't they? As a possible alternative, we could change the header from "why free hosts are bad" to "why shared hosts are bad", and leave the content intact.
"Shared" hosts is maybe not the right term, any VPS is technically shared as long as you don't have a dedicated server.
GitHub Actions run 12144624887 succeeded.
Connected!
GitHub Actions run 12153171631 succeeded.
GitHub Actions run 12181719796 succeeded.
GitHub Actions run 12184097427 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.19.0
GitHub Actions run 12185601147 succeeded.
Connected!
GitHub Actions run 12187765014 succeeded.
GitHub Actions run 12187831201 succeeded.
Connected!
GitHub Actions run 12193264010 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.19.1
GitHub Actions run 12196075517 succeeded.
GitHub Actions run 12196267828 succeeded.
GitHub Actions run 12197587154 succeeded.
GitHub Actions run 12197595011 succeeded.
GitHub Actions run 12199107150 succeeded.
GitHub Actions run 12199120331 succeeded.
GitHub Actions run 12200052871 succeeded.
GitHub Actions run 12204305209 succeeded.
GitHub Actions run 12227714717 succeeded.
GitHub Actions run 12229631380 succeeded.
GitHub Actions run 12269565042 succeeded.
GitHub Actions run 12297778547 succeeded.
GitHub Actions run 12300206661 succeeded.
Often people join the leaderboard, link their account, and then use our leaderboard command. An issue pops up if the leaderboard refresh is still on cooldown. An error message pops up and prevents the user from viewing the current leaderboard until the leaderboard refreshes.
Instead, we should still show an error message indicated we cannot find their username, but still show the regular leaderboard embed so they can still view leaderboard information.
GitHub Actions run 12310019020 succeeded.
GitHub Actions run 12313740879 succeeded.