Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

John Carmack fumbled #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

John Carmack fumbled #29

wants to merge 1 commit into from

Conversation

kenan238
Copy link

PI is messed up

@expected-ingot
Copy link

LGTM

1 similar comment
@cacheninetynine
Copy link

LGTM

sapbotgit
sapbotgit reviewed Sep 14, 2024
View reviewed changes
Copy link

@sapbotgit sapbotgit left a comment

Choose a reason for hiding this comment

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

This is not a big change, and it's src archive. It will NOT be changed.

SafinW
SafinW approved these changes Sep 15, 2024
View reviewed changes
Copy link

@SafinW SafinW left a comment

Choose a reason for hiding this comment

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

no

@bcsanches
Copy link

Now people try to get likes nitpicking archival source code on github....

ScalarJerk
ScalarJerk approved these changes Oct 10, 2024
View reviewed changes
Copy link

@ScalarJerk ScalarJerk left a comment

Choose a reason for hiding this comment

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

approved. john carmark did fumble 💯

felipecastilhos
felipecastilhos approved these changes Nov 20, 2024
View reviewed changes
Copy link

@felipecastilhos felipecastilhos left a comment

Choose a reason for hiding this comment

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

LGTM 💯% fumbled

Royvdelft
Royvdelft approved these changes Nov 27, 2024
View reviewed changes
Copy link

@Royvdelft Royvdelft left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sapbotgit sapbotgit sapbotgit left review comments

@SafinW SafinW SafinW approved these changes

@Royvdelft Royvdelft Royvdelft approved these changes

@ScalarJerk ScalarJerk ScalarJerk approved these changes

@felipecastilhos felipecastilhos felipecastilhos approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@kenan238 @expected-ingot @cacheninetynine @bcsanches @felipecastilhos @ScalarJerk @Royvdelft @sapbotgit @SafinW