dev-log

[regal arch]

shtlrs

[python-discord/king-arthur] New comment on pull request #169: Paginate pod logs instead of truncating them

We could, and will, I just don't know which of the output samples I gave are more appreciated.

I like the first one more, but I would strongly prefer if all the messages were wrapped in a single codeblock, therefore the prefix and suffix suggestion.

How do you like this ?

Esther-Goldberg

[python-discord/snekbox] New comment on issue #205: Weird-byte filenames trigger internal server error on json encode

Can I work on this?

[regal arch]

MarkKoz

[python-discord/snekbox] New comment on issue #205: Weird-byte filenames trigger internal server error on json encode

Can I work on this?

Yes!

[regal arch]

Esther-Goldberg

[snekbox:invalid_bytes_in_filename] 1 new commit

26c83cc raise error if attachment pathname contains inv... - Esther-Goldberg

Esther-Goldberg

[python-discord/snekbox] New branch created: invalid_bytes_in_filename

Esther-Goldberg

[python-discord/snekbox] Pull request opened: #209 raise error if attachment pathname contains invalid bytes

safe_path is used for input, to parse a file path from the dict. This error is occurring when doing the reverse. This change throws an EvalError if any file created by the Python code isn't able to be encoded in UTF-8.

[snekbox] GitHub Actions checks failure on pull request #209

[regal arch]

[snekbox] GitHub Actions checks success on pull request #209

[regal arch]

jb3

[metricity:jb3/dep-bumps] 7 new commits

f1933ed Bump Poetry Python version to 3.12 - jb3
72b2a26 Fix no active event loop in create_metricity_db.py - jb3
834f4b2 Remove redundant version in docker-compose.yml - jb3
4ffff02 Bump Docker image to Python 3.12 - jb3
566a194 Upgrade dependencies - jb3

jb3

[python-discord/metricity] New branch created: jb3/dep-bumps

jb3

[python-discord/metricity] Pull request opened: #142 Bump Poetry Python version to 3.12

• Bump Python version to 3.12
• Bump all dependencies to close dependabot PRs
• Fix an event loop issue in create_metricity_db.py
• Fix new lint issues from Ruff version bump

jb3

[python-discord/metricity] Pull request closed: #142 Bump Poetry Python version to 3.12

jb3

[metricity:main] 8 new commits

f1933ed Bump Poetry Python version to 3.12 - jb3
72b2a26 Fix no active event loop in create_metricity_db.py - jb3
834f4b2 Remove redundant version in docker-compose.yml - jb3
4ffff02 Bump Docker image to Python 3.12 - jb3
566a194 Upgrade dependencies - jb3

jb3

[python-discord/metricity] branch deleted: jb3/dep-bumps

[regal arch]

jb3

[metricity:jb3/deleted-channel-column] 3 new commits

ed795f3 Add column for a channel that has been deleted - jb3
ef0c908 Add migration for Channel.deleted column - jb3
d192c7d Update channel sync to keep Channel.deleted att... - jb3

jb3

[python-discord/metricity] New branch created: jb3/deleted-channel-column

jb3

[python-discord/metricity] Pull request opened: #143 Add column for a channel that has been deleted

• Add a database column and migration for tracking if a channel has been deleted
• Update the channel syncer to set the deleted flag to True for all channels not in guild.channels, on channel create, delete and bot start-up.

jb3

[python-discord/metricity] Pull request closed: #143 Add column for a channel that has been deleted

jb3

[metricity:main] 4 new commits

ed795f3 Add column for a channel that has been deleted - jb3
ef0c908 Add migration for Channel.deleted column - jb3
d192c7d Update channel sync to keep Channel.deleted att... - jb3
a14f454 Merge pull request #143 from python-discord/jb3... - jb3

jb3

[python-discord/metricity] branch deleted: jb3/deleted-channel-column

jb3

[metricity:main] 1 new commit

d792e1d Move banner onto GitHub instead of Discord CDN - jb3

jb3

[metricity:main] 1 new commit

13c6a31 Version 2.5.0 - jb3

jb3

[python-discord/metricity] New tag created: v2.5.0

wookie184

[python-discord/bot] Pull request review submitted: #3005 Fix showing `!source` on tags when tags cog is reloaded

wookie184

[python-discord/bot] New review comment on pull request #3005: Fix showing `!source` on tags when tags cog is reloaded

    async def get_source_object(ctx: commands.Context, argument: str) -> tuple[SourceObject, SourceType]:

[odd spire]

[python-discord/bot] Checks Cancelled on PR: #3000 Add feature to append additional text to nomination reasons

GitHub Actions run 8532894848 was cancelled.

PR Author

Esther-Goldberg

Workflow Run

CI #1682

Source Branch

append-nominations

[regal arch]

Esther-Goldberg

[python-discord/snekbox] New comment on pull request #209: raise error if attachment pathname contains invalid bytes

This would close #205

[regal arch]

hedyhli

[bot] Branch fix/source-command-for-tags was force-pushed to `ff1d3a8`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3005 Fix showing `!source` on tags when tags cog is reloaded

GitHub Actions run 8592865279 succeeded.

PR Author

hedyhli

Workflow Run

CI #1696

Source Branch

fix/source-command-for-tags

[regal arch]

jb3

[metricity:jb3/deleted-category-attribute] 1 new commit

48bcde8 Track whether a category has been deleted - jb3

jb3

[python-discord/metricity] New branch created: jb3/deleted-category-attribute

jb3

[python-discord/metricity] Pull request opened: #144 Category deletion tracking

jb3

[python-discord/metricity] Pull request closed: #144 Category deletion tracking

jb3

[metricity:main] 2 new commits

48bcde8 Track whether a category has been deleted - jb3
9d30baa Merge pull request #144 from python-discord/jb3... - jb3

jb3

[python-discord/metricity] branch deleted: jb3/deleted-category-attribute

jb3

[metricity:main] 1 new commit

04698e1 Set server_default of deleted flag to False - jb3

jb3

[metricity:main] 1 new commit

b8f2165 Version 2.5.1 - jb3

jb3

[python-discord/metricity] New tag created: v2.5.1

Esther-Goldberg

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

is this being worked on, or can I work on it?

jb3

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

Doesn't look like anyone has assigned to this issue yet, I will assign you now!

[regal arch]

Esther-Goldberg

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

It looks like the current filtering system doesn't have a setting to send embeds in the original channel, so that would have to be added

[regal arch]

Esther-Goldberg

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

assuming we also want this for .py files

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1508 Bump rapidfuzz from 3.7.0 to 3.8.1

GitHub Actions run 8598378047 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #3989

Source Branch

dependabot/pip/rapidfuzz...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3006 Bump rapidfuzz from 3.7.0 to 3.8.1

GitHub Actions run 8599902842 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1697

Source Branch

dependabot/pip/rapidfuzz-3.8.1

[python-discord/bot] Checks Successful on PR: #3007 Bump deepdiff from 6.7.1 to 7.0.0

GitHub Actions run 8599908667 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1698

Source Branch

dependabot/pip/deepdiff-7.0.0

[regal arch]

MarkKoz

[python-discord/snekbox] New comment on pull request #209: raise error if attachment pathname contains invalid bytes

Thanks for the PR! Can you add a test for this as well?

MarkKoz

[python-discord/snekbox] New comment on pull request #204: Bump the docker-dependencies group with 1 update

I guess if the CI passes them the pinned dependencies work as is worth 3.12? My instinct would be to recompile them. I don't know if I quite have enough confidence in our integration tests.

[regal arch]

jb3

[python-discord/snekbox] Pull request review submitted: #204 Bump the docker-dependencies group with 1 update

Tested locally, builds and works fine!

jb3

[snekbox:main] 2 new commits

8a9b1f8 Bump the docker-dependencies group with 1 update - dependabot[bot]
f45ae67 Merge pull request #204 from python-discord/dep... - jb3

jb3

[python-discord/snekbox] Pull request closed: #204 Bump the docker-dependencies group with 1 update

[snekbox] GitHub Actions checks success on main

[snekbox] GitHub Actions checks success on main

[regal arch]

Esther-Goldberg

[python-discord/snekbox] New comment on pull request #209: raise error if attachment pathname contains invalid bytes

Sure! I'll add that once I get it done

[regal arch]

Esther-Goldberg

[snekbox:invalid_bytes_in_filename] 1 new commit

fe2d933 test error for invalid bytes in filename - Esther-Goldberg

Esther-Goldberg

[python-discord/snekbox] New comment on pull request #209: raise error if attachment pathname contains invalid bytes

Is a test for the error returned by nsjail enough, or should the function that creates files from paths also have a test?

[snekbox] GitHub Actions checks failure on pull request #209

[regal arch]

MarkKoz

[python-discord/snekbox] New comment on pull request #209: raise error if attachment pathname contains invalid bytes

That test is sufficient - it's what I also had in mind. However, the test is failing in CI.

[regal arch]

mbaruh

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

Yeah some setting(s) needs to be added here.

I'm wondering what the correct approach here is: We could have a setting for whether to DM the message or send it in the public channel (in which case it might make sense to change the names of dm_content and dm_embed since it wouldn't necessarily be a DM anymore), or we could add a setting that allows sending the message publicly as a fallback if sending the DM fails (usually because they have DMs off).

[regal arch]

Hashed0719

[python-discord/bot] Pull request closed: #3001 fixed typos

[thin oyster]

Sir Robin

Connected!

[night lily]

Sir Lancebot

Connected!

[regal arch]

sentry-io[bot]

[python-discord/sir-robin] New comment on pull request #116: Our Glorious Python Discord Games

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ ConnectionError: Connection closed by server. bot.exts.games in super_game View Issue

_{Did you find this useful? React with a 👍 or 👎}

[oak estuary]

Python Bot

Connected!

[regal arch]

sentry-io[bot]

[python-discord/bot] New comment on pull request #2991: Also sync user's display_name

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ **ResponseCodeError: Status: 500 Response: ** bot.exts.backend.sync View Issue

_{Did you find this useful? React with a 👍 or 👎}

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3008 Bump deepdiff from 6.7.1 to 7.0.1

GitHub Actions run 8616126852 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1699

Source Branch

dependabot/pip/deepdiff-7.0.1

[regal arch]

Esther-Goldberg

[snekbox:invalid_bytes_in_filename] 1 new commit

9739a0a switch to raw string for test - Esther-Goldberg

Esther-Goldberg

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

People don't necessarily check their DMs in the middle of using a help channel, so in this case we probably want to put an embed in the channel even if they have DMs open.

[snekbox] GitHub Actions checks failure on pull request #209

[snekbox] GitHub Actions checks failure on pull request #209

[snekbox] GitHub Actions checks success on pull request #209

[regal arch]

MarkKoz

[python-discord/snekbox] Pull request review submitted: #209 raise error if attachment pathname contains invalid bytes

MarkKoz

[snekbox:invalid_bytes_in_filename] 3 new commits

8a9b1f8 Bump the docker-dependencies group with 1 update - dependabot[bot]
f45ae67 Merge pull request #204 from python-discord/dep... - jb3
c11b4e8 Merge branch 'main' into invalid_bytes_in_filename - MarkKoz

MarkKoz

[python-discord/snekbox] Pull request closed: #209 raise error if attachment pathname contains invalid bytes

MarkKoz

[snekbox:main] 1 new commit

5b606b1 Raise error if attachment pathname contains inv... - Esther-Goldberg

MarkKoz

[python-discord/snekbox] Issue closed: #205 Weird-byte filenames trigger internal server error on json encode

MarkKoz

[python-discord/snekbox] branch deleted: invalid_bytes_in_filename

[snekbox] GitHub Actions checks success on main

[snekbox] GitHub Actions checks success on invalid_bytes_in_filename

[snekbox] GitHub Actions checks success on main

jb3

[snekbox:jb3/pooch] 1 new commit

1bc37c4 Add pooch to eval deps - jb3

jb3

[python-discord/snekbox] New branch created: jb3/pooch

jb3

[python-discord/snekbox] branch deleted: jb3/pooch

jb3

[python-discord/snekbox] branch deleted: prctl-reparenting

[regal arch]

mbaruh

[python-discord/bot] New comment on issue #2854: Revert to previous txt file deletion behavior

Yeah that makes sense

[night lily]

Sir Lancebot

Connected!

[regal arch]

jb3

[olli:main] 1 new commit

e3aece6 Support new Loki 3.0 service_name - jb3

jb3

[olli:main] 1 new commit

07ed8eb Relock file - jb3

jb3

[olli:main] 1 new commit

91ba297 Bump to 2.0.0 - jb3

jb3

[python-discord/olli] New tag created: v2.0.0

jb3

[python-discord/olli] New release published: v2.0.0

[regal arch]

jb3

[workers:main] 1 new commit

13456ce Version bumps in all workers - jb3

[workers] lint / Lint code success on main

[workers] build-deploy / Deploy Workers success on main

[workers] GitHub Actions checks success on main

jb3

[workers:main] 1 new commit

4a09928 Remove dependabot rules for github-filter - jb3

[workers] Dependabot checks success on main

[workers] Dependabot config file validation on main

All changes look good

[workers] lint / Lint code success on main

[workers] build-deploy / Deploy Workers success on main

[workers] GitHub Actions checks success on main

jb3

[python-discord/workers] New comment on pull request #235: Bump wrangler from 3.7.0 to 3.19.0 in /serve-robots

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #234: Bump wrangler from 3.7.0 to 3.19.0 in /url-unfurler

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #233: Bump wrangler from 3.7.0 to 3.19.0 in /report-uri

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #232: Bump wrangler from 3.7.0 to 3.19.0 in /short-urls

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #227: Bump miniflare and wrangler in /serve-robots

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #226: Bump miniflare and wrangler in /report-uri

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #225: Bump miniflare and wrangler in /short-urls

@dependabot recreate

jb3

[python-discord/workers] New comment on pull request #224: Bump miniflare and wrangler in /url-unfurler

@dependabot recreate

[regal arch]

jb3

[python-discord/king-arthur] New branch created: jb3/python-3.12

jb3

[king-arthur:jb3/python-3.12] 4 new commits

1f0595a Update pyproject.toml and poetry.lock - jb3
4d77c17 Add packages section to pyproject.toml - jb3
99cb9bf Upgrade Dockerfile to use 3.12 image - jb3
f63aada Update lint workflow to use Python 3.12 - jb3

jb3

[python-discord/king-arthur] Pull request opened: #174 Bump to Python 3.12

Bump all dependencies and Python version to Python 3.12

jb3

[king-arthur:jb3/python-3.12] 1 new commit

edee6fc Run ruff format to fix format changes - jb3

[regal arch]

jb3

[king-arthur:jb3/python-3.12] 1 new commit

12641ee Change Poetry to install --no-root in Docker - jb3

jb3

[python-discord/king-arthur] Pull request closed: #174 Bump to Python 3.12

jb3

[king-arthur:main] 7 new commits

1f0595a Update pyproject.toml and poetry.lock - jb3
4d77c17 Add packages section to pyproject.toml - jb3
99cb9bf Upgrade Dockerfile to use 3.12 image - jb3
f63aada Update lint workflow to use Python 3.12 - jb3
edee6fc Run ruff format to fix format changes - jb3

jb3

[python-discord/king-arthur] branch deleted: jb3/python-3.12

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 8637801233 succeeded.

Actor

jb3

Workflow Run

Status embed #325

Commit

eb615ca

[odd spire]

[python-discord/site] Checks Successful on PR: #1294 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8642074449 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1126

Source Branch

dependabot/pip/sentry-sdk-1.45.0

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1509 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8645555790 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #3990

Source Branch

dependabot/pip/sentry-sd...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3009 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8646398474 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1700

Source Branch

dependabot/pip/sentry-sdk-1.45.0

[regal arch]

jchristgit

[python-discord/site] Pull request review submitted: #1294 Bump sentry-sdk from 1.44.1 to 1.45.0

jchristgit

[python-discord/site] Pull request closed: #1294 Bump sentry-sdk from 1.44.1 to 1.45.0

jchristgit

[site:main] 2 new commits

30753c9 Bump sentry-sdk from 1.44.1 to 1.45.0 - dependabot[bot]
e2b0090 Merge pull request #1294 from python-discord/de... - jchristgit

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1510 Bump idna from 3.6 to 3.7

GitHub Actions run 8656769328 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #3991

Source Branch

dependabot/pip/idna-3.7

[python-discord/bot] Checks Successful on PR: #3010 Bump idna from 3.6 to 3.7

GitHub Actions run 8656757737 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1701

Source Branch

dependabot/pip/idna-3.7

[odd spire]

[python-discord/site] Checks Successful on PR: #1295 Bump idna from 3.6 to 3.7

GitHub Actions run 8657182930 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1128

Source Branch

dependabot/pip/idna-3.7

[odd spire]

[python-discord/site] Checks Successful on PR: #1296 Bump ruff from 0.3.5 to 0.3.7

GitHub Actions run 8657577369 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1129

Source Branch

dependabot/pip/ruff-0.3.7

[regal arch]

Hashed0719

[python-discord/bot] Pull request opened: #3011 fixed typos in tags embed.

Fixed typos discussed in #2583. Removed unnecessary merge commit in previous(closed) PR #3001 .

[odd spire]

[python-discord/bot] Checks Successful on PR: #3011 fixed typos in tags embed.

GitHub Actions run 8657660250 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1702

Source Branch

Hashed0719:main

[regal arch]

GriceTurrble

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

GriceTurrble

[python-discord/site] Pull request review submitted: #1295 Bump idna from 3.6 to 3.7

GriceTurrble

[python-discord/olli] Pull request review submitted: #35 Bump idna from 3.6 to 3.7

GriceTurrble

[python-discord/sir-lancebot] Pull request review submitted: #1510 Bump idna from 3.6 to 3.7

GriceTurrble

[python-discord/sir-robin] Pull request review submitted: #121 Bump idna from 3.6 to 3.7

GriceTurrble

[python-discord/bot] Pull request review submitted: #3010 Bump idna from 3.6 to 3.7

[regal arch]

hedyhli

[python-discord/site] Pull request review submitted: #1295 Bump idna from 3.6 to 3.7

hedyhli

[python-discord/site] Pull request closed: #1295 Bump idna from 3.6 to 3.7

hedyhli

[site:main] 2 new commits

a90bc0e Bump idna from 3.6 to 3.7 - dependabot[bot]
c27b112 Merge pull request #1295 from python-discord/de... - hedyhli

[odd spire]

[python-discord/site] Checks Successful on PR: #1296 Bump ruff from 0.3.5 to 0.3.7

GitHub Actions run 8660088238 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1131

Source Branch

dependabot/pip/ruff-0.3.7

[regal arch]

hedyhli

[python-discord/sir-lancebot] Pull request review submitted: #1510 Bump idna from 3.6 to 3.7

hedyhli

[sir-lancebot:main] 2 new commits

972f524 Bump idna from 3.6 to 3.7 - dependabot[bot]
4990401 Merge pull request #1510 from python-discord/de... - hedyhli

hedyhli

[python-discord/sir-lancebot] Pull request closed: #1510 Bump idna from 3.6 to 3.7

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1511 Bump ruff from 0.3.5 to 0.3.7

GitHub Actions run 8660791311 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #3995

Source Branch

dependabot/pip/ruff-0.3.7

[odd spire]

[python-discord/bot] Checks Successful on PR: #3012 Bump ruff from 0.3.5 to 0.3.7

GitHub Actions run 8661698941 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1703

Source Branch

dependabot/pip/ruff-0.3.7

[odd spire]

[python-discord/bot] Checks Successful on PR: #3011 fixed typos in tags embed.

GitHub Actions run 8663051648 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1704

Source Branch

Hashed0719:main

[regal arch]

GriceTurrble

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3011 fixed typos in tags embed.

GitHub Actions run 8663711372 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1705

Source Branch

Hashed0719:main

[regal arch]

Hashed0719

[python-discord/bot] New comment on pull request #3011: fixed typos in tags embed.

ok it seems like i can't really open two different PRs from same branch, the changes are referential until its merged, do i need to create branches locally for that ?

Hashed0719

[python-discord/bot] New comment on pull request #3011: fixed typos in tags embed.

@GriceTurrble sorry for inconvenience but would you be able to reverify the changes if a rebase and recommit in this PR only or do i have to open a new one ?

[regal arch]

GriceTurrble

[python-discord/bot] New comment on pull request #3011: fixed typos in tags embed.

@GriceTurrble sorry for inconvenience but would you be able to reverify the changes if a rebase and recommit in this PR only or do i have to open a new one ?

Either way, I suppose?

[regal arch]

Abdikano

[python-discord/king-arthur] New star added

[odd spire]

[python-discord/bot] Checks Successful on PR: #3011 fixed typos in tags embed.

GitHub Actions run 8666186128 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1706

Source Branch

Hashed0719:main

[regal arch]

Hashed0719

[python-discord/bot] New comment on pull request #3011: fixed typos in tags embed.

@GriceTurrble done.

Hashed0719

[python-discord/bot] New comment on pull request #3011: fixed typos in tags embed.

Again sorry for the inconvenience.

ChrisLovering

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

[regal arch]

GriceTurrble

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

[regal arch]

vivekashok1221

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

LGTM.

hedyhli

[python-discord/bot] Pull request review submitted: #3010 Bump idna from 3.6 to 3.7

hedyhli

[python-discord/bot] Pull request closed: #3010 Bump idna from 3.6 to 3.7

hedyhli

[bot:main] 2 new commits

354e92a Bump idna from 3.6 to 3.7 - dependabot[bot]
a6c8a09 Merge pull request #3010 from python-discord/de... - hedyhli

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3009 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8669516332 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1708

Source Branch

dependabot/pip/sentry-sdk-1.45.0

[regal arch]

hedyhli

[python-discord/bot] Pull request review submitted: #3011 fixed typos in tags embed.

thanks!

hedyhli

[python-discord/bot] Pull request closed: #3011 fixed typos in tags embed.

hedyhli

[bot:main] 2 new commits

4473f10 Fixed typos in tags embed. - Hashed0719
15f4d03 Merge pull request #3011 from Hashed0719/main - hedyhli

[odd spire]

[python-discord/bot] Checks Successful on PR: #3011 fixed typos in tags embed.

GitHub Actions run 8669905027 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1712

Source Branch

Hashed0719:main

[oak estuary]

Python Bot

Connected!

[regal arch]

Hashed0719

[python-discord/bot] Pull request opened: #3013 Removed rich embed filter as its no longer necessary.

closes #2977 .

[odd spire]

[python-discord/bot] Checks Successful on PR: #3013 Removed rich embed filter as its no longer necessary.

GitHub Actions run 8671107843 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1714

Source Branch

Hashed0719:main

[regal arch]

hedyhli

[python-discord/bot] Issue opened: #3014 Document how to invoke tags within tag groups in `/tag` command

We don't currently have any tag groups, as far as I know (#2423), but the /tag command docstring references it and it can perhaps be made clearer (or more concisely describe) how tags in tag groups can be invoked.

Proposed by @RohanJnr in #2583:

We need better help message to tell users on how to use tags from a group. Currently, the syntax to view a tag from a group is /tag name: , this has to be mentioned in the tag help message.

![image](https://github.com/python-discord/bo...

hedyhli

[python-discord/bot] Issue closed: #2583 `/tag` command output bugs

hedyhli

[python-discord/bot] New comment on issue #2583: `/tag` command output bugs

An update on the points proposed in the issue:

• [x] Empty bullet (#3011)
• [x] "Did you mean ..." (#3011)
• [x] Tag embed footer (#2987)
• [ ] Extra aliases in tag suggestions

@RohanJnr, regarding improving the docs for /tag -- although we don't actually have any tag groups yet, I agree it could perhaps be made clearer. I've opened a separate issue for it since this is unrelated to the /tag output in this issue. (#3014)

Now it looks like the aliases cluttering already have an...

[regal arch]

Hashed0719

[python-discord/bot] New comment on issue #2977: Remove `rich_embed` filter

@wookie184 can you review the PR and delete the filter settings.

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3013 Removed rich embed filter as its no longer necessary.

LGTM, thanks.

I'll remove the filter on the server just before merging, once this has another approval from someone.

[regal arch]

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1508 Bump rapidfuzz from 3.7.0 to 3.8.1

wookie184

[python-discord/sir-lancebot] Pull request closed: #1508 Bump rapidfuzz from 3.7.0 to 3.8.1

wookie184

[sir-lancebot:main] 1 new commit

cfffe4a Bump rapidfuzz from 3.7.0 to 3.8.1 (#1508) - dependabot[bot]

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1511 Bump ruff from 0.3.5 to 0.3.7

[night lily]

Sir Lancebot

Connected!

[regal arch]

wookie184

[python-discord/sir-lancebot] Pull request closed: #1511 Bump ruff from 0.3.5 to 0.3.7

wookie184

[sir-lancebot:main] 1 new commit

2f2bd16 Bump ruff from 0.3.5 to 0.3.7 (#1511) - dependabot[bot]

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1509 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8679586802 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #3999

Source Branch

dependabot/pip/sentry-sd...

[regal arch]

wookie184

[sir-lancebot:remove-sentry-experiment] 1 new commit

a1c3cb1 Replace sentry experiment with standard version... - wookie184

wookie184

[python-discord/sir-lancebot] New branch created: remove-sentry-experiment

wookie184

[python-discord/sir-lancebot] Pull request opened: #1512 Replace sentry experiment with standard version and add AsyncioIntegration

There aren't any transactions currently, so this wont do much, but the experiment config will be removed in the future so we shouldn't use it, and this brings us more in line with bot's config.

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1512 Replace sentry experiment with standard version and add AsyncioIntegration

GitHub Actions run 8679694957 succeeded.

PR Author

wookie184

Workflow Run

Status Embed #4001

Source Branch

remove-sentry-experiment

[regal arch]

ChrisLovering

[python-discord/sir-lancebot] Pull request closed: #1512 Replace sentry experiment with standard version and add AsyncioIntegration

ChrisLovering

[sir-lancebot:main] 1 new commit

051d38f Replace sentry experiment with standard version... - wookie184

ChrisLovering

[python-discord/sir-lancebot] branch deleted: remove-sentry-experiment

[night lily]

Sir Lancebot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3005 Fix showing `!source` on tags when tags cog is reloaded

Looks great, thanks!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3006 Bump rapidfuzz from 3.7.0 to 3.8.1

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1509 Bump sentry-sdk from 1.44.1 to 1.45.0

wookie184

[python-discord/sir-lancebot] New comment on pull request #1509: Bump sentry-sdk from 1.44.1 to 1.45.0

@dependabot rebase

wookie184

[python-discord/bot] Pull request review submitted: #3012 Bump ruff from 0.3.5 to 0.3.7

wookie184

[python-discord/sir-lancebot] Pull request closed: #1509 Bump sentry-sdk from 1.44.1 to 1.45.0

wookie184

[sir-lancebot:main] 1 new commit

18bcb9a Bump sentry-sdk from 1.44.1 to 1.45.0 (#1509) - dependabot[bot]

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1509 Bump sentry-sdk from 1.44.1 to 1.45.0

GitHub Actions run 8679826592 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4003

Source Branch

dependabot/pip/sentry-sd...

[regal arch]

mbaruh

[python-discord/bot] Pull request review dismissed: #2982 Fix case sensitivity bug in phishing button

[night lily]

Sir Lancebot

Connected!

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #3006: Bump rapidfuzz from 3.7.0 to 3.8.1

@dependabot rebase

wookie184

[python-discord/bot] Pull request closed: #3006 Bump rapidfuzz from 3.7.0 to 3.8.1

wookie184

[bot:main] 1 new commit

8732b9e Bump rapidfuzz from 3.7.0 to 3.8.1 (#3006) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3006 Bump rapidfuzz from 3.7.0 to 3.8.1

GitHub Actions run 8679833033 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1715

Source Branch

dependabot/pip/rapidfuzz-3.8.1

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request closed: #3012 Bump ruff from 0.3.5 to 0.3.7

wookie184

[bot:main] 1 new commit

30fd13c Bump ruff from 0.3.5 to 0.3.7 (#3012) - dependabot[bot]

[odd spire]

[python-discord/bot] Workflow Cancelled

GitHub Actions run 8679834639 was cancelled.

Actor

wookie184

Workflow Run

CI #1716

Commit

8732b9e

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #2982 Fix case sensitivity bug in phishing button

No issues I can see, just some questions/clarifications

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

I understand the change to use re, but does the variable name change + assignment on line 695 fix anything or is it just to make the code clearer?

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

Is there a reason we don't normalise urls to lowercase before adding them to the filter list?

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

Is this added because if they were autobanned then it's probably because the URL they sent is already added as a filter? If so, I think the comment should explain that.

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3009 Bump sentry-sdk from 1.44.1 to 1.45.0

wookie184

[python-discord/bot] Pull request closed: #3009 Bump sentry-sdk from 1.44.1 to 1.45.0

wookie184

[bot:main] 1 new commit

324cb65 Bump sentry-sdk from 1.44.1 to 1.45.0 (#3009) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3008 Bump deepdiff from 6.7.1 to 7.0.1

GitHub Actions run 8680108176 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1724

Source Branch

dependabot/pip/deepdiff-7.0.1

[regal arch]

vivekashok1221

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

I'm not sure. One potential reason I can think of is that that normalizing URLs to lowercase might lead to loss of distinction between case-sensitive URLs (example.com/aBcD vs example.com/abcd) so we will not be able to visit the website ourselves later (, perhaps, for verification reasons). Although I guess we could search for the message with the original link but that would be tedious.

[regal arch]

vivekashok1221

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

The former.

This change fixe the first bug I've mentioned in the PR description.
Invite links of the form https://discord.gg/aBcDeFgH will be present in content_list twice. Once as an invite filter and once as a domain filter (..and in this order from my testing. This is why we continue to the next iteration of the loop if it's domain filter and the content of the filter matches the pattern of a guild invite.

Before this change, at the point where we encounter continue, `conte...

vivekashok1221

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

The former.

This change fixe the first bug I've mentioned in the PR description.
Invite links of the form https://discord.gg/aBcDeFgH will be present in content_list twice. Once as an invite filter and once as a domain filter (..and in this order from my testing. This is why we continue to the next iteration of the loop if it's domain filter and the content of the filter matches the pattern of a guild invite.

Before this change, at the point where we encounter continue, `conte...

vivekashok1221

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

            # If the user has already been banned, do not attempt to add phishing button since the URL or guild invite
            # is probably already added as a filter

something like this?

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

Fair, not something worth changing in this PR at least.

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

Right, right. Thanks for explaining, that makes sense.

wookie184

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

that looks good to me 👍

wookie184

[python-discord/bot] New comment on issue #1621: Truncate response for embedding lines from github insted of silently failing.

File preview on mobile still isn't a thing, so I think we should go ahead with the originally proposed behaviour.

wookie184

[python-discord/bot] New comment on issue #1802: Remove or update code snippets on message edit

This seems fine to me, though i'm not 100% sure it's worth the added complexity for a fairly rare case. I don't mind too much either way so if someone else vouches for this being a useful feature I think it would be ok to add.

wookie184

[python-discord/bot] New comment on issue #2264: Use link buttons for showing full output, if truncated, of the eval command

This sounds like a good idea to me, we already have a delete button, so this would reduce the screen space taken up by the eval message.

wookie184

[python-discord/bot] New comment on issue #1697: Remove line numbers from !eval command output.

I wouldn't mind trialling removing the numbers and see whether people notice/care.

wookie184

[python-discord/bot] Issue closed: #2253 Wonky syntax highlighting

wookie184

[python-discord/bot] New comment on issue #2253: Wonky syntax highlighting

I don't think this is enough of an issue to be worth fixing.

wookie184

[python-discord/bot] Issue closed: #2363 Remove 3.10 as an option to run as.

wookie184

[python-discord/bot] New comment on issue #2363: Remove 3.10 as an option to run as.

This was done somewhere, currently we only support 3.12

wookie184

[python-discord/bot] Issue closed: #1911 Upgrade the test suite mocks

wookie184

[python-discord/bot] New comment on issue #1911: Upgrade the test suite mocks

I think it's fine to only extend our mocks as needed.

[regal arch]

wookie184

[python-discord/bot] Issue closed: #1957 YTDL tag does not state the same as rule 5

wookie184

[python-discord/bot] New comment on issue #1957: YTDL tag does not state the same as rule 5

The ytdl tag is still accurate, and I think it's fine to leave as is. It shouldn't really be used other than in cases where it's obvious the person is working on ytdl as in more non-obvious cases (e.g. if they were working on a project using ytld earlier and you think they still are) the decision on whether it's ok should probably be left to a moderator.

[regal arch]

wookie184

[python-discord/bot] Issue opened: #3015 Avoid deleting reminders silently

Currently a reminder will be ignored+deleted if the bot cannot get the channel it was meant to reply in:
https://github.com/python-discord/bot/blob/324cb65b5e2fbc52cb494f02a11dc40e56f0a5cb/bot/exts/utils/reminders.py#L112-L124

This could happen if the remind command was run in a thread, or a channel that was deleted.

There are a couple of options here:

• Don't allow the command in threads, or always send the reminder to e.g. #bot-commands
• Send the reminder to e.g. #bot-commands...

wookie184

[python-discord/bot] Issue closed: #1916 Missing Reminders

wookie184

[python-discord/bot] New comment on issue #1916: Missing Reminders

I think the original issue here was fixed, though I noticed another way reminders may go missing, which i've opened a new issue for https://github.com/python-discord/bot/issues/3015.

wookie184

[python-discord/bot] Issue closed: #663 Adding extra functionality and refinements to !remind

wookie184

[python-discord/bot] New comment on issue #663: Adding extra functionality and refinements to !remind

The points here other than "Repeating reminders" and "DM reminders" seem to have been completed or have their own tracking issues. I don't think repeating/dm reminders are super important so i'll close this and let anyone create separate issues for them if they have a use case.

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3008 Bump deepdiff from 6.7.1 to 7.0.1

wookie184

[python-discord/bot] Pull request closed: #3008 Bump deepdiff from 6.7.1 to 7.0.1

wookie184

[bot:main] 1 new commit

ac69865 Bump deepdiff from 6.7.1 to 7.0.1 (#3008) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

Python Bot

Connected!

[regal arch]

jb3

[metricity:main] 1 new commit

f726f85 Move namespace to bots - jb3

jb3

[metricity:main] 1 new commit

2f42c64 Update deployment workflow to new namespace - jb3

[night lily]

Sir Lancebot

Connected!

[thin oyster]

Sir Robin

Connected!

[regal arch]

jb3

[bot:main] 1 new commit

8376d65 Update deployment namespace - jb3

jb3

[sir-lancebot:main] 1 new commit

2a6f144 Update deployment namespace - jb3

jb3

[sir-robin:main] 1 new commit

7e536fc Update deployment namespace - jb3

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

[thin oyster]

Sir Robin

Connected!

[regal arch]

jb3

[king-arthur:main] 1 new commit

fa22342 Update build-deploy.yaml - jb3

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 8681306279 succeeded.

Actor

jb3

Workflow Run

Status embed #330

Commit

fa22342

[regal arch]

vivekashok1221

[bot:vivek/fix-phishing-button] 1 new commit

dce89a9 Update comment for clarity on skipping adding o... - vivekashok1221

[odd spire]

[python-discord/bot] Checks Successful on PR: #2982 Fix case sensitivity bug in phishing button

GitHub Actions run 8681561710 succeeded.

PR Author

vivekashok1221

Workflow Run

CI #1727

Source Branch

vivek/fix-phishing-button

[regal arch]

vivekashok1221

[python-discord/bot] New review comment on pull request #2982: Fix case sensitivity bug in phishing button

https://github.com/python-discord/bot/pull/2982/commits/dce89a9ab3be38a4f88fab449a7158c6c75989fc

[regal arch]

jb3

[snekbox:main] 1 new commit

052e88c Update deployment namespace - jb3

[snekbox] GitHub Actions checks success on main

jb3

[snekbox:main] 1 new commit

435fcad Update deployment namespace in CI - jb3

[snekbox] GitHub Actions checks success on main

jb3

[bot:main] 1 new commit

2e369d6 Update snekbox namespace - jb3

[oak estuary]

Python Bot

Connected!

[regal arch]

jb3

[forms-backend:main] 1 new commit

3ce2704 Update deployment namespace in manifest - jb3

jb3

[forms-backend:main] 1 new commit

59dd269 Update deployment namespace in CI - jb3

[odd spire]

[python-discord/forms-backend] Workflow Failed

GitHub Actions run 8687711919 failed.

Actor

jb3

Workflow Run

Forms Backend #653

Commit

3ce2704

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 8687715323 succeeded.

Actor

jb3

Workflow Run

Forms Backend #654

Commit

59dd269

[regal arch]

jb3

[sir-robin:main] 1 new commit

592e9ae Update URL for code-jam-management - jb3

[thin oyster]

Sir Robin

Connected!

[regal arch]

jb3

[site:main] 1 new commit

330faa9 Update new web namespace in ALLOWED_HOSTS - jb3

jb3

[site:main] 1 new commit

a032933 Update deployment manifest with new namespace - jb3

[odd spire]

[python-discord/site] Workflow Cancelled

GitHub Actions run 8687953373 was cancelled.

Actor

jb3

Workflow Run

CI #1132

Commit

330faa9

[regal arch]

[snekbox] GitHub Actions checks failure on main

[snekbox] GitHub Actions checks failure on main

[odd spire]

[python-discord/site] Workflow Failed

GitHub Actions run 8687959127 failed.

Actor

jb3

Workflow Run

CI #1133

Commit

a032933

[regal arch]

jb3

[site:main] 1 new commit

0dd8de1 Update deployment manifest path - jb3

jb3

[snekbox:main] 1 new commit

d386948 Correct namespace in deployment stage - jb3

[snekbox] GitHub Actions checks success on main

wookie184

[python-discord/bot] Pull request review submitted: #2982 Fix case sensitivity bug in phishing button

Looks good

jb3

[bot:main] 1 new commit

8383d5c Update site namespace in constants - jb3

[oak estuary]

Python Bot

Connected!

[regal arch]

[snekbox] GitHub Actions checks success on main

[night lily]

Sir Lancebot

Connected!

[regal arch]

jb3

[olli:main] 1 new commit

e91786e Update deployment manifest with new namespace - jb3

jb3

[olli:main] 1 new commit

0a271cd Update CI with new namespace - jb3

[odd spire]

[python-discord/bot] Checks Successful on PR: #3016 Bump aiohttp from 3.9.3 to 3.9.4

GitHub Actions run 8688748774 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1730

Source Branch

dependabot/pip/aiohttp-3.9.4

[regal arch]

jchristgit

[site:netcup-link-add-ref] 1 new commit

a8e6c74 Add our reference to Netcup links - jchristgit

jchristgit

[python-discord/site] New branch created: netcup-link-add-ref

jchristgit

[python-discord/site] Pull request opened: #1297 Add our reference to Netcup links

[odd spire]

[python-discord/site] Checks Successful on PR: #1297 Add our reference to Netcup links

GitHub Actions run 8689012029 succeeded.

PR Author

jchristgit

Workflow Run

CI #1135

Source Branch

netcup-link-add-ref

[regal arch]

ChrisLovering

[python-discord/site] Pull request review submitted: #1297 Add our reference to Netcup links

GriceTurrble

[python-discord/bot] Pull request review submitted: #2982 Fix case sensitivity bug in phishing button

GriceTurrble

[bot:vivek/fix-phishing-button] 53 new commits

288cc09 Move sentry initialisation to after the event l... - ChrisLovering
0b224f2 Merge pull request #2981 from python-discord/in... - shtlrs
7df12bf Enable Sentry performance tracing - ChrisLovering
42558cf Bump markdownify from 0.11.6 to 0.12.1 (#2983) - dependabot[bot]
d757301 Reference the /tag slash-command instead in tag... - hedyhli

[odd spire]

[python-discord/bot] Checks Successful on PR: #2982 Fix case sensitivity bug in phishing button

GitHub Actions run 8689173252 succeeded.

PR Author

vivekashok1221

Workflow Run

CI #1731

Source Branch

vivek/fix-phishing-button

[regal arch]

jb3

[bot:main] 1 new commit

4dcaefa Update metabase namespace - jb3

[oak estuary]

Python Bot

Connected!

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

[thin oyster]

Sir Robin

Connected!

[regal arch]

jb3

[bot:main] 1 new commit

3633f92 Update Redis service address - jb3

jb3

[sir-robin:main] 1 new commit

fdf29d4 Update Redis service address - jb3

jb3

[sir-lancebot:main] 1 new commit

9f1d8fe Update Redis service address - jb3

[thin oyster]

Sir Robin

Connected!

[oak estuary]

Python Bot

Connected!

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1298 Bump sqlparse from 0.4.4 to 0.5.0

GitHub Actions run 8697151821 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1136

Source Branch

dependabot/pip/sqlparse-0.5.0

[regal arch]

sentry-io[bot]

[python-discord/sir-lancebot] Issue opened: #1513 TypeError: can't compare offset-naive and offset-aware datetimes

Sentry Issue: SIR-LANCEBOT-BV

TypeError: can't compare offset-naive and offset-aware datetimes
  File "bot/exts/utilities/emoji.py", line 111, in info_command
    **Created:** {time_since(emoji.created_at.replace(tzinfo=None), precision="hours")}
  File "bot/utils/time.py", line 79, in time_since
    delta = abs(relativedelta(now, past_datetime))

Unhandled command error: can't compare offset-naive and off...

[regal arch]

wookie184

[python-discord/site] Pull request review submitted: #1298 Bump sqlparse from 0.4.4 to 0.5.0

wookie184

[python-discord/site] Pull request closed: #1298 Bump sqlparse from 0.4.4 to 0.5.0

wookie184

[site:main] 1 new commit

e766cff Bump sqlparse from 0.4.4 to 0.5.0 (#1298) - dependabot[bot]

[odd spire]

[python-discord/site] Checks Successful on PR: #1296 Bump ruff from 0.3.5 to 0.3.7

GitHub Actions run 8704235585 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1138

Source Branch

dependabot/pip/ruff-0.3.7

[regal arch]

wookie184

[python-discord/sir-lancebot] New comment on issue #143: Independence Day

Anyone is free to work on it. It can be started again from scratch, or the progress in the linked closed PR can be built upon, either way considering the review comments on that linked PR.

wookie184

[python-discord/sir-lancebot] New comment on pull request #1466: Added "christmasify" avatar effect command.

Ideally we'd use assets that don't require attribution, since that's the easiest option where possible.

For ones that do require attribution, I think it's OK to put it in the command's help text.

[regal arch]

wookie184

[python-discord/site] New comment on issue #1269: Event scheduling system

Personally I think I would be in favour of the second option, defining event files using markdown/YAML.

A few benefits I think this has over the first option:

• It's a lot simpler to implement and maintain.
• I think Discord commands could end up being a bit fiddly to use and it could be hard to remember how to do different things. With GitHub it should be more obvious, just add a new file, using a template/existing event for reference. It would also be much easier to do things like mak...

wookie184

[python-discord/sir-lancebot] New comment on issue #1495: Add random graph trivia questions

Thanks for writing up a detailed issue. I think adding graphviz for this is fine, and is the best option here. Marked as approved, feel free to work on this.

wookie184

[python-discord/bot] New comment on pull request #2869: Branding: improve IO resilience

The tenacity lib is licensed under Apache 2.0 which should permit us to use it, but I think we may have to declare it in the third party license declaration file. Could you confirm whether that's the right approach? The file does not currently mention the Apache license.

I'm not an expert on licenses, but I don't think that should be necessary, the third party lice...

wookie184

[python-discord/site] Pull request review submitted: #1296 Bump ruff from 0.3.5 to 0.3.7

wookie184

[python-discord/site] Pull request review submitted: #1296 Bump ruff from 0.3.5 to 0.3.7

wookie184

[python-discord/site] Pull request closed: #1296 Bump ruff from 0.3.5 to 0.3.7

wookie184

[site:main] 1 new commit

8000a1b Bump ruff from 0.3.5 to 0.3.7 (#1296) - dependabot[bot]

wookie184

[python-discord/bot] Pull request review submitted: #3016 Bump aiohttp from 3.9.3 to 3.9.4

wookie184

[python-discord/bot] New comment on pull request #3016: Bump aiohttp from 3.9.3 to 3.9.4

@dependabot rebase

wookie184

[python-discord/bot] Pull request closed: #3016 Bump aiohttp from 3.9.3 to 3.9.4

wookie184

[bot:main] 1 new commit

dfb5800 Bump aiohttp from 3.9.3 to 3.9.4 (#3016) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3016 Bump aiohttp from 3.9.3 to 3.9.4

GitHub Actions run 8704969971 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1734

Source Branch

dependabot/pip/aiohttp-3.9.4

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #2845: Add a cog to manage tracking a user's alt accounts

I think that the current type isn't very useful ,str | int | list[int] makes the usages below technically incorrect since whenever accessing a value they should handle it being any of those types.

I would be find with any of these, since they're all "correct", it just depends what you consider in-scope for the PR given our typing elsewhere with infractions isn't great either afaik:

1. list[dict[str, typing.Any]]
2. Use a TypedDict
3. Use a Pydantic model

[regal arch]

wookie184

[python-discord/bot] New comment on issue #999: Empty data for watched user

I agree with your theory. It looks like there's no need for it to be a defaultdict, so I'll try just making it a dict

[regal arch]

kwzrd

[python-discord/bot] New comment on pull request #2869: Branding: improve IO resilience

@wookie184 Makes sense, thanks for the answer. I'll work on the suggested improvements.

[regal arch]

wookie184

[bot:fix-watchchannel-bugs] 4 new commits

7d84cb8 Make watched_users a dict, not a defaultdict - wookie184
fb21018 Check if user is watched before sending message... - wookie184
36f335f Pass watch info through to functions to avoid K... - wookie184
bfd9e44 Fix: dict.values() -> dict.items() - wookie184

wookie184

[python-discord/bot] New branch created: fix-watchchannel-bugs

wookie184

[python-discord/bot] Pull request opened: #3017 Fix watchchannel bugs

Fixes 3 related issues, details in commit messages.

Closes #999
Closes #1068
Closes #1184

I decided not to typehint more precisely than dict because the rest of the cog doesn't do so and I wanted to keep this change as focused as possible.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3017 Fix watchchannel bugs

GitHub Actions run 8707606594 succeeded.

PR Author

wookie184

Workflow Run

CI #1736

Source Branch

fix-watchchannel-bugs

[regal arch]

ChrisLovering

[python-discord/bot] Pull request review submitted: #3017 Fix watchchannel bugs

wookie184

[python-discord/bot] Issue closed: #1230 403 not being caught in send_attachments

wookie184

[python-discord/bot] New comment on issue #1230: 403 not being caught in send_attachments

Since this doesn't seem to have come up since and I don't see how this should happen I'm going to assume this was a Discord bug. We can investigate further if it comes up again (the sentry issue is deleted now so it's hard to investigate this much further here).

[regal arch]

wookie184

[bot:vivek/fix-phishing-button] 4 new commits

4dcaefa Update metabase namespace - jb3
3633f92 Update Redis service address - jb3
dfb5800 Bump aiohttp from 3.9.3 to 3.9.4 (#3016) - dependabot[bot]
1780886 Merge branch 'main' into vivek/fix-phishing-button - wookie184

wookie184

[python-discord/bot] Pull request closed: #2982 Fix case sensitivity bug in phishing button

wookie184

[bot:main] 8 new commits

7569771 Fix case sensitivity bug in phishing button - vivekashok1221
3996fa6 Update regex to discard trailing ')' and lowerc... - vivekashok1221
ea10480 Skip adding phishing button if user already banned - vivekashok1221
69bc65b Use regex to detect Discord invite link - vivekashok1221
dce89a9 Update comment for clarity on skipping adding o... - vivekashok1221

wookie184

[python-discord/bot] branch deleted: vivek/fix-phishing-button

[odd spire]

[python-discord/bot] Checks Successful on PR: #2982 Fix case sensitivity bug in phishing button

GitHub Actions run 8708666342 succeeded.

PR Author

vivekashok1221

Workflow Run

CI #1737

Source Branch

vivek/fix-phishing-button

[regal arch]

GriceTurrble

[python-discord/bot] New branch created: feat/tag-list-iterate-and-remove-gotcha

[oak estuary]

Python Bot

Connected!

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

913f240 feat: tag loop-remove - GriceTurrble

GriceTurrble

[python-discord/bot] Pull request opened: #3018 feat: tag loop-remove

Details the gotcha of removing items from a collection while iterating that collection inside a for loop.

Drafted up, ended up a bit lengthy. Would appreciate some proofreading.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8708731393 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1739

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

jchristgit

[python-discord/site] Pull request closed: #1297 Add our reference to Netcup links

jchristgit

[site:main] 2 new commits

a8e6c74 Add our reference to Netcup links - jchristgit
064b695 Merge pull request #1297 from python-discord/ne... - jchristgit

jchristgit

[python-discord/site] branch deleted: netcup-link-add-ref

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

8b56a83 fix: condense some whitespace around code blocks - GriceTurrble

wookie184

[python-discord/bot] Pull request review submitted: #2973 Button for others to opt-in to be notified for reminders

This is all great, I have no suggestions. Nice work 💯

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709158502 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1740

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

8e51e49 fix: further condense first code block - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709233690 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1741

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

52107ec fix: further condense the second block - GriceTurrble

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

5b32632 fix: remove a line - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709266070 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1742

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709279070 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1743

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

791b334 fix: bold for list comp name, clean up some lan... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709304959 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1744

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #3018: feat: tag loop-remove

Technically it's not the for loop tracking it, it's the list's iterator. I don't think this really matters, but I also think it's not too hard to word it to avoid suggesting that. I'm also not sure how much "pointer" will mean to someone without knowledge in a language like C. Maybe something along the lines of this would be clearer?

While iterating with a for loop, an index tracks the current position in the list. If the list is modified, this index may no longer refer to the...

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

585f98e fix: correct language for how the iterator does... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709485055 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1745

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

ae2a703 fix: slightly condensed language - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709589609 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1746

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

d3e2538 fix: include "adding" context, and add aliases ... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709637527 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1747

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

9c0f936 fix: text-only explanation - GriceTurrble

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 1 new commit

9e72c6d fix: "in the example", less terse and less conf... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709755417 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1748

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8709770894 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1749

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

wookie184

[python-discord/bot] Issue closed: #2488 KeyError on webhooks when running botstrap

wookie184

[python-discord/bot] New comment on issue #2488: KeyError on webhooks when running botstrap

I think this was fixed anyway as a side effect of https://github.com/python-discord/bot/commit/b30475b1fbabbf970dfe1af4801152f70d917080

jchristgit

[python-discord/forms-backend] Issue opened: #273 PostgreSQL support

Description

We'd like to evaluate whether we can support using or switch to using PostgreSQL as the forms database altogether.

Justification

At DevOps, we try to keep the services we need to run our infrastructure to a minimum, and would like to focus on running PostgreSQL as our single source of truth in the future, especially because we don't really have MongoDB expertise on hand.

Implementation

A possible, but very big alternative might be integrating the forms backend int...

[odd spire]

[python-discord/site] Checks Successful on PR: #1299 Bump gunicorn from 21.2.0 to 22.0.0

GitHub Actions run 8715395519 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1141

Source Branch

dependabot/pip/gunicorn-22.0.0

[regal arch]

wookie184

[python-discord/site] Pull request review submitted: #1299 Bump gunicorn from 21.2.0 to 22.0.0

wookie184

[python-discord/site] Pull request closed: #1299 Bump gunicorn from 21.2.0 to 22.0.0

wookie184

[site:main] 1 new commit

17355ae Bump gunicorn from 21.2.0 to 22.0.0 (#1299) - dependabot[bot]

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #2330: Abbreviate automated messages

@swfarnsworth Are you still interested in recreating this PR?

[odd spire]

[python-discord/bot] Checks Successful on PR: #3019 Bump aiohttp from 3.9.4 to 3.9.5

GitHub Actions run 8722030739 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1750

Source Branch

dependabot/pip/aiohttp-3.9.5

[python-discord/bot] Checks Successful on PR: #3020 Bump regex from 2023.12.25 to 2024.4.16

GitHub Actions run 8722064421 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1751

Source Branch

dependabot/pip/regex-2024.4.16

[regal arch]

wookie184

[python-discord/bot] New comment on issue #2559: Temporary bans are incorrectly assumed permanent when infractions overlap

After some internal discussion I think we want to keep the current behaviour of not allowing a tempban to override a permaban. It's just the message sent that needs fixing.

wookie184

[bot:fix-tempban-on-permaban] 1 new commit

ea6661d Use correct key to check if infraction has expiry - wookie184

wookie184

[python-discord/bot] New branch created: fix-tempban-on-permaban

wookie184

[python-discord/bot] Pull request opened: #3021 Use correct key to check if infraction has expiry

Closes #2559

Preserve existing behaviour of not allowing a permaban to override a tempban

[odd spire]

[python-discord/bot] Checks Successful on PR: #3021 Use correct key to check if infraction has expiry

GitHub Actions run 8725379774 succeeded.

PR Author

wookie184

Workflow Run

CI #1752

Source Branch

fix-tempban-on-permaban

[regal arch]

wookie184

[python-discord/bot] Issue closed: #855 Discord Channel Logs Improvements

wookie184

[python-discord/bot] New comment on issue #855: Discord Channel Logs Improvements

I don't think there's mutch actionable here at the moment, so i'll close this issue.

[regal arch]

wookie184

[python-discord/bot] New comment on issue #1960: Improvements to `helpdm` and a feature request for help channel notifications.

Helpdm was removed in https://github.com/python-discord/bot/pull/2968 so i'll close this issue.

Any suggestions for helpdm-like features that would still be relevant given we're now using forum channels can be put in https://github.com/python-discord/bot/issues/2975

wookie184

[python-discord/bot] Issue closed: #1960 Improvements to `helpdm` and a feature request for help channel notifications.

wookie184

[python-discord/bot] New comment on issue #1964: Unfurl Discord links in more places

Is this still something we want? Discord links on Android are much improved now (for me at least).

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3019 Bump aiohttp from 3.9.4 to 3.9.5

wookie184

[python-discord/bot] Pull request closed: #3019 Bump aiohttp from 3.9.4 to 3.9.5

wookie184

[bot:main] 1 new commit

1be6a41 Bump aiohttp from 3.9.4 to 3.9.5 (#3019) - dependabot[bot]

wookie184

[python-discord/bot] Pull request review submitted: #3020 Bump regex from 2023.12.25 to 2024.4.16

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request closed: #3020 Bump regex from 2023.12.25 to 2024.4.16

wookie184

[bot:main] 1 new commit

4d6781c Bump regex from 2023.12.25 to 2024.4.16 (#3020) - dependabot[bot]

[odd spire]

[python-discord/bot] Workflow Cancelled

GitHub Actions run 8729617278 was cancelled.

Actor

wookie184

Workflow Run

CI #1753

Commit

1be6a41

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] New comment on issue #2692: Automatically close help thread on ban

I'll go ahead with that, and can get more thoughts from people reviewing the PR.

wookie184

[bot:help-thread-owner-leave-message] 1 new commit

3c7701a Notify help threads if the owner is no longer o... - wookie184

wookie184

[python-discord/bot] New branch created: help-thread-owner-leave-message

wookie184

[python-discord/bot] Pull request opened: #3022 Notify help threads if the owner is no longer on the server

Closes #2692

Happy to discuss if you think a different behaviour (e.g. closing automatically) would still be better.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3022 Notify help threads if the owner is no longer on the server

GitHub Actions run 8736475264 succeeded.

PR Author

wookie184

Workflow Run

CI #1756

Source Branch

help-thread-owner-leave-message

[regal arch]

wookie184

[python-discord/bot] New comment on issue #1004: Antispam attempts to create duplicate DeletedMessages in the DB

The background for this issue is no longer relevant given the filters rewrite, but the problem still exists https://python-discord.sentry.io/issues/4251592323/

ResponseCodeError: Status: 400 Response: {'deletedmessage_set': [{'id': ['deleted message with this ID already exists.']}]}
  File "pydis_core/utils/scheduling.py", line 257, in _coro_wrapper
    await coro
  File "bot/exts/filtering/_filter_lists/antispam.py", line 131, in process_deletion_context
    await deletion_co...

[regal arch]

wookie184

[python-discord/bot] New comment on issue #2963: Automatically apply pardoned timeouts upon rejoining the server

This should probably just require adding something like

elif <user_has_timeout>:
    <remove_user_timeout>

here https://github.com/python-discord/bot/blob/3c7701a64c708ad1c4acf0c8604400721aee830c/bot/exts/moderation/infraction/infractions.py#L650-L672

[regal arch]

wookie184

[python-discord/bot] New comment on issue #779: Syncer and on_member_join race condition for adding user data to API.

There's a similar issue in the on_member_join method (https://python-discord.sentry.io/issues/5216780974) though it seems to be due to the on_member_join method being duplicated.

I think the best solution in both of these cases is to silently ignore the error. We know in both cases the user was only just created so we can assume it's up to date, and relying on the database is much simpler and more reliable than working out our own locking solution.

IMO it's fine to rely on "already ...

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 3 new commits

1be6a41 Bump aiohttp from 3.9.4 to 3.9.5 (#3019) - dependabot[bot]
4d6781c Bump regex from 2023.12.25 to 2024.4.16 (#3020) - dependabot[bot]
165c1f9 Merge branch 'main' into feat/tag-list-iterate-... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8738611067 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1757

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

Ribudna

[python-discord/bot] Issue opened: #3023 from telegram.ext import Updater, CommandHandler # Функция, которая будет вызываться при команде /apply def apply_to_group(update, context): c...

jb3

[python-discord/bot] Issue closed: #3023 from telegram.ext import Updater, CommandHandler # Функция, которая будет вызываться при команде /apply def apply_to_group(update, context): c...

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1514 Bump aiohttp from 3.9.3 to 3.9.4

GitHub Actions run 8741341019 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4007

Source Branch

dependabot/pip/aiohttp-3...

[regal arch]

BruDriguezz

[python-discord/bot] Pull request opened: #3024 Remove trailing line from System Python tag

As per @import-pandas-as-numpy's request, this PR removes the trailing line on the System Python's tag.

[odd spire]

[python-discord/bot] Checks Failed on PR: #3024 Remove trailing line from System Python tag

GitHub Actions run 8741714663 failed.

PR Author

BruDriguezz

Workflow Run

CI #1758

Source Branch

BruDriguezz:main

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #3024: Remove trailing line from System Python tag

The trailing newline is necessary for linting and shouldn't affect how the tag looks. I believe the suggestion was to remove some of the empty lines in the tag (e.g. between bullet points).

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1514 Bump aiohttp from 3.9.3 to 3.9.4

wookie184

[python-discord/sir-lancebot] Pull request closed: #1514 Bump aiohttp from 3.9.3 to 3.9.4

wookie184

[sir-lancebot:main] 1 new commit

a2dfaf0 Bump aiohttp from 3.9.3 to 3.9.4 (#1514) - dependabot[bot]

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1300 Bump pymdown-extensions from 10.7.1 to 10.8

GitHub Actions run 8748674037 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1143

Source Branch

dependabot/pip/pymdown-extensions-...

[python-discord/site] Checks Successful on PR: #1301 Bump ruff from 0.3.7 to 0.4.0

GitHub Actions run 8748683710 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1144

Source Branch

dependabot/pip/ruff-0.4.0

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1515 Bump ruff from 0.3.7 to 0.4.0

GitHub Actions run 8752195584 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4009

Source Branch

dependabot/pip/ruff-0.4.0

[regal arch]

import-pandas-as-numpy

[python-discord/bot] New comment on pull request #3024: Remove trailing line from System Python tag

The trailing newline is necessary for linting and shouldn't affect how the tag looks. I believe the suggestion was to remove some of the empty lines in the tag (e.g. between bullet points).

@BruDriguezz
This is correct. Apologies if that wasn't clear. The embed takes up a significant amount of white space when posted:

This may be a result of Discord's native list formatting (I'm u...

[regal arch]

jchristgit

[python-discord/site] Pull request review submitted: #1301 Bump ruff from 0.3.7 to 0.4.0

jchristgit

[python-discord/site] Pull request review submitted: #1300 Bump pymdown-extensions from 10.7.1 to 10.8

jchristgit

[python-discord/site] Pull request closed: #1300 Bump pymdown-extensions from 10.7.1 to 10.8

jchristgit

[site:main] 2 new commits

1d052d3 Bump pymdown-extensions from 10.7.1 to 10.8 - dependabot[bot]
c2e781f Merge pull request #1300 from python-discord/de... - jchristgit

[odd spire]

[python-discord/site] Checks Successful on PR: #1301 Bump ruff from 0.3.7 to 0.4.0

GitHub Actions run 8752691664 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1146

Source Branch

dependabot/pip/ruff-0.4.0

[odd spire]

[python-discord/bot] Checks Successful on PR: #3025 Bump ruff from 0.3.7 to 0.4.1

GitHub Actions run 8753979004 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1759

Source Branch

dependabot/pip/ruff-0.4.1

[regal arch]

wookie184

[python-discord/bot] Issue closed: #2368 Add a !mypy command

wookie184

[python-discord/bot] New comment on issue #2368: Add a !mypy command

Closing as a duplicate of https://github.com/python-discord/bot/issues/1880, since it's still not approved I think it makes sense to keep discussion in once place.

wookie184

[python-discord/bot] New comment on issue #1154: Add projects subcommands to source command

We have the .github command on lancebot:

I don't think it's worth keeping separate custom markdown descriptions here. I don't think it would add much extra value to the info from .github, and could take a decent amount of work to keep up to date.

[regal arch]

wookie184

[python-discord/bot] New comment on issue #2564: ClientException: Parent channel not found

Seems to be because the channel was deleted but this didn't remove the thread from guild.threads.

Reported as an issue in discord.py: https://github.com/Rapptz/discord.py/issues/9795, we should be able to close this when the linked PR there is merged.

[regal arch]

vivekashok1221

[python-discord/bot] Pull request review submitted: #3021 Use correct key to check if infraction has expiry

Looks good to me :star2:

wookie184

[python-discord/bot] Pull request review submitted: #3025 Bump ruff from 0.3.7 to 0.4.1

wookie184

[python-discord/bot] Pull request closed: #3025 Bump ruff from 0.3.7 to 0.4.1

wookie184

[bot:main] 1 new commit

0d82308 Bump ruff from 0.3.7 to 0.4.1 (#3025) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[regal arch]

vivax3794

[python-discord/bot] New comment on pull request #3018: feat: tag loop-remove

I made a gif for this as discussed on discord, you ofc have my full permission to use it. it has a similar quality/fps to the return-gif one (actually slightly bigger).

[regal arch]

wookie184

[python-discord/bot] Issue closed: #1154 Add projects subcommands to source command

[regal arch]

GriceTurrble

[bot:feat/tag-list-iterate-and-remove-gotcha] 2 new commits

0d82308 Bump ruff from 0.3.7 to 0.4.1 (#3025) - dependabot[bot]
2822896 Merge branch 'main' into feat/tag-list-iterate-... - GriceTurrble

[odd spire]

[python-discord/bot] Checks Successful on PR: #3018 feat: tag loop-remove

GitHub Actions run 8769795233 succeeded.

PR Author

GriceTurrble

Workflow Run

CI #1761

Source Branch

feat/tag-list-iterate-and-remove-gotcha

[regal arch]

wookie184

[python-discord/site] Pull request closed: #1301 Bump ruff from 0.3.7 to 0.4.0

wookie184

[site:main] 1 new commit

c21698a Bump ruff from 0.3.7 to 0.4.0 (#1301) - dependabot[bot]

[odd spire]

[python-discord/site] Checks Successful on PR: #1302 Bump ruff from 0.4.0 to 0.4.1

GitHub Actions run 8779140230 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1148

Source Branch

dependabot/pip/ruff-0.4.1

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1302 Bump ruff from 0.4.0 to 0.4.1

Xithrius

[python-discord/site] Pull request closed: #1302 Bump ruff from 0.4.0 to 0.4.1

Xithrius

[site:main] 2 new commits

9ee5fbf Bump ruff from 0.4.0 to 0.4.1 - dependabot[bot]
c1ebc4a Merge pull request #1302 from python-discord/de... - Xithrius

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1516 Bump ruff from 0.3.7 to 0.4.1

GitHub Actions run 8782728076 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4010

Source Branch

dependabot/pip/ruff-0.4.1

[python-discord/sir-lancebot] Checks Successful on PR: #1517 Bump emoji from 2.11.0 to 2.11.1

GitHub Actions run 8782731860 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4011

Source Branch

dependabot/pip/emoji-2.11.1

[odd spire]

[python-discord/bot] Checks Successful on PR: #3026 Bump emoji from 2.11.0 to 2.11.1

GitHub Actions run 8783703299 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1762

Source Branch

dependabot/pip/emoji-2.11.1

[regal arch]

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1517 Bump emoji from 2.11.0 to 2.11.1

wookie184

[python-discord/sir-lancebot] Pull request closed: #1517 Bump emoji from 2.11.0 to 2.11.1

wookie184

[sir-lancebot:main] 1 new commit

f6cafc0 Bump emoji from 2.11.0 to 2.11.1 (#1517) - dependabot[bot]

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1516 Bump ruff from 0.3.7 to 0.4.1

GitHub Actions run 8789614383 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4012

Source Branch

dependabot/pip/ruff-0.4.1

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3026 Bump emoji from 2.11.0 to 2.11.1

wookie184

[python-discord/bot] Pull request closed: #3026 Bump emoji from 2.11.0 to 2.11.1

wookie184

[bot:main] 1 new commit

cbea647 Bump emoji from 2.11.0 to 2.11.1 (#3026) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3000 Add feature to append additional text to nomination reasons

I haven't tested yet, but a few comments.

wookie184

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

        old_reason = next((e.reason for e in nomination_entries if e.actor_id == nominator.id), None)

next will raise StopIteration, not return None if nominator didn't nominate the user.

wookie184

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

        if old_reason:
            add_period = not old_reason.endswith((".", "!", "?"))
            reason = old_reason + (". " if add_period else " ") + reason

If no reason was given for the nomination, old_reason will be an empty string. In that case, reason should be left as is.

Also, I don't think reason can be None, if no reason was given it should be an empty string. It doesn't make sense to append nothing though so maybe we should handle that case and...

wookie184

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

I think get_nomination will raise an error if the id doesn't exist instead of returning None.

[odd spire]

[python-discord/site] Checks Successful on PR: #1303 Bump coverage from 7.4.4 to 7.5.0

GitHub Actions run 8811488349 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1150

Source Branch

dependabot/pip/coverage-7.5.0

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1303 Bump coverage from 7.4.4 to 7.5.0

Xithrius

[python-discord/site] Pull request closed: #1303 Bump coverage from 7.4.4 to 7.5.0

Xithrius

[site:main] 2 new commits

867f8ee Bump coverage from 7.4.4 to 7.5.0 - dependabot[bot]
24f2cd4 Merge pull request #1303 from python-discord/de... - Xithrius

[odd spire]

[python-discord/bot] Checks Successful on PR: #3027 Bump coverage from 7.4.4 to 7.5.0

GitHub Actions run 8816409239 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1764

Source Branch

dependabot/pip/coverage-7.5.0

[regal arch]

hedyhli

[python-discord/bot] New review comment on pull request #3022: Notify help threads if the owner is no longer on the server

A slight nit - the rest looks good as a code review 👍

Looking at the one instance this was needed from the linked issue, it may not always (or even likely) be the owner deliberately leaving the server rather than getting a ban in such cases. How's this instead?

                await thread.send(":warning: The owner of this post is no longer in the server.")

[regal arch]

Xithrius

[python-discord/bot] Pull request review submitted: #3027 Bump coverage from 7.4.4 to 7.5.0

Xithrius

[python-discord/bot] Pull request closed: #3027 Bump coverage from 7.4.4 to 7.5.0

Xithrius

[bot:main] 1 new commit

e04474d Bump coverage from 7.4.4 to 7.5.0 (#3027) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1518 Bump sentry-sdk from 1.45.0 to 2.0.0

GitHub Actions run 8830802592 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4014

Source Branch

dependabot/pip/sentry-sd...

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #3022: Notify help threads if the owner is no longer on the server

Yeah, that's better 👍

wookie184

[bot:help-thread-owner-leave-message] 1 new commit

91a5353 Change message to account for banned users - wookie184

[odd spire]

[python-discord/bot] Checks Successful on PR: #3022 Notify help threads if the owner is no longer on the server

GitHub Actions run 8830832789 succeeded.

PR Author

wookie184

Workflow Run

CI #1766

Source Branch

help-thread-owner-leave-message

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #3022: Notify help threads if the owner is no longer on the server

Actually i'll change the log above too so it matches.

wookie184

[bot] Branch help-thread-owner-leave-message was force-pushed to `239ba92`

Compare changes

wookie184

[python-discord/bot] New review comment on pull request #3022: Notify help threads if the owner is no longer on the server

done

[odd spire]

[python-discord/bot] Checks Successful on PR: #3022 Notify help threads if the owner is no longer on the server

GitHub Actions run 8830944987 succeeded.

PR Author

wookie184

Workflow Run

CI #1767

Source Branch

help-thread-owner-leave-message

[odd spire]

[python-discord/bot] Checks Successful on PR: #3028 Bump sentry-sdk from 1.45.0 to 2.0.0

GitHub Actions run 8832248054 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1768

Source Branch

dependabot/pip/sentry-sdk-2.0.0

[regal arch]

SockYeh

[python-discord/sir-lancebot] New comment on pull request #1466: Added "christmasify" avatar effect command.

I am unable to find images so would be grateful if someone could find one to add to the bot. Thanks.

[regal arch]

vivekashok1221

[python-discord/bot] Pull request review submitted: #3022 Notify help threads if the owner is no longer on the server

LGTM :star2:

[regal arch]

BruDriguezz

[python-discord/bot] Pull request closed: #3024 Remove trailing line from System Python tag

[regal arch]

BruDriguezz

[python-discord/bot] Pull request opened: #3029 Update system-python.md: Improve bullet point formatting

This PR removes unnecessary newlines between bullet points, reducing overall size and improving clarity.

[odd spire]

[python-discord/bot] Checks Failed on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8843193310 failed.

PR Author

BruDriguezz

Workflow Run

CI #1769

Source Branch

BruDriguezz:fix-system-python-tag

[python-discord/bot] Checks Successful on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8843231370 succeeded.

PR Author

BruDriguezz

Workflow Run

CI #1770

Source Branch

BruDriguezz:fix-system-python-tag

[odd spire]

[python-discord/bot] Checks Successful on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8843329276 succeeded.

PR Author

BruDriguezz

Workflow Run

CI #1771

Source Branch

BruDriguezz:fix-system-python-tag

[odd spire]

[python-discord/site] Checks Successful on PR: #1304 Bump sentry-sdk from 1.45.0 to 2.0.0

GitHub Actions run 8843538635 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1152

Source Branch

dependabot/pip/sentry-sdk-2.0.0

[python-discord/site] Checks Successful on PR: #1305 Bump ruff from 0.4.1 to 0.4.2

GitHub Actions run 8843541708 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1153

Source Branch

dependabot/pip/ruff-0.4.2

[odd spire]

[python-discord/bot] Checks Failed on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8843685272 failed.

PR Author

BruDriguezz

Workflow Run

CI #1772

Source Branch

BruDriguezz:fix-system-python-tag

[python-discord/bot] Checks Successful on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8843720892 succeeded.

PR Author

BruDriguezz

Workflow Run

CI #1773

Source Branch

BruDriguezz:fix-system-python-tag

[regal arch]

jgyasu

[python-discord/sir-lancebot] New comment on issue #1477: Option to send public valentines privately through DMs

Hello, I would like to work on this issue. This would be my first time contributing to open-source so I would be grateful for some help.

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1519 Bump sentry-sdk from 1.45.0 to 2.0.1

GitHub Actions run 8846608103 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4015

Source Branch

dependabot/pip/sentry-sd...

[python-discord/sir-lancebot] Checks Failed on PR: #1520 Bump ruff from 0.3.7 to 0.4.2

GitHub Actions run 8846614057 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4016

Source Branch

dependabot/pip/ruff-0.4.2

[odd spire]

[python-discord/bot] Checks Successful on PR: #3030 Bump ruff from 0.4.1 to 0.4.2

GitHub Actions run 8848086014 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1774

Source Branch

dependabot/pip/ruff-0.4.2

[python-discord/bot] Checks Successful on PR: #3031 Bump sentry-sdk from 1.45.0 to 2.0.1

GitHub Actions run 8848092954 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1775

Source Branch

dependabot/pip/sentry-sdk-2.0.1

[night lily]

Sir Lancebot

Connected!

[thin oyster]

Sir Robin

Connected!

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

[regal arch]

3ddP

[python-discord/sir-lancebot] Issue opened: #1521 latex preamble update part 2: fun with TikZ

Description

The latex preamble determines which packages/commands are available to the user when writing .tex input for .latex command. This proposal involves adding fun TikZ-related packages to the latex preamble, which is located in sir-lancebot/bot/resources/fun/latex_template.txt. The proposed packages are:

• tikzducks, which allows the creation of (modestly cursed) ducks with a simple command
• [tikz-among-us](https://ctan.org/pkg/tikz-amon...

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1305 Bump ruff from 0.4.1 to 0.4.2

Xithrius

[python-discord/site] Pull request closed: #1305 Bump ruff from 0.4.1 to 0.4.2

Xithrius

[site:main] 2 new commits

2829b6a Bump ruff from 0.4.1 to 0.4.2 - dependabot[bot]
6bae48a Merge pull request #1305 from python-discord/de... - Xithrius

Xithrius

[python-discord/site] New comment on pull request #1304: Bump sentry-sdk from 1.45.0 to 2.0.0

Hopefully nothing explodes

Xithrius

[python-discord/site] Pull request review submitted: #1304 Bump sentry-sdk from 1.45.0 to 2.0.0

Xithrius

[python-discord/bot] Pull request review submitted: #3031 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[python-discord/bot] Pull request closed: #3031 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[bot:main] 1 new commit

bdb22af Bump sentry-sdk from 1.45.0 to 2.0.1 (#3031) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request review submitted: #3030 Bump ruff from 0.4.1 to 0.4.2

Xithrius

[python-discord/sir-lancebot] Pull request review submitted: #1519 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[python-discord/sir-lancebot] Pull request closed: #1519 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[sir-lancebot:main] 1 new commit

d14a231 Bump sentry-sdk from 1.45.0 to 2.0.1 (#1519) - dependabot[bot]

Xithrius

[python-discord/site] Pull request review submitted: #1306 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[python-discord/sir-lancebot] New comment on pull request #1520: Bump ruff from 0.3.7 to 0.4.2

@dependabot recreate

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1520 Bump ruff from 0.3.7 to 0.4.2

GitHub Actions run 8859124637 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4017

Source Branch

dependabot/pip/ruff-0.4.2

[regal arch]

Xithrius

[python-discord/site] Pull request closed: #1306 Bump sentry-sdk from 1.45.0 to 2.0.1

Xithrius

[site:main] 2 new commits

d796ae3 Bump sentry-sdk from 1.45.0 to 2.0.1 - dependabot[bot]
fb06d31 Merge pull request #1306 from python-discord/de... - Xithrius

Xithrius

[python-discord/bot] Pull request closed: #3030 Bump ruff from 0.4.1 to 0.4.2

Xithrius

[bot:main] 1 new commit

102c5b2 Bump ruff from 0.4.1 to 0.4.2 (#3030) - dependabot[bot]

[odd spire]

[python-discord/site] Checks Successful on PR: #1306 Bump sentry-sdk from 1.45.0 to 2.0.1

GitHub Actions run 8859113826 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1155

Source Branch

dependabot/pip/sentry-sdk-2.0.1

[python-discord/site] Workflow Cancelled

GitHub Actions run 8859108439 was cancelled.

Actor

Xithrius

Workflow Run

CI #1154

Commit

6bae48a

[python-discord/bot] Workflow Cancelled

GitHub Actions run 8859113726 was cancelled.

Actor

Xithrius

Workflow Run

CI #1776

Commit

bdb22af

[regal arch]

Xithrius

[python-discord/bot] New comment on pull request #3029: Update system-python.md: Improve bullet point formatting

Hello,

Has this been discussed with a core developer, or is there an open issue for this? If not, please see this page on how to contribute to this repository.

Thanks!

[oak estuary]

Python Bot

Connected!

[regal arch]

Xithrius

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

I'm for the auto-close feature, but not instant. Perhaps we could send a message saying that the thread will close in a minute instead? Such that if a user gets banned within a thread, messages can still go through for a bit.

In addition to that, perhaps an :x: could be added to the sent message to prevent the thread from closing. Might be out of scope for this PR though.

Xithrius

[python-discord/bot] Pull request review submitted: #3021 Use correct key to check if infraction has expiry

:shipit:

Xithrius

[bot:fix-tempban-on-permaban] 8 new commits

1be6a41 Bump aiohttp from 3.9.4 to 3.9.5 (#3019) - dependabot[bot]
4d6781c Bump regex from 2023.12.25 to 2024.4.16 (#3020) - dependabot[bot]
0d82308 Bump ruff from 0.3.7 to 0.4.1 (#3025) - dependabot[bot]
cbea647 Bump emoji from 2.11.0 to 2.11.1 (#3026) - dependabot[bot]
e04474d Bump coverage from 7.4.4 to 7.5.0 (#3027) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request closed: #3021 Use correct key to check if infraction has expiry

Xithrius

[bot:main] 1 new commit

161370e Use correct key to check if infraction has expi... - wookie184

Xithrius

[python-discord/bot] Issue closed: #2559 Temporary bans are incorrectly assumed permanent when infractions overlap

Xithrius

[python-discord/bot] branch deleted: fix-tempban-on-permaban

[odd spire]

[python-discord/bot] Checks Successful on PR: #3021 Use correct key to check if infraction has expiry

GitHub Actions run 8859196630 succeeded.

PR Author

wookie184

Workflow Run

CI #1779

Source Branch

fix-tempban-on-permaban

[oak estuary]

Python Bot

Connected!

[regal arch]

Xithrius

[python-discord/bot] Pull request review submitted: #3013 Removed rich embed filter as its no longer necessary.

yep yep yep

Xithrius

[python-discord/bot] New comment on pull request #3013: Removed rich embed filter as its no longer necessary.

LGTM, thanks.

I'll remove the filter on the server just before merging, once this has another approval from someone.

I have removed the filter.

Xithrius

[python-discord/bot] Pull request closed: #3013 Removed rich embed filter as its no longer necessary.

Xithrius

[bot:main] 1 new commit

a47d9ed Removed rich embed filter as its no longer nece... - Hashed0719

Xithrius

[python-discord/bot] Issue closed: #2977 Remove `rich_embed` filter

Xithrius

[python-discord/bot] Issue closed: #1938 Handle an edge case in the rich embed filter.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3013 Removed rich embed filter as its no longer necessary.

GitHub Actions run 8859253653 succeeded.

PR Author

Hashed0719

Workflow Run

CI #1781

Source Branch

Hashed0719:main

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

[regal arch]

BruDriguezz

[python-discord/bot] New comment on pull request #3029: Update system-python.md: Improve bullet point formatting

Hello,

Has this been discussed with a core developer, or is there an open issue for this? If not, please see this page on how to contribute to this repository.

Thanks!

Hello,
This PR addresses an issue previously raised by @import-pandas-as-numpy. I had opened a similar PR earlier but closed it due to repository sync problems.

[regal arch]

Xithrius

[python-discord/bot] New comment on pull request #3029: Update system-python.md: Improve bullet point formatting

Ah, perfect. Thanks!

Xithrius

[python-discord/bot] Pull request review submitted: #3029 Update system-python.md: Improve bullet point formatting

Xithrius

[python-discord/bot] Pull request closed: #3029 Update system-python.md: Improve bullet point formatting

Xithrius

[bot:main] 1 new commit

5e2efb5 Update system-python.md: Improve bullet point f... - BruDriguezz

[odd spire]

[python-discord/bot] Checks Successful on PR: #3029 Update system-python.md: Improve bullet point formatting

GitHub Actions run 8860687914 succeeded.

PR Author

BruDriguezz

Workflow Run

CI #1783

Source Branch

BruDriguezz:fix-system-python-tag

[oak estuary]

Python Bot

Connected!

[regal arch]

vivekashok1221

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

I'm for the auto-close feature, but not instant. Perhaps we could send a message saying that the thread will close in a minute instead? Such that if a user gets banned within a thread, messages can still go through for a bit.

In addition to that, perhaps an ❌ could be added to the sent message to prevent the thread from closing. Might be out of scope for this PR though.

I'm mostly not in favour of closing the help thread unless the helpee has been banned- but I can probably be con...

[regal arch]

ZARA7X

[python-discord/bot] Issue opened: #3032 ALVEER BOT ON

Bot

[regal arch]

wookie184

[python-discord/bot] Issue closed: #3032 ALVEER BOT ON

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

The initial reasoning behind the issue was to remove the confusion on where the user went if someone left/was banned, and I think sending a message is the simplest and least intrusive way of solving that.

I don't think auto-closing is really worth the extra complexity given it's such an edge case (and it could potentially be unwanted in some cases) unless there's a reason for it separate to the original motivation of the issue.

Xithrius

[python-discord/bot] Pull request review submitted: #3022 Notify help threads if the owner is no longer on the server

True. Just the message will do then. If we really want to add the auto-closing feature, we can always do it later.

Xithrius

[bot:help-thread-owner-leave-message] 9 new commits

0d82308 Bump ruff from 0.3.7 to 0.4.1 (#3025) - dependabot[bot]
cbea647 Bump emoji from 2.11.0 to 2.11.1 (#3026) - dependabot[bot]
e04474d Bump coverage from 7.4.4 to 7.5.0 (#3027) - dependabot[bot]
bdb22af Bump sentry-sdk from 1.45.0 to 2.0.1 (#3031) - dependabot[bot]
102c5b2 Bump ruff from 0.4.1 to 0.4.2 (#3030) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request closed: #3022 Notify help threads if the owner is no longer on the server

Xithrius

[bot:main] 1 new commit

bbfb333 Notify help threads if the owner is no longer o... - wookie184

Xithrius

[python-discord/bot] Issue closed: #2692 Automatically close help thread on ban

Xithrius

[python-discord/bot] branch deleted: help-thread-owner-leave-message

[odd spire]

[python-discord/bot] Checks Successful on PR: #3022 Notify help threads if the owner is no longer on the server

GitHub Actions run 8871229928 succeeded.

PR Author

wookie184

Workflow Run

CI #1785

Source Branch

help-thread-owner-leave-message

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1307 Bump pymdown-extensions from 10.8 to 10.8.1

GitHub Actions run 8873539978 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1157

Source Branch

dependabot/pip/pymdown-extensions-...

[odd spire]

[python-discord/site] Checks Successful on PR: #1308 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

GitHub Actions run 8873668337 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1158

Source Branch

dependabot/github_actions/HassanAb...

[regal arch]

ChrisLovering

[python-discord/site] Pull request closed: #1308 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

ChrisLovering

[site:main] 1 new commit

67e5a7d Bump HassanAbouelela/actions from setup-python_... - dependabot[bot]

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1307 Bump pymdown-extensions from 10.8 to 10.8.1

Xithrius

[python-discord/site] New comment on pull request #1307: Bump pymdown-extensions from 10.8 to 10.8.1

@dependabot recreate

Xithrius

[python-discord/site] Pull request closed: #1307 Bump pymdown-extensions from 10.8 to 10.8.1

Xithrius

[site:main] 2 new commits

6f00bde Bump pymdown-extensions from 10.8 to 10.8.1 - dependabot[bot]
ad6910c Merge pull request #1307 from python-discord/de... - Xithrius

[odd spire]

[python-discord/site] Checks Successful on PR: #1307 Bump pymdown-extensions from 10.8 to 10.8.1

GitHub Actions run 8875119908 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1160

Source Branch

dependabot/pip/pymdown-extensions-...

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1522 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

GitHub Actions run 8877388713 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4019

Source Branch

dependabot/github_action...

[regal arch]

Xithrius

[python-discord/sir-lancebot] Pull request review submitted: #1522 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

Xithrius

[python-discord/sir-lancebot] Pull request closed: #1522 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

Xithrius

[sir-lancebot:main] 1 new commit

8a28fda Bump HassanAbouelela/actions from setup-python_... - dependabot[bot]

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3033 Bump pytest from 8.1.1 to 8.2.0

GitHub Actions run 8878196828 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1787

Source Branch

dependabot/pip/pytest-8.2.0

[python-discord/bot] Checks Successful on PR: #3034 Bump pytest-xdist from 3.5.0 to 3.6.1

GitHub Actions run 8878207319 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1788

Source Branch

dependabot/pip/pytest-xdist-3.6.1

[python-discord/bot] Checks Successful on PR: #3035 Bump regex from 2024.4.16 to 2024.4.28

GitHub Actions run 8878218680 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1789

Source Branch

dependabot/pip/regex-2024.4.28

[odd spire]

[python-discord/bot] Checks Successful on PR: #3036 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

GitHub Actions run 8878568825 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1790

Source Branch

dependabot/github_actions/HassanAb...

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3033 Bump pytest from 8.1.1 to 8.2.0

wookie184

[python-discord/bot] Pull request closed: #3033 Bump pytest from 8.1.1 to 8.2.0

wookie184

[bot:main] 1 new commit

fba297d Bump pytest from 8.1.1 to 8.2.0 (#3033) - dependabot[bot]

wookie184

[python-discord/bot] Pull request review submitted: #3036 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3034 Bump pytest-xdist from 3.5.0 to 3.6.1

wookie184

[python-discord/bot] Pull request closed: #3034 Bump pytest-xdist from 3.5.0 to 3.6.1

wookie184

[bot:main] 1 new commit

da6e7cb Bump pytest-xdist from 3.5.0 to 3.6.1 (#3034) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3035 Bump regex from 2024.4.16 to 2024.4.28

GitHub Actions run 8884431786 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1793

Source Branch

dependabot/pip/regex-2024.4.28

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3035 Bump regex from 2024.4.16 to 2024.4.28

wookie184

[python-discord/bot] Pull request review submitted: #3035 Bump regex from 2024.4.16 to 2024.4.28

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request closed: #3035 Bump regex from 2024.4.16 to 2024.4.28

wookie184

[bot:main] 1 new commit

dc21c17 Bump regex from 2024.4.16 to 2024.4.28 (#3035) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[regal arch]

ChrisLovering

[python-discord/bot] Pull request closed: #3036 Bump HassanAbouelela/actions from setup-python_v1.4.2 to 1.5.0

ChrisLovering

[bot:main] 1 new commit

7d079d9 Bump HassanAbouelela/actions from setup-python_... - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[regal arch]

thatbirdguythatuknownot

[python-discord/bot] Issue opened: #3037 Newline count special-casing

Due to a recent incident, it is very important to note the need to special-case codeblocks' newline count.

It is important to note that, as shown in the image, a codeblock renders about 2 extra lines worth of space compared to a regular text line. It is also recommended to special-case headers, as they make text a lot larger than usual (although I haven't computed it yet).

[regal arch]

Kajo11

[python-discord/bot] New comment on commit `7d079d9`

how draw heart

Kajo11

[python-discord/bot] New comment on commit `7d079d9`

how draw heart

[regal arch]

ChrisLovering

[python-discord/bot] New comment on issue #3037: Newline count special-casing

There's quite a lot of features of this bot that interact with codeblocks, so this issue doesn't give us much to go off of for potentially fixing what you believe to be incorrect.

Could you specify where we should special case this count?

It might be easier if you provide reproduction steps, what the expected behaviour is, and what the current behaviour is.

If you believe this is an exploitable bug and shouldn't be discussed publicly, please follow our [security notice](https://githu...

ChrisLovering

[python-discord/bot] Pull request closed: #2908 Move url regex to constants

ChrisLovering

[python-discord/bot] New comment on pull request #2908: Move url regex to constants

I'm not going to pursue this further

ChrisLovering

[python-discord/bot] branch deleted: move-url-regex-to-constants

ChrisLovering

[bot] Branch feat/1353/alt-users was force-pushed to `98eea89`

Compare changes

ChrisLovering

[python-discord/bot] Pull request review submitted: #2845 Add a cog to manage tracking a user's alt accounts

ChrisLovering

[python-discord/bot] New review comment on pull request #2845: Add a cog to manage tracking a user's alt accounts

I have corrected my accent in https://github.com/python-discord/bot/pull/2845/commits/98eea891c25f8beb7a99685fe1daa023654d2866

ChrisLovering

[python-discord/bot] New review comment on pull request #2845: Add a cog to manage tracking a user's alt accounts

Simplified the type annotation in https://github.com/python-discord/bot/pull/2845/commits/a380d2e030cf3f206e5e4d359db228e06cac81f5 (option 1)

ChrisLovering

[python-discord/bot] New review comment on pull request #2845: Add a cog to manage tracking a user's alt accounts

We use 0-based indexing around here!

[odd spire]

[python-discord/bot] Checks Successful on PR: #2845 Add a cog to manage tracking a user's alt accounts

GitHub Actions run 8891160562 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1798

Source Branch

feat/1353/alt-users

[regal arch]

shtlrs

[python-discord/bot] Pull request review submitted: #2845 Add a cog to manage tracking a user's alt accounts

ChrisLovering

[python-discord/bot] New branch created: feat/1037/push-raw-to-pastebin-if-too-long

ChrisLovering

[bot:feat/1037/push-raw-to-pastebin-if-too-long] 1 new commit

f2e5d01 Push raw to pastebin if too long for Discord - ChrisLovering

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `112c234`

Compare changes

ChrisLovering

[python-discord/bot] Pull request opened: #3038 Push raw to pastebin if too long for Discord

This also removes the use of a paginator entirely.

Closes #1037

Examples

Short messages

Before:

After:

Long messages

Before:
Error, see sentry issue linked above
After:

Whi...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 8891567017 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1799

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[regal arch]

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `c3e6740`

Compare changes

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `3f2eb47`

Compare changes

[odd spire]

[python-discord/bot] Checks Cancelled on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 8891595671 was cancelled.

PR Author

ChrisLovering

Workflow Run

CI #1800

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[python-discord/bot] Checks Successful on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 8891601095 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1801

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[regal arch]

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `10032e6`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 8891653852 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1802

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[regal arch]

thatbirdguythatuknownot

[python-discord/bot] New comment on issue #3037: Newline count special-casing

There was recently an incident with spam in #python-discussion where a player sent a lot of backticks that turned into codeblocks that flooded the chat. I think the relevant code is here.

[regal arch]

ChrisLovering

[python-discord/bot] New comment on issue #3037: Newline count special-casing

So just to be clear then, this issue is related to the new lines filter potentially not working correctly for messages with backticks?

thatbirdguythatuknownot

[python-discord/bot] New comment on issue #3037: Newline count special-casing

Yup.

[regal arch]

hedyhli

[site:feat/timeline-from-yaml] 1 new commit

69a4115 Timeline: Migrate content - hedyhli

[odd spire]

[python-discord/site] Checks Successful on PR: #1293 Fetch timeline entries from Markdown files with YAML frontmatter

GitHub Actions run 8936827140 succeeded.

PR Author

hedyhli

Workflow Run

CI #1162

Source Branch

feat/timeline-from-yaml

[regal arch]

hedyhli

[site:feat/timeline-from-yaml] 27 new commits

30753c9 Bump sentry-sdk from 1.44.1 to 1.45.0 - dependabot[bot]
e2b0090 Merge pull request #1294 from python-discord/de... - jchristgit
a90bc0e Bump idna from 3.6 to 3.7 - dependabot[bot]
c27b112 Merge pull request #1295 from python-discord/de... - hedyhli
330faa9 Update new web namespace in ALLOWED_HOSTS - jb3

[odd spire]

[python-discord/site] Checks Successful on PR: #1293 Fetch timeline entries from Markdown files with YAML frontmatter

GitHub Actions run 8936936336 succeeded.

PR Author

hedyhli

Workflow Run

CI #1163

Source Branch

feat/timeline-from-yaml

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1523 Bump rapidfuzz from 3.8.1 to 3.9.0

GitHub Actions run 8937556284 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4021

Source Branch

dependabot/pip/rapidfuzz...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3039 Bump rapidfuzz from 3.8.1 to 3.9.0

GitHub Actions run 8939331017 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1803

Source Branch

dependabot/pip/rapidfuzz-3.9.0

[regal arch]

jchristgit

[python-discord/site] New comment on pull request #1293: Markdown + YAML for the timeline

This is still great !!!!!!!!!!!!!

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1309 Bump coverage from 7.5.0 to 7.5.1

GitHub Actions run 8964295328 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1164

Source Branch

dependabot/pip/coverage-7.5.1

[python-discord/site] Checks Successful on PR: #1310 Bump ruff from 0.4.2 to 0.4.3

GitHub Actions run 8964298681 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1165

Source Branch

dependabot/pip/ruff-0.4.3

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1524 Bump ruff from 0.3.7 to 0.4.3

GitHub Actions run 8967613684 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4022

Source Branch

dependabot/pip/ruff-0.4.3

[python-discord/sir-lancebot] Checks Successful on PR: #1525 Bump sentry-sdk from 2.0.1 to 2.1.0

GitHub Actions run 8967617596 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4023

Source Branch

dependabot/pip/sentry-sd...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3040 Bump sentry-sdk from 2.0.1 to 2.1.1

GitHub Actions run 8969145245 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1804

Source Branch

dependabot/pip/sentry-sdk-2.1.1

[python-discord/bot] Checks Successful on PR: #3041 Bump coverage from 7.5.0 to 7.5.1

GitHub Actions run 8969158664 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1805

Source Branch

dependabot/pip/coverage-7.5.1

[python-discord/bot] Checks Successful on PR: #3042 Bump ruff from 0.4.2 to 0.4.3

GitHub Actions run 8969170825 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1806

Source Branch

dependabot/pip/ruff-0.4.3

[regal arch]

jahrules

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

@wookie184 today I saw the bot post the departed message twice. Any idea why?

https://discord.com/channels/267624335836053506/1237066212593242163

[regal arch]

Xithrius

[python-discord/sir-lancebot] Pull request review submitted: #1523 Bump rapidfuzz from 3.8.1 to 3.9.0

Xithrius

[python-discord/sir-lancebot] Pull request closed: #1523 Bump rapidfuzz from 3.8.1 to 3.9.0

Xithrius

[sir-lancebot:main] 1 new commit

7bd936f Bump rapidfuzz from 3.8.1 to 3.9.0 (#1523) - dependabot[bot]

Xithrius

[python-discord/site] Pull request review submitted: #1310 Bump ruff from 0.4.2 to 0.4.3

Xithrius

[python-discord/site] Pull request closed: #1310 Bump ruff from 0.4.2 to 0.4.3

Xithrius

[site:main] 2 new commits

060dae9 Bump ruff from 0.4.2 to 0.4.3 - dependabot[bot]
eab5be4 Merge pull request #1310 from python-discord/de... - Xithrius

Xithrius

[python-discord/site] Pull request review submitted: #1309 Bump coverage from 7.5.0 to 7.5.1

[night lily]

Sir Lancebot

Connected!

[regal arch]

Xithrius

[python-discord/site] Pull request closed: #1309 Bump coverage from 7.5.0 to 7.5.1

Xithrius

[site:main] 2 new commits

9c7381d Bump coverage from 7.5.0 to 7.5.1 - dependabot[bot]
d5707c5 Merge pull request #1309 from python-discord/de... - Xithrius

[odd spire]

[python-discord/site] Checks Successful on PR: #1309 Bump coverage from 7.5.0 to 7.5.1

GitHub Actions run 8972809688 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1167

Source Branch

dependabot/pip/coverage-7.5.1

[python-discord/site] Workflow Cancelled

GitHub Actions run 8972800007 was cancelled.

Actor

Xithrius

Workflow Run

CI #1166

Commit

eab5be4

[python-discord/sir-lancebot] Checks Failed on PR: #1524 Bump ruff from 0.3.7 to 0.4.3

GitHub Actions run 8972849691 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4026

Source Branch

dependabot/pip/ruff-0.4.3

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

@wookie184 today I saw the bot post the departed message twice. Any idea why?

https://discord.com/channels/267624335836053506/1237066212593242163

[ ![Screenshot_20240506-115343](https://private-user-images.githubusercontent.com/63260884/328239882-91dc5bea-4b5a-48f1-9e7d-8696c3dee845.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTUwMTg5MjksIm5iZiI6MTcxNTAxODYyOSwicGF0aCI6Ii82MzI2...

wookie184

[python-discord/bot] Issue closed: #2564 ClientException: Parent channel not found

wookie184

[python-discord/bot] New comment on issue #2564: ClientException: Parent channel not found

Upstream issue was fixed so should be sorted in the next discord.py release.

wookie184

[python-discord/sir-lancebot] New comment on pull request #1519: Bump sentry-sdk from 1.45.0 to 2.0.1

We should make sure we aren't using any of the methods deprecated in this major release, in the migration guide (https://docs.sentry.io/platforms/python/migration/1.x-to-2.x) push_scope is mentioned as being deprecated but we still use it here: https://github.com/python-discord/sir-lancebot/blob/7bd936f50668791d404358bcd74fb82b6e6656f6/bot/exts/core/error_handler.py#L8

wookie184

[python-discord/bot] New comment on pull request #3031: Bump sentry-sdk from 1.45.0 to 2.0.1

Same as mentioned on the lancebot PR, we should make sure we aren't using any of the methods deprecated in this major release, in the migration guide (https://docs.sentry.io/platforms/python/migration/1.x-to-2.x). We use push_scope in a few places but it is now deprecated.

wookie184

[python-discord/bot] New comment on issue #2938: A context menu command to nominate users

Hi @BruDriguezz, are you still interested in working on this?

wookie184

[python-discord/bot] Pull request review submitted: #3042 Bump ruff from 0.4.2 to 0.4.3

wookie184

[python-discord/bot] Pull request closed: #3042 Bump ruff from 0.4.2 to 0.4.3

wookie184

[bot:main] 1 new commit

b60085b Bump ruff from 0.4.2 to 0.4.3 (#3042) - dependabot[bot]

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3040 Bump sentry-sdk from 2.0.1 to 2.1.1

GitHub Actions run 8973794764 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1809

Source Branch

dependabot/pip/sentry-sdk-2.1.1

[regal arch]

jahrules

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

Or maybe a timeout? Could someone theoretically use this behavior to time
the bot out by leaving and rejoing real fast?

On Mon, May 6, 2024, 2:06 PM wookie184 @.***> wrote:

@wookie184 https://github.com/wookie184 today I saw the bot post the
departed message twice. Any idea why?

https://discord.com/channels/267624335836053506/1237066212593242163

[image: Screenshot_20240506-115343] http://url

It looks like they left, rejoined, and then left again pretty ...

[regal arch]

vivekashok1221

[python-discord/bot] New comment on pull request #3022: Notify help threads if the owner is no longer on the server

Or maybe a timeout? Could someone theoretically use this behavior to time the bot out by leaving and rejoing real fast?

I think the bot should be able to handle it. We already log whenever a user joins or leaves the server in a mod-only log channel. This additional message in the help thread shouldn't be much of a concern.

[regal arch]

ChrisLovering

[king-arthur:main] 2 new commits

5290e4b Bump all dependencies to latest - ChrisLovering
7140ac2 Update devops rules cog due to new infra repo l... - ChrisLovering

ChrisLovering

[python-discord/king-arthur] Pull request closed: #176 build(deps): bump idna from 3.6 to 3.7

ChrisLovering

[python-discord/king-arthur] Pull request closed: #179 build(deps): bump aiohttp from 3.9.3 to 3.9.4

ChrisLovering

[python-discord/king-arthur] Pull request closed: #181 build(deps): bump pydantic from 2.6.4 to 2.7.1

ChrisLovering

[python-discord/king-arthur] Pull request closed: #185 build(deps): bump sentry-sdk from 1.45.0 to 2.1.1

ChrisLovering

[python-discord/king-arthur] Pull request closed: #186 build(deps-dev): bump ruff from 0.3.5 to 0.4.3

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 8977129386 succeeded.

Actor

ChrisLovering

Workflow Run

Status embed #339

Commit

7140ac2

[regal arch]

ChrisLovering

[king-arthur:main] 2 new commits

bbee723 Update setup-python CI step to 1.5.0 - ChrisLovering
f7e2d72 Bump kubernetes CI steps to latest version - ChrisLovering

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 8977146470 succeeded.

Actor

ChrisLovering

Workflow Run

Status embed #340

Commit

f7e2d72

[regal arch]

ChrisLovering

[king-arthur:main] 1 new commit

8cf5684 Update devops rules parsing - ChrisLovering

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 8977194767 succeeded.

Actor

ChrisLovering

Workflow Run

Status embed #341

Commit

8cf5684

[odd spire]

[python-discord/site] Checks Successful on PR: #1311 Bump django from 5.0.4 to 5.0.5

GitHub Actions run 8980177749 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1169

Source Branch

dependabot/pip/django-5.0.5

[python-discord/site] Checks Successful on PR: #1312 Bump sentry-sdk from 2.0.1 to 2.1.1

GitHub Actions run 8980183855 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1170

Source Branch

dependabot/pip/sentry-sdk-2.1.1

[regal arch]

BruDriguezz

[python-discord/bot] New comment on issue #2938: A context menu command to nominate users

@wookie184,
My apologies, this issue completely slipped my mind!

I actually made some initial progress on it, but unfortunately, a data corruption issue wiped out all my work. This discouraged me a bit, and I decided to focus on other things with the intention of coming back to it later. Unfortunately, that later never arrived, and it completely fell off my radar.

Given my current workload, I wouldn't be able to dedicate time to this issue in the foreseeable future. If you'd be inter...

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1313 Bump django from 5.0.4 to 5.0.6

GitHub Actions run 8996750493 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1171

Source Branch

dependabot/pip/django-5.0.6

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1312 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[python-discord/site] Pull request closed: #1312 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[site:main] 2 new commits

94510e2 Bump sentry-sdk from 2.0.1 to 2.1.1 - dependabot[bot]
d651dc2 Merge pull request #1312 from python-discord/de... - Xithrius

Xithrius

[python-discord/site] Pull request review submitted: #1313 Bump django from 5.0.4 to 5.0.6

Xithrius

[python-discord/bot] Pull request review submitted: #3041 Bump coverage from 7.5.0 to 7.5.1

Xithrius

[python-discord/bot] Pull request closed: #3041 Bump coverage from 7.5.0 to 7.5.1

Xithrius

[bot:main] 1 new commit

71a7f77 Bump coverage from 7.5.0 to 7.5.1 (#3041) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request review submitted: #3040 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[python-discord/bot] Pull request review submitted: #3039 Bump rapidfuzz from 3.8.1 to 3.9.0

Xithrius

[python-discord/sir-lancebot] Pull request review submitted: #1526 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[python-discord/sir-lancebot] Pull request closed: #1526 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[sir-lancebot:main] 1 new commit

70e6aaf Bump sentry-sdk from 2.0.1 to 2.1.1 (#1526) - dependabot[bot]

Xithrius

[python-discord/site] Pull request closed: #1313 Bump django from 5.0.4 to 5.0.6

Xithrius

[site:main] 2 new commits

ca337f7 Bump django from 5.0.4 to 5.0.6 - dependabot[bot]
3d59587 Merge pull request #1313 from python-discord/de... - Xithrius

[odd spire]

[python-discord/site] Checks Successful on PR: #1313 Bump django from 5.0.4 to 5.0.6

GitHub Actions run 8998587767 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1173

Source Branch

dependabot/pip/django-5.0.6

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/site] Workflow Cancelled

GitHub Actions run 8998575150 was cancelled.

Actor

Xithrius

Workflow Run

CI #1172

Commit

d651dc2

[python-discord/sir-lancebot] Checks Failed on PR: #1524 Bump ruff from 0.3.7 to 0.4.3

GitHub Actions run 8998628780 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4028

Source Branch

dependabot/pip/ruff-0.4.3

[regal arch]

Xithrius

[python-discord/bot] Pull request closed: #3040 Bump sentry-sdk from 2.0.1 to 2.1.1

Xithrius

[bot:main] 1 new commit

4f8aecc Bump sentry-sdk from 2.0.1 to 2.1.1 (#3040) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3040 Bump sentry-sdk from 2.0.1 to 2.1.1

GitHub Actions run 8998635360 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1812

Source Branch

dependabot/pip/sentry-sdk-2.1.1

[python-discord/bot] Checks Successful on PR: #3039 Bump rapidfuzz from 3.8.1 to 3.9.0

GitHub Actions run 8998641477 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1813

Source Branch

dependabot/pip/rapidfuzz-3.9.0

[oak estuary]

Python Bot

Connected!

[regal arch]

Xithrius

[python-discord/bot] Pull request closed: #3039 Bump rapidfuzz from 3.8.1 to 3.9.0

Xithrius

[bot:main] 1 new commit

10a5736 Bump rapidfuzz from 3.8.1 to 3.9.0 (#3039) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3039 Bump rapidfuzz from 3.8.1 to 3.9.0

GitHub Actions run 8998688393 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1815

Source Branch

dependabot/pip/rapidfuzz-3.9.0

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

Python Bot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1314 Bump coverallsapp/github-action from 2.2.3 to 2.3.0

GitHub Actions run 9012992169 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1175

Source Branch

dependabot/github_actions/coverall...

[regal arch]

ChrisLovering

[site:soft-delete-reminders] 1 new commit

571f0c8 Soft-delete reminders on DELETE instead of remo... - ChrisLovering

ChrisLovering

[python-discord/site] New branch created: soft-delete-reminders

ChrisLovering

[python-discord/site] Pull request opened: #1315 Soft-delete reminders on DELETE instead of removing entirely

[odd spire]

[python-discord/site] Checks Failed on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9017149085 failed.

PR Author

ChrisLovering

Workflow Run

CI #1176

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `15efa78`

Compare changes

[odd spire]

[python-discord/site] Checks Failed on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9018699236 failed.

PR Author

ChrisLovering

Workflow Run

CI #1177

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `b2fe248`

Compare changes

[odd spire]

[python-discord/site] Checks Failed on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9018755801 failed.

PR Author

ChrisLovering

Workflow Run

CI #1178

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `91cc0b7`

Compare changes

[odd spire]

[python-discord/site] Checks Failed on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9018787426 failed.

PR Author

ChrisLovering

Workflow Run

CI #1179

Source Branch

soft-delete-reminders

[regal arch]

jb3

[king-arthur:main] 1 new commit

d308f0c Increase number of members returned by GitHub A... - jb3

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 9027086362 succeeded.

Actor

jb3

Workflow Run

Status embed #342

Commit

d308f0c

[odd spire]

[python-discord/site] Checks Successful on PR: #1316 Bump ruff from 0.4.3 to 0.4.4

GitHub Actions run 9027623701 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1180

Source Branch

dependabot/pip/ruff-0.4.4

[regal arch]

Xithrius

[python-discord/site] Pull request review submitted: #1316 Bump ruff from 0.4.3 to 0.4.4

Xithrius

[python-discord/site] Pull request closed: #1316 Bump ruff from 0.4.3 to 0.4.4

Xithrius

[site:main] 2 new commits

ae8c4d1 Bump ruff from 0.4.3 to 0.4.4 - dependabot[bot]
e1ae079 Merge pull request #1316 from python-discord/de... - Xithrius

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `52537fb`

Compare changes

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `e9d441b`

Compare changes

[odd spire]

[python-discord/site] Checks Cancelled on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029438279 was cancelled.

PR Author

ChrisLovering

Workflow Run

CI #1182

Source Branch

soft-delete-reminders

[python-discord/site] Checks Failed on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029450302 failed.

PR Author

ChrisLovering

Workflow Run

CI #1183

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `d88dbca`

Compare changes

[odd spire]

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029751140 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1184

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `0fbaec0`

Compare changes

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `7bce2d0`

Compare changes

[odd spire]

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029788190 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1185

Source Branch

soft-delete-reminders

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029798811 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1186

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `cedb0f1`

Compare changes

[odd spire]

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9029889101 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1187

Source Branch

soft-delete-reminders

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1527 Bump ruff from 0.3.7 to 0.4.4

GitHub Actions run 9030939569 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4029

Source Branch

dependabot/pip/ruff-0.4.4

[odd spire]

[python-discord/bot] Checks Successful on PR: #3043 Bump ruff from 0.4.3 to 0.4.4

GitHub Actions run 9032110463 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1817

Source Branch

dependabot/pip/ruff-0.4.4

[python-discord/bot] Checks Successful on PR: #3044 Bump regex from 2024.4.28 to 2024.5.10

GitHub Actions run 9032115897 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1818

Source Branch

dependabot/pip/regex-2024.5.10

[regal arch]

jchristgit

[python-discord/site] New review comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

        response = self.client.get(url)

jchristgit

[python-discord/site] New review comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

shouldn't this be include_inactive?

ChrisLovering

[king-arthur:main] 2 new commits

39e2077 Reduce the number of github team members fetche... - ChrisLovering
19d25e7 Warn when the max number of members per page is... - ChrisLovering

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `52395f8`

Compare changes

ChrisLovering

[python-discord/site] New review comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

Thanks, sorted in https://github.com/python-discord/site/compare/cedb0f1d724f4f2b9cb99d792331338695c3f7c3..52395f86fa3dd703372c878c739a225c366da98f

ChrisLovering

[python-discord/site] New review comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

Yes, sorted in https://github.com/python-discord/site/compare/cedb0f1d724f4f2b9cb99d792331338695c3f7c3..52395f86fa3dd703372c878c739a225c366da98f

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 9035102471 succeeded.

Actor

ChrisLovering

Workflow Run

Status embed #343

Commit

19d25e7

[regal arch]

ChrisLovering

[site] Branch soft-delete-reminders was force-pushed to `3c1c958`

Compare changes

[odd spire]

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9035098533 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1188

Source Branch

soft-delete-reminders

[regal arch]

ChrisLovering

[python-discord/site] New comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

2nd forcepush was a rebase onto main

[odd spire]

[python-discord/site] Checks Successful on PR: #1315 Soft-delete reminders on DELETE instead of removing entirely

GitHub Actions run 9035113414 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1189

Source Branch

soft-delete-reminders

[regal arch]

jchristgit

[python-discord/site] Pull request review submitted: #1315 Soft-delete reminders on DELETE instead of removing entirely

Ja, Stabil Bruder!

ChrisLovering

[python-discord/site] New comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

Ja, Stabil Bruder!

Du bist der Hammer!

[regal arch]

jchristgit

[python-discord/site] New comment on pull request #1315: Soft-delete reminders on DELETE instead of removing entirely

Du bist der Hammer!

Ja danke Maschine! Abfahrt!

[regal arch]

jchristgit

[python-discord/site] Pull request review submitted: #1314 Bump coverallsapp/github-action from 2.2.3 to 2.3.0

jchristgit

[python-discord/site] Pull request closed: #1314 Bump coverallsapp/github-action from 2.2.3 to 2.3.0

jchristgit

[site:main] 2 new commits

a66331e Bump coverallsapp/github-action from 2.2.3 to 2... - dependabot[bot]
b41b582 Merge pull request #1314 from python-discord/de... - jchristgit

[regal arch]

vivekashok1221

[python-discord/bot] Issue opened: #3045 !eval output and pastebin content are same when the output is 11 lines

If the output generated by snekbox has 11 lines, the output sent to Discord as well as the contents uploaded to our pastebin is the exact same.

This is because we always [truncate the output to 11 lines](https://github.com/python-discord/bot/blob/10a57366f4243ce5d22c6f868f0545f678fe21e3/...

vivekashok1221

[bot:vivek/fix-eval-truncation] 1 new commit

6ccff70 Don't upload to pastebin when no truncation - vivekashok1221

vivekashok1221

[python-discord/bot] New branch created: vivek/fix-eval-truncation

wookie184

[python-discord/bot] New comment on issue #3045: !eval output and pastebin content are same when the output is 11 lines

Is this the same as https://github.com/python-discord/bot/issues/2492?

vivekashok1221

[python-discord/bot] New comment on issue #3045: !eval output and pastebin content are same when the output is 11 lines

I didn't notice that issue, you're right.

[regal arch]

jchristgit

[python-discord/site] Pull request closed: #1315 Soft-delete reminders on DELETE instead of removing entirely

jchristgit

[site:main] 3 new commits

b92aa76 Soft-delete reminders on DELETE instead of remo... - ChrisLovering
3c1c958 Update reminder tests for new soft-deleted beha... - ChrisLovering
f0b3a9b Merge pull request #1315 from python-discord/so... - jchristgit

jchristgit

[python-discord/site] branch deleted: soft-delete-reminders

[regal arch]

Xithrius

[python-discord/bot] Pull request review submitted: #3044 Bump regex from 2024.4.28 to 2024.5.10

Xithrius

[python-discord/bot] Pull request closed: #3044 Bump regex from 2024.4.28 to 2024.5.10

Xithrius

[bot:main] 1 new commit

8fade13 Bump regex from 2024.4.28 to 2024.5.10 (#3044) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request review submitted: #3043 Bump ruff from 0.4.3 to 0.4.4

[oak estuary]

Python Bot

Connected!

[regal arch]

Xithrius

[python-discord/bot] Pull request closed: #3043 Bump ruff from 0.4.3 to 0.4.4

Xithrius

[bot:main] 1 new commit

3e6281d Bump ruff from 0.4.3 to 0.4.4 (#3043) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3043 Bump ruff from 0.4.3 to 0.4.4

GitHub Actions run 9046508825 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1820

Source Branch

dependabot/pip/ruff-0.4.4

[oak estuary]

Python Bot

Connected!

[regal arch]

Xithrius

[python-discord/sir-lancebot] New comment on pull request #1148: Add ability to view previous topics with .topic command

@Anonymous4045 Hello! Any updates on the progress of this PR? Thanks!

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1148 Add ability to view previous topics with .topic command

GitHub Actions run 9046750460 succeeded.

PR Author

Anonymous4045

Workflow Run

Status Embed #4030

Source Branch

Anonymous4045:previous_topics

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1148 Add ability to view previous topics with .topic command

GitHub Actions run 9047753111 succeeded.

PR Author

Anonymous4045

Workflow Run

Status Embed #4031

Source Branch

Anonymous4045:previous_topics

[regal arch]

Anonymous4045

[python-discord/sir-lancebot] New comment on pull request #1148: Add ability to view previous topics with .topic command

@Xithrius thanks for the reminder! I've implemented most of what you said. I also tried switching the suggestion form to be a footer, but this seems to break the hyperlink. Do you know of a way to have links in the footer? We could also just make it in the description, but I think it looks better as a footer for formatting purposes.

Xithrius

[python-discord/sir-lancebot] New comment on pull request #1148: Add ability to view previous topics with .topic command

Thank you for the update!

Footers in embeds do not support hyperlinks, unfortunately. I believe previously the link was just a part of the description.

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1148 Add ability to view previous topics with .topic command

GitHub Actions run 9048030587 failed.

PR Author

Anonymous4045

Workflow Run

Status Embed #4032

Source Branch

Anonymous4045:previous_topics

[regal arch]

Anonymous4045

[python-discord/sir-lancebot] New comment on pull request #1148: Add ability to view previous topics with .topic command

@Xithrius bummer. How's this instead?

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1148 Add ability to view previous topics with .topic command

GitHub Actions run 9048085810 succeeded.

PR Author

Anonymous4045

Workflow Run

Status Embed #4033

Source Branch

Anonymous4045:previous_topics

[regal arch]

Xithrius

[python-discord/sir-lancebot] New comment on pull request #1148: Add ability to view previous topics with .topic command

Looks good to me!

[regal arch]

jb3

[python-discord/bot] New comment on issue #2938: A context menu command to nominate users

I'll grab this one!

[regal arch]

wookie184

[python-discord/bot] Issue closed: #3045 !eval output and pastebin content are same when the output is 11 lines

wookie184

[python-discord/bot] New comment on issue #3045: !eval output and pastebin content are same when the output is 11 lines

I'll close this one and assign you to the other one 👍

vivekashok1221

[python-discord/bot] Pull request opened: #3046 Don't upload to pastebin when no truncation

See issue #3045

New behaviour:

[odd spire]

[python-discord/bot] Checks Successful on PR: #3046 Don't upload to pastebin when no truncation

GitHub Actions run 9054708005 succeeded.

PR Author

vivekashok1221

Workflow Run

CI #1822

Source Branch

vivek/fix-eval-truncation

[regal arch]

jb3

[python-discord/bot] New branch created: jb3/2938-nominate-context-menu

jb3

[bot:jb3/2938-nominate-context-menu] 2 new commits

dd2abe5 Add new Talentpool API method for fetching a no... - jb3
f246afe Add new context menu command for nominating users - jb3

jb3

[python-discord/bot] Pull request opened: #3047 Add context menu command for user nomination

This PR introduces a context menu command that will be available to all Helper+
staff members allowing them to add users to the nomination pool with additional
context.

Upon selecting a message and executing the Nominate User command, the message
jump hyperlink will either be appending to their existing nomination, or a new
nomination will be created with this hyperlink.

The command defaults to being unavailable to all users so upon merge an
Administrator will need to amend the applicatio...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9055257835 succeeded.

PR Author

jb3

Workflow Run

CI #1823

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[python-discord/bot] New comment on pull request #3047: Add context menu command for user nomination

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `6da4bd3`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9055307603 succeeded.

PR Author

jb3

Workflow Run

CI #1824

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

Xithrius

[python-discord/bot] New comment on pull request #3000: Add feature to append additional text to nomination reasons

Hey @Esther-Goldberg, any updates on this PR? Thanks.

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `de90de4`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9055436727 succeeded.

PR Author

jb3

Workflow Run

CI #1825

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `f2cb454`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9055463789 succeeded.

PR Author

jb3

Workflow Run

CI #1826

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[python-discord/bot] New comment on pull request #3047: Add context menu command for user nomination

As mentioned on Discord, context menu commands cannot be created in discord.py
using the decorators when they are a part of a class, there seem to be technical
limitations under the hood of discord.py preventing that.

There is more information available
here
that I will link for reference as to why I have implemented the context menu
commands in a slightly odd/manual way.

[regal arch]

Esther-Goldberg

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

old_reason is the original nomination reason, so if the caller never nominated this user before, it will be None. An error for this will be shown later, in _edit_nomination_reason, so it doesn't really matter what we do here. I'm actually not really sure if reason can ever be None, it can only ever be an empty string. We can fail it in that case I guess.

Esther-Goldberg

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

This is a good point, I'll make that change.

[regal arch]

Esther-Goldberg

[bot:append-nominations] 1 new commit

381daad Apply suggestions from code review - Esther-Goldberg

Esther-Goldberg

[bot:append-nominations] 2 new commits

8dd148e handle 404 error from get_nomination - Esther-Goldberg
becb5f9 Merge branch 'append-nominations' of https://gi... - Esther-Goldberg

[odd spire]

[python-discord/bot] Checks Cancelled on PR: #3000 Add feature to append additional text to nomination reasons

GitHub Actions run 9056296886 was cancelled.

PR Author

Esther-Goldberg

Workflow Run

CI #1827

Source Branch

append-nominations

[regal arch]

Esther-Goldberg

[bot:append-nominations] 1 new commit

4ecc8ef throw error if new reason is not provided - Esther-Goldberg

Esther-Goldberg

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

I added a check for an empty reason earlier, and here I only check if old_reason exists.

[odd spire]

[python-discord/bot] Checks Successful on PR: #3000 Add feature to append additional text to nomination reasons

GitHub Actions run 9056358338 succeeded.

PR Author

Esther-Goldberg

Workflow Run

CI #1829

Source Branch

append-nominations

[odd spire]

[python-discord/site] Checks Successful on PR: #1317 Bump psycopg from 3.1.18 to 3.1.19

GitHub Actions run 9057496013 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1192

Source Branch

dependabot/pip/psycopg-3.1.19

[python-discord/site] Checks Successful on PR: #1318 Bump pre-commit from 3.7.0 to 3.7.1

GitHub Actions run 9057504943 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1193

Source Branch

dependabot/pip/pre-commit-3.7.1

[regal arch]

ChrisLovering

[python-discord/bot] Pull request review submitted: #3047 Add context menu command for user nomination

None of my comments should block this form being merged, looks good!

ChrisLovering

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

nice

ChrisLovering

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

We could early return after this line, rather than having a big else block

ChrisLovering

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

Could be cool if, in the future, we spawn a modal when executing the content menu to capture a reason from the user, which could be pre-filled with their existing resona dn the message jump link.

ChrisLovering

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

This is deprecated as of sentry_sdk 2.0. If you want to change this in this PR then:

            with sentry_sdk.new_scope() as scope:

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1528 Bump lxml from 5.2.1 to 5.2.2

GitHub Actions run 9061664568 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4034

Source Branch

dependabot/pip/lxml-5.2.2

[python-discord/sir-lancebot] Checks Successful on PR: #1529 Bump pre-commit from 3.7.0 to 3.7.1

GitHub Actions run 9061668082 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4035

Source Branch

dependabot/pip/pre-commi...

[regal arch]

ChrisLovering

[python-discord/bot] Pull request review submitted: #3046 Don't upload to pastebin when no truncation

ChrisLovering

[python-discord/bot] Pull request review submitted: #2999 Reply to eval command

[odd spire]

[python-discord/bot] Checks Successful on PR: #3048 Bump pre-commit from 3.7.0 to 3.7.1

GitHub Actions run 9062625068 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1830

Source Branch

dependabot/pip/pre-commit-3.7.1

[python-discord/bot] Checks Successful on PR: #3049 Bump lxml from 5.2.1 to 5.2.2

GitHub Actions run 9062632786 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1831

Source Branch

dependabot/pip/lxml-5.2.2

[regal arch]

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

I'll change it here and add a commit to change it in the other places

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

i hate it as much as you

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

good call

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

It's copied from the other error handling, but I'll add a warning here

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

I don't think we need one on the other one as the messages are not ephemeral

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

We only remove the handler, surely it gets re-added when the setup function is called again?

This does not synchronise through to Discord AFAIK, it just stops erroneous dispatches for commands to cogs that are not loaded.

ChrisLovering

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

Ah right, simple thing to check using the extensions command I guess, would be interested in how it works

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `0aa83c1`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9063189936 succeeded.

PR Author

jb3

Workflow Run

CI #1832

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `414b25f`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9063254345 succeeded.

PR Author

jb3

Workflow Run

CI #1833

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `2c43ff8`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9063295391 succeeded.

PR Author

jb3

Workflow Run

CI #1834

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

ChrisLovering

[python-discord/bot] Pull request review submitted: #3047 Add context menu command for user nomination

Our CI is infallible, so approving without running locally

[regal arch]

ShakyaMajumdar

[python-discord/bot] New comment on pull request #3047: Add context menu command for user nomination

Can we consider relaying a message to #nominations as well?
When one person nominates a member it often prompts other people to add their nominations/anti-nominations too

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `f42666b`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9064428006 succeeded.

PR Author

jb3

Workflow Run

CI #1835

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `d42e3ec`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9064543051 succeeded.

PR Author

jb3

Workflow Run

CI #1836

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `e799849`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9064654832 succeeded.

PR Author

jb3

Workflow Run

CI #1837

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

ChrisLovering

[python-discord/bot] Pull request review submitted: #3047 Add context menu command for user nomination

[regal arch]

jb3

[bot] Branch jb3/2938-nominate-context-menu was force-pushed to `5b209b1`

Compare changes

[odd spire]

[python-discord/bot] Checks Successful on PR: #3047 Add context menu command for user nomination

GitHub Actions run 9065154729 succeeded.

PR Author

jb3

Workflow Run

CI #1838

Source Branch

jb3/2938-nominate-context-menu

[regal arch]

ShakyaMajumdar

[python-discord/bot] Pull request review submitted: #3047 Add context menu command for user nomination

:shipit:

jb3

[python-discord/bot] Pull request closed: #3047 Add context menu command for user nomination

jb3

[bot:main] 4 new commits

dd2abe5 Add new Talentpool API method for fetching a no... - jb3
51e9e3e Add new context menu command for nominating users - jb3
5b209b1 Move from sentry_sdk.push_scope to sentry_sdk.n... - jb3
72b642b Merge pull request #3047 from python-discord/jb... - jb3

jb3

[python-discord/bot] branch deleted: jb3/2938-nominate-context-menu

jb3

[python-discord/bot] Issue closed: #2938 A context menu command to nominate users

jb3

[python-discord/bot] New comment on pull request #3047: Add context menu command for user nomination

Thanks!

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

Do we need to handle the reason being too long @jb3?

jb3

[python-discord/bot] New review comment on pull request #3047: Add context menu command for user nomination

I ignorantly hoped people would just not add a shit ton of messages, but yes we should handle this, PR in a second.

jb3

[bot:jb3/nom-context-char-limit] 1 new commit

5870a74 Check length of nomination reason before attemp... - jb3

jb3

[python-discord/bot] New branch created: jb3/nom-context-char-limit

jb3

[python-discord/bot] Pull request opened: #3050 Check length of nomination reason before attempting to send to site API

This ensures we will not attempt to send a reason that is longer than
the REASON_MAX_CHARS value, which can quickly be reached through the
addition of message links in context.

ChrisLovering

[python-discord/bot] Pull request review submitted: #3050 Check length of nomination reason before attempting to send to site API

[odd spire]

[python-discord/bot] Checks Successful on PR: #3050 Check length of nomination reason before attempting to send to site API

GitHub Actions run 9066797051 succeeded.

PR Author

jb3

Workflow Run

CI #1840

Source Branch

jb3/nom-context-char-limit

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3050 Check length of nomination reason before attempting to send to site API

👍

jb3

[python-discord/bot] Pull request closed: #3050 Check length of nomination reason before attempting to send to site API

jb3

[bot:main] 2 new commits

5870a74 Check length of nomination reason before attemp... - jb3
f7d1d8c Merge pull request #3050 from python-discord/jb... - jb3

jb3

[python-discord/bot] branch deleted: jb3/nom-context-char-limit

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3046 Don't upload to pastebin when no truncation

If we're adding a line to say that the message was truncated, but we're only truncating one line, wouldn't it be better to not truncate and the result would be the same number of lines?

That's the proposal I put forward here https://github.com/python-discord/bot/pull/2498#issuecomment-1493339577, essentially, for max_lines=10:

• 9 lines: not truncated
• 10 lines: not truncated
• 11 lines: truncated to 9 lines, plus the truncation message.

This is an improvement though, i'm happy to...

[regal arch]

wookie184

[python-discord/bot] New review comment on pull request #3038: Push raw to pastebin if too long for Discord

            # Replace backticks as it breaks out of code blocks.
            # An invisible character seemed to be the most reasonable solution.

It'll work properly if this puts it above 2k chars now so I think this part of the comment can be removed.

Also, we could make it so it doesn't do this replacement when uploading to pastebin, though it doesn't really matter.

wookie184

[python-discord/bot] New review comment on pull request #3038: Push raw to pastebin if too long for Discord

I think we want to use the plain text lexer here.

wookie184

[python-discord/bot] Pull request review submitted: #3038 Push raw to pastebin if too long for Discord

Works great, minor comments.

ChrisLovering

[python-discord/bot] New comment on issue #2679: Code snippets for paste service links

We have now forked pinnwand and pushed the latest changes to paste.pythondiscord.com

https://paste.pythondiscord.com/api/v1/paste/{id} can be used to get the paste data, and lexer for the code block.

EG https://paste.pythondiscord.com/api/v1/paste/NCXY2ZKJ2TT2E6RBEZHMTJ52VQ

wookie184

[python-discord/bot] New review comment on pull request #3000: Add feature to append additional text to nomination reasons

old_reason is the original nomination reason, so if the caller never nominated this user before, it will be None.

Yeah, but if they did nominate the user but with no reason it will be a blank string. In that case we don't want to add a . before the new reason.

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `56ecf7d`

Compare changes

ChrisLovering

[python-discord/bot] New review comment on pull request #3038: Push raw to pastebin if too long for Discord

Good point, done in the forcepush

ChrisLovering

[python-discord/bot] New review comment on pull request #3038: Push raw to pastebin if too long for Discord

Good point, done in the forcepush

I didn't fancy refactoring at this point to only add invisible chars if sending to discord.

Second forcepush is just a rebase onto main.

ChrisLovering

[bot] Branch feat/1037/push-raw-to-pastebin-if-too-long was force-pushed to `e383e22`

Compare changes

[odd spire]

[python-discord/bot] Checks Cancelled on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 9067791376 was cancelled.

PR Author

ChrisLovering

Workflow Run

CI #1842

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[python-discord/bot] Checks Successful on PR: #3038 Push raw to pastebin if too long for Discord

GitHub Actions run 9067808211 succeeded.

PR Author

ChrisLovering

Workflow Run

CI #1843

Source Branch

feat/1037/push-raw-to-pastebin-if-to...

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3038 Push raw to pastebin if too long for Discord

Great stuff! Thanks!

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1529 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[python-discord/sir-lancebot] Pull request closed: #1529 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[sir-lancebot:main] 1 new commit

60897c1 Bump pre-commit from 3.7.0 to 3.7.1 (#1529) - dependabot[bot]

wookie184

[python-discord/sir-lancebot] Issue opened: #1530 Go through sentry 2.0 migration guide

We should make sure we aren't using any of the methods deprecated in this major release, in the migration guide (https://docs.sentry.io/platforms/python/migration/1.x-to-2.x) push_scope is mentioned as being deprecated but we still use it here: https://github.com/python-discord/sir-lancebot/blob/7bd936f50668791d404358bcd74fb82b6e6656f6/bot/exts/core/error_handler.py#L8

_Originally posted by @wookie184 in https://github.com/python-discord/sir-lancebot/issues/1519#issuecommen...

wookie184

[python-discord/sir-lancebot] Pull request review submitted: #1528 Bump lxml from 5.2.1 to 5.2.2

[night lily]

Sir Lancebot

Connected!

[regal arch]

wookie184

[python-discord/sir-lancebot] Pull request closed: #1528 Bump lxml from 5.2.1 to 5.2.2

wookie184

[sir-lancebot:main] 1 new commit

bf16294 Bump lxml from 5.2.1 to 5.2.2 (#1528) - dependabot[bot]

[night lily]

Sir Lancebot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Failed on PR: #1527 Bump ruff from 0.3.7 to 0.4.4

GitHub Actions run 9068283325 failed.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4039

Source Branch

dependabot/pip/ruff-0.4.4

[regal arch]

wookie184

[python-discord/site] Pull request review submitted: #1318 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[python-discord/site] Pull request closed: #1318 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[site:main] 1 new commit

1a76087 Bump pre-commit from 3.7.0 to 3.7.1 (#1318) - dependabot[bot]

wookie184

[python-discord/bot] Pull request review submitted: #3048 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[python-discord/bot] New comment on pull request #3048: Bump pre-commit from 3.7.0 to 3.7.1

@dependabot rebase

[odd spire]

[python-discord/site] Checks Successful on PR: #1317 Bump psycopg from 3.1.18 to 3.1.19

GitHub Actions run 9068315621 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1195

Source Branch

dependabot/pip/psycopg-3.1.19

[regal arch]

wookie184

[python-discord/bot] Pull request closed: #3048 Bump pre-commit from 3.7.0 to 3.7.1

wookie184

[bot:main] 1 new commit

2a9f284 Bump pre-commit from 3.7.0 to 3.7.1 (#3048) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3048 Bump pre-commit from 3.7.0 to 3.7.1

GitHub Actions run 9068321989 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1844

Source Branch

dependabot/pip/pre-commit-3.7.1

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3049 Bump lxml from 5.2.1 to 5.2.2

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[python-discord/bot] Pull request closed: #3049 Bump lxml from 5.2.1 to 5.2.2

wookie184

[bot:main] 1 new commit

861cee4 Bump lxml from 5.2.1 to 5.2.2 (#3049) - dependabot[bot]

[odd spire]

[python-discord/bot] Checks Successful on PR: #3049 Bump lxml from 5.2.1 to 5.2.2

GitHub Actions run 9068352638 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1846

Source Branch

dependabot/pip/lxml-5.2.2

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

[oak estuary]

Python Bot

Connected!

Python Bot

Connected!

[thin oyster]

Sir Robin

Connected!

[regal arch]

jb3

[king-arthur:main] 1 new commit

8f98046 Update Grafana URL - jb3

[odd spire]

[python-discord/king-arthur] Workflow Successful

GitHub Actions run 9072562977 succeeded.

Actor

jb3

Workflow Run

Status embed #346

Commit

8f98046

[regal arch]

jb3

[bot:main] 1 new commit

0fac72f Update Metabase URL - jb3

[oak estuary]

Python Bot

Connected!

[regal arch]

wookie184

[bot-core:fix-dependabot] 1 new commit

f2c67b3 Use python version specifier that dependabot ca... - wookie184

wookie184

[python-discord/bot-core] New branch created: fix-dependabot

wookie184

[python-discord/bot-core] Pull request opened: #212 Use python version specifier that dependabot can understand

Dependabot was failing with logs saying Illformed requirement ["~>3.11.0 || 3"]

Confirmed on a fork that this was the issue, changing the python version specifier made dependabot alerts work properly.

wookie184

[python-discord/bot-core] New review comment on pull request #212: Use python version specifier that dependabot can understand

Interesting, I think that 3.11.* includes alpha/beta releases, but ^3.11 doesn't. I think not including them is correct.

ChrisLovering

[python-discord/bot-core] Pull request review submitted: #212 Use python version specifier that dependabot can understand

ChrisLovering

[python-discord/bot-core] New comment on pull request #212: Use python version specifier that dependabot can understand

I don't think we need to cut a release for this, since it's specifcally to fix a dependabot issue.

ChrisLovering

[python-discord/bot-core] Pull request closed: #212 Use python version specifier that dependabot can understand

ChrisLovering

[bot-core:main] 1 new commit

29234ba Use python version specifier that dependabot ca... - wookie184

[regal arch]

wookie184

[python-discord/bot] New comment on pull request #2980: Change color of cancel staff infraction button from green to gray

@Robin5605 any update on this?

[regal arch]

Robin5605

[python-discord/bot] Issue opened: #3051 Context Menu for incidenting messages

I think similar to https://github.com/python-discord/bot/pull/3047, it would be helpful to have some sort of "Quick Incident" context menu that lets Helpers+ quickly right click a message and incident it, instead of having to copy the message link, go to the incidents channel, then come back.

This works great for quickly incidenting messages that are clearly breaking the rules, but is not meant to replace the #incidents channel, because it allows for adding multiple messages for referen...

[regal arch]

Robin5605

[bot:Robin5605-patch-1] 1 new commit

5b1e301 Make docstring for confirm callback more accurate - Robin5605

[odd spire]

[python-discord/bot] Checks Successful on PR: #2980 Change color of cancel staff infraction button from green to gray

GitHub Actions run 9083186627 succeeded.

PR Author

Robin5605

Workflow Run

CI #1849

Source Branch

Robin5605-patch-1

[regal arch]

Robin5605

[bot:Robin5605-patch-1] 1 new commit

4cab68c Rename "InfractionConfirmationView" to "BanConf... - Robin5605

Robin5605

[bot:Robin5605-patch-1] 95 new commits

79b2844 Remove sentry sample rates and add asyncio inte... - ChrisLovering
288cc09 Move sentry initialisation to after the event l... - ChrisLovering
0b224f2 Merge pull request #2981 from python-discord/in... - shtlrs
7df12bf Enable Sentry performance tracing - ChrisLovering
7569771 Fix case sensitivity bug in phishing button - vivekashok1221

Robin5605

[python-discord/bot] New comment on pull request #2980: Change color of cancel staff infraction button from green to gray

@wookie184 I've addressed @ichard26's comment and @vivekashok1221's review. Should be good to go now, let me know if I've missed anything.

[odd spire]

[python-discord/bot] Checks Successful on PR: #2980 Change color of cancel staff infraction button from green to gray

GitHub Actions run 9083395321 succeeded.

PR Author

Robin5605

Workflow Run

CI #1850

Source Branch

Robin5605-patch-1

[python-discord/bot] Checks Successful on PR: #2980 Change color of cancel staff infraction button from green to gray

GitHub Actions run 9083408082 succeeded.

PR Author

Robin5605

Workflow Run

CI #1851

Source Branch

Robin5605-patch-1

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #2980 Change color of cancel staff infraction button from green to gray

Looks good, thanks

[regal arch]

jchristgit

[python-discord/site] Pull request review submitted: #1317 Bump psycopg from 3.1.18 to 3.1.19

jchristgit

[python-discord/site] Pull request closed: #1317 Bump psycopg from 3.1.18 to 3.1.19

jchristgit

[site:main] 2 new commits

227c897 Bump psycopg from 3.1.18 to 3.1.19 - dependabot[bot]
58ed38d Merge pull request #1317 from python-discord/de... - jchristgit

shtlrs

[sir-lancebot] Branch cleanup-constants was force-pushed to `c5e9b96`

Compare changes

shtlrs

[python-discord/sir-lancebot] New review comment on pull request #1484: Cleanup constants

Comments have been addressed

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1484 Cleanup constants

GitHub Actions run 9083873372 succeeded.

PR Author

shtlrs

Workflow Run

Status Embed #4040

Source Branch

cleanup-constants

[regal arch]

vivekashok1221

[python-discord/bot] Pull request review submitted: #2980 Change color of cancel staff infraction button from green to gray

LGTM :star2:

wookie184

[python-discord/bot] Pull request closed: #2980 Change color of cancel staff infraction button from green to gray

wookie184

[bot:main] 1 new commit

4917ddb Improve cancel staff infraction button colours ... - Robin5605

wookie184

[python-discord/bot] Issue closed: #2978 Update ban confirmation button labels and colours

wookie184

[python-discord/bot] branch deleted: Robin5605-patch-1

[odd spire]

[python-discord/bot] Workflow Failed

GitHub Actions run 9084000814 failed.

Actor

wookie184

Workflow Run

CI #1852

Commit

4917ddb

[regal arch]

wookie184

[python-discord/sir-lancebot] New comment on pull request #1484: Cleanup constants

Force pushing changes along with a rebase onto main makes it so I can't see what the changes made since my approval were so it's difficult to re-review (the compare view shows unrelated changes as well).

[oak estuary]

Python Bot

Connected!

[regal arch]

shtlrs

[python-discord/sir-lancebot] New comment on pull request #1484: Cleanup constants

Force pushing changes along with a rebase onto main makes it so I can't see what the changes made since my approval were so it's difficult to re-review (the compare view shows unrelated changes as well).

My bad, it's just that the changes made sense to be part of the commit that says "reuse forgotten emojis", which is why i rebased & force pushed.

[regal arch]

ChrisLovering

[modmail:master] 1 new commit

285e336 Fix Docker file permission issue (#3323) - raidensakura

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/sir-lancebot] Checks Successful on PR: #1531 Bump sentry-sdk from 2.1.1 to 2.2.0

GitHub Actions run 9110336808 succeeded.

PR Author

dependabot[bot]

Workflow Run

Status Embed #4041

Source Branch

dependabot/pip/sentry-sd...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3052 Bump sentry-sdk from 2.1.1 to 2.2.0

GitHub Actions run 9112476094 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1853

Source Branch

dependabot/pip/sentry-sdk-2.2.0

[python-discord/bot] Checks Successful on PR: #3053 Bump regex from 2024.5.10 to 2024.5.15

GitHub Actions run 9112484178 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1854

Source Branch

dependabot/pip/regex-2024.5.15

[regal arch]

minalike

[python-discord/bot] Issue opened: #3054 Have bot update nomination thread once 3 days has passed

Occasionally we forget about the policy of leaving Helper candidate reviews up for 3 days before actioning them (whether that be by opening a ticket to onboard them or ending the review and removing them from the talent pool).

Bot could keep track of new reviews when they start and post a simple "This review is now 3 days old" message once 3 days has passed.

[night lily]

Sir Lancebot

Connected!

[oak estuary]

Python Bot

Connected!

Python Bot

Connected!

[odd spire]

[python-discord/site] Checks Successful on PR: #1319 Bump sentry-sdk from 2.1.1 to 2.2.0

GitHub Actions run 9123498459 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1197

Source Branch

dependabot/pip/sentry-sdk-2.2.0

[regal arch]

janine9vn

[python-discord/bot] Issue opened: #3055 Post log from `!clean` (and simialr commands) in #message-change-log

Currently, if we do !cban or similar clean commands, the record of the messages being cleared gets output to #mods.

This can cause some confusion if two moderators are investigating and messages suddenly disappear, but there isn't a record in #message-change-log. If messages get deleted or edited, the logical place for a record of that to appear would be in #message-change-log.

So we should change where the record of deleted messages get sent to and send it to `#message-change-...

[regal arch]

Xithrius

[python-discord/bot] Pull request review submitted: #3053 Bump regex from 2024.5.10 to 2024.5.15

Xithrius

[python-discord/bot] Pull request closed: #3053 Bump regex from 2024.5.10 to 2024.5.15

Xithrius

[bot:main] 1 new commit

67feb2f Bump regex from 2024.5.10 to 2024.5.15 (#3053) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request review submitted: #3052 Bump sentry-sdk from 2.1.1 to 2.2.0

Xithrius

[python-discord/bot] New comment on pull request #3052: Bump sentry-sdk from 2.1.1 to 2.2.0

@dependabot recreate

Xithrius

[python-discord/bot] Pull request review submitted: #3038 Push raw to pastebin if too long for Discord

long

Xithrius

[bot:feat/1037/push-raw-to-pastebin-if-too-long] 6 new commits

2a9f284 Bump pre-commit from 3.7.0 to 3.7.1 (#3048) - dependabot[bot]
861cee4 Bump lxml from 5.2.1 to 5.2.2 (#3049) - dependabot[bot]
0fac72f Update Metabase URL - jb3
4917ddb Improve cancel staff infraction button colours ... - Robin5605
67feb2f Bump regex from 2024.5.10 to 2024.5.15 (#3053) - dependabot[bot]

Xithrius

[python-discord/bot] Pull request closed: #3038 Push raw to pastebin if too long for Discord

Xithrius

[bot:main] 1 new commit

1e73aa4 Push raw to pastebin if too long for Discord (#... - ChrisLovering

Xithrius

[python-discord/bot] Issue closed: #1037 raw command doesn't trim lines that are too long for the paginator

Xithrius

[python-discord/bot] branch deleted: feat/1037/push-raw-to-pastebin-if-too-long

[oak estuary]

Python Bot

Connected!

[odd spire]

[python-discord/bot] Checks Successful on PR: #3052 Bump sentry-sdk from 2.1.1 to 2.2.0

GitHub Actions run 9132690893 succeeded.

PR Author

dependabot[bot]

Workflow Run

CI #1856

Source Branch

dependabot/pip/sentry-sdk-2.2.0

[oak estuary]

Python Bot

Connected!

[regal arch]

jb3

[bot:jb3/active-nomvote-thresholds] 1 new commit

ca5a6c5 Allow for more than 3 nomination votes if other... - jb3

jb3

[python-discord/bot] New branch created: jb3/active-nomvote-thresholds

jb3

[python-discord/bot] Pull request opened: #3056 Allow for more than 3 nomination votes if others are ticketed

This changes how we calculate whether we are ready for another vote in
the talentpool autonomination system.

Instead of hard-locking at 3 votes in the pool, we factor in whether an
admin-task has been created for the vote (signified by a reaction with
the :ticket: emoji).

We allow for up to 10 votes to be in the pool total, which in theory
allows 3 active votes and 7 votes waiting on the admin-task
completion (though we should never hit this threshold realistically).

This should allow for ...

[odd spire]

[python-discord/bot] Checks Successful on PR: #3056 Allow for more than 3 nomination votes if others are ticketed

GitHub Actions run 9135245028 succeeded.

PR Author

jb3

Workflow Run

CI #1859

Source Branch

jb3/active-nomvote-thresholds

[regal arch]

wookie184

[python-discord/bot] Pull request review submitted: #3056 Allow for more than 3 nomination votes if others are ticketed

Looks good, the tests could maybe do with updating but up to you.

[regal arch]

wookie184

[python-discord/bot] New comment on issue #3055: Post log from `!clean` (and simialr commands) in #message-change-log

The deletion is currently logged in #mod-log. I can see the argument beind having it there, since it is the result of a moderation action, though I agree it would also fit #message-change-log. I'm not sure where I'd guess it would be myself if I didn't know.

[regal arch]

jb3

[bot:jb3/active-nomvote-thresholds] 1 new commit

d7f158b Update tests for new autonomination threshold l... - jb3

jb3

[python-discord/bot] New comment on pull request #3056: Allow for more than 3 nomination votes if others are ticketed

Good spot on the tests, I've updated them now to add 2 new cases for "above active, below total" and "above active, above total".