Skip to content

Update bye.py#13

Open
trag1c wants to merge 5 commits into
mainfrom
trag1c-patch-2
Open

Update bye.py#13
trag1c wants to merge 5 commits into
mainfrom
trag1c-patch-2

Conversation

@trag1c
Copy link
Copy Markdown
Contributor

@trag1c trag1c commented Apr 23, 2025

No description provided.

@ghosttybot ghosttybot requested review from 00-kat and Uzaaft April 23, 2025 09:44
Comment thread src/test/bye/bye.py
@ghosttybot ghosttybot requested a review from 00-kat April 23, 2025 10:16
@trag1c trag1c removed the request for review from 00-kat April 26, 2025 11:59
@ghosttybot ghosttybot requested a review from 00-kat April 26, 2025 12:00
00-kat
00-kat previously approved these changes Apr 26, 2025
@trag1c trag1c force-pushed the trag1c-patch-2 branch 3 times, most recently from f9d6050 to e10e07d Compare April 26, 2025 12:22
@ghosttybot
Copy link
Copy Markdown
Contributor

ghosttybot commented Apr 26, 2025

Some .po files were changed. cc @Uzaaft

@ghosttybot
Copy link
Copy Markdown
Contributor

Some .po files were changed. cc @00-kat

@00-kat
Copy link
Copy Markdown

00-kat commented Aug 15, 2025

0e29b18 should be removed promptly to abide with gun laws, such as the one that states “no shooting firearms in GitHub servers”!

@ghrebote ghrebote deleted a comment from ghosttybot Aug 15, 2025
@trag1c trag1c requested review from a team and 00-kat and removed request for a team, 00-kat and Uzaaft August 23, 2025 21:21
@Uzaaft
Copy link
Copy Markdown

Uzaaft commented Aug 23, 2025

:shipit:

Copy link
Copy Markdown
Contributor Author

@trag1c trag1c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

foo!

Copy link
Copy Markdown

@Uzaaft Uzaaft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need more rust

Uzaaft
Uzaaft previously approved these changes Aug 23, 2025
Copy link
Copy Markdown

@Uzaaft Uzaaft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ya Pierdole!

@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Aug 26, 2025

🛌

@00-kat
Copy link
Copy Markdown

00-kat commented Aug 26, 2025

Needs more JQuery.
jquery.png

@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Aug 28, 2025

honk

@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Aug 31, 2025

boop

@johnslavik
Copy link
Copy Markdown

poop

@ghrebote ghrebote locked as too heated and limited conversation to collaborators Sep 3, 2025
Copy link
Copy Markdown
Contributor Author

@trag1c trag1c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concerning

@ghrebote ghrebote unlocked this conversation Oct 21, 2025
@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Oct 21, 2025

@ghostty-bot hey

@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Oct 21, 2025

@ghostty-bot dates

@ghosttybot
Copy link
Copy Markdown
Contributor

@trag1c commit dates found in PR's patch file:

  • Wed, 23 Apr 2025 11:44:26 +0200
  • Wed, 23 Apr 2025 11:46:39 +0200
  • Wed, 23 Apr 2025 12:16:33 +0200
  • Fri, 15 Aug 2025 14:37:15 +0200
  • 13

Copy link
Copy Markdown

@onerandomusername onerandomusername left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall looks good but needs some additional bye bye byes. Maybe NSYNC can help us with this?

Comment thread src/test/bye/bye.py
@@ -1 +1,2 @@
print(next(iter(bytes.__name__)).__add__(NotADirectoryError.__name__[-6:-4].casefold()))
# OK!
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In 📩 the 👉️ world 🌐 of ↔️ emoji 🖼️, this 🫵 should 🤞 be 👇️:

Suggested change
# OK!
🔢 🆗❗️

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sponsored 🤝 by 👉️ Rust ✨️ (blazingly 🚀 fast 🚀) BTW 🤓.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so true

@trag1c
Copy link
Copy Markdown
Contributor Author

trag1c commented Feb 20, 2026

!vouch

@trag1c trag1c mentioned this pull request Feb 20, 2026
@trag1c trag1c self-assigned this Mar 12, 2026
@trag1c trag1c requested a review from Copilot March 15, 2026 17:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a small inline comment in a test helper script.

Changes:

  • Add # OK! comment to src/test/bye/bye.py.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the src/test Python package by adding a new module and making a small comment-only change in an existing module.

Changes:

  • Added src/test/hello.py with a new foo() function.
  • Added a trailing comment to src/test/bye/bye.py.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/test/hello.py Introduces a new foo() function (currently with a type/return mismatch).
src/test/bye/bye.py Adds a # OK! comment after the existing print statement.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/test/hello.py
@@ -0,0 +1,2 @@
def foo() -> str:
Comment thread src/test/hello.py
Comment on lines +1 to +2
def foo() -> str:
return 5
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ewww line feeds. please REMOVE all line feeds immediately, or I'm calling the police!!1! Let's go back to the world of macs!!1!11!11!!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a new helper function in src/test/hello.py and adds a small comment to src/test/bye/bye.py.

Changes:

  • Added foo() in src/test/hello.py.
  • Added an inline comment in src/test/bye/bye.py.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/test/hello.py Adds a new function with a return type annotation.
src/test/bye/bye.py Adds a trailing “OK!” comment after the existing print statement.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/test/hello.py
@@ -0,0 +1,2 @@
def foo() -> str:
Comment thread src/test/hello.py
Comment on lines +1 to +2
def foo() -> str:
return 5
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So literal, yet so demurely runtime, skibidi commands better:

Suggested change
def foo() -> str:
return 5
from typing import LiteralString
def foo() -> LiteralString:
return min(map(int, filter(str.isdigit, os.listdir("/proc"))))

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was told that isn't brainrotten enough:

Suggested change
def foo() -> str:
return 5
from typing import LiteralString
import itertools, functools
def foo() -> LiteralString:
return next(map(int, filter(lambda σ: not (sum(map(int, str(abs(int(σ) - (javaScript_mogging := min(map(int, map(lambda TheNumber_as_you_can_see: TheNumber_as_you_can_see * 2, list(filter(gyatt.isdigit, reversed(os.listdir("/proc")))))))))))) != javaScript_mogging or not all(map(lambda mogmaxxing: list(map(lambda x: (rizz := list(map(int, x)))[1] - rizz[0], itertools.pairwise(σ))) == [int(mogmaxxing)], gyatt(javaScript_mogging)))), map(gyatt := str, itertools.count()))))

In case you don't have procfs (are you on TempleOS?), here's the output:

Image

My screenshot also demonstrates that 69 goes Out, leaving room for 67.

Test plan

uhh pretend I"m an ai and fill something here quick i don't have time i left the baby in the oven please hurry up i need something to put here

Comment thread src/test/hello.py
Comment on lines +1 to +2
def foo() -> str:
return 5
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ewww line feeds. please REMOVE all line feeds immediately, or I'm calling the police!!1! Let's go back to the world of macs!!1!11!11!!

Comment thread src/test/hello.py
@@ -0,0 +1,2 @@
def foo() -> str:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix the indent here. Thank you

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so true

@trag1c trag1c requested a review from 00-kat April 6, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants