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

Delete Twitter #122

Closed
wants to merge 18 commits into from
Closed

Delete Twitter #122

wants to merge 18 commits into from

Conversation

sunglocto
Copy link

This is for the greater good.

@ghost
Copy link

ghost commented Mar 31, 2023

LGTM

@HyflerSD
Copy link

What is the time/space complexity of this implementation?

@highnet
Copy link

highnet commented Mar 31, 2023

LGTM

@XXbruhmanXX
Copy link

Please review this

@sunglocto
Copy link
Author

What is the time/space complexity of this implementation?

I don't know what this means, I am working for Twitter and get paid 5 cents an hour
Code is now blazingly fast though, i think

@piq9117
Copy link

piq9117 commented Mar 31, 2023

LGTM

@SivkovSavely
Copy link

What is the time/space complexity of this implementation?

@HyflerSD it's so complex that it warps the space-time continuum and creates a black hole once merged.

@GalvinPython
Copy link

lgtm

1 similar comment
@Marcellofabrizio
Copy link

lgtm

@anthuanvasquez
Copy link

LGTM

@dlaufer
Copy link

dlaufer commented Mar 31, 2023

Can this be done in Qbasic?

@OneAndOnlyFinbar
Copy link

Lgtm, deploying to prod

@sunglocto
Copy link
Author

NOOOOOO!

@Redeem-Grimm-Satoshi
Copy link

Reasons like this is why we don't make things open source!

@ghost
Copy link

ghost commented Mar 31, 2023

1984

@SeymoreBiggins
Copy link

LGTM 👍

@Debagnik
Copy link

Looks good to me

@josphatmwania
Copy link

Interesting.

@ItsAstronomical
Copy link

lgtm

umutbasal
umutbasal reviewed Mar 31, 2023
View reviewed changes
Copy link

@umutbasal umutbasal left a comment

Choose a reason for hiding this comment

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

LGTM

@rahmanharoon
Copy link

LGTM

1 similar comment
@BarasaJnr
Copy link

LGTM

@hellofaizan
Copy link

Reasons like this is why we don't make things open source!

Ah it's just Sarcastic PR.

@yrdzrfxndfvh
Copy link

Looking into this.

@Romejanic
Copy link

this can't be merged fast enough

@jordanbertasso
Copy link

lgtm

@argahutama
Copy link

LGTM!

@Airdel
Copy link

Airdel commented Apr 1, 2023

lgtm 🤙

@kururuchan
Copy link

LGTM

@DenizOzii
Copy link

idk why this was closed but LGTM

@x5px
Copy link

x5px commented Apr 1, 2023

LGTM, ship it

@stingalleman
Copy link

lets goo

JakeyWasTaken
JakeyWasTaken reviewed Apr 1, 2023
View reviewed changes
Copy link

@JakeyWasTaken JakeyWasTaken left a comment

Choose a reason for hiding this comment

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

LGTM not sure why this was closed

@Synergetic00
Copy link

LGTM

1 similar comment
@d0nker420
Copy link

LGTM

@itallix
Copy link

itallix commented Apr 3, 2023

Merge it! 🚀

@ektedar
Copy link

ektedar commented Apr 4, 2023

Reasons like this is why we don't make things open source!

Allow transparency so people have insight on how Twitter functions and eventually build trust and eliminate bias ❌
Not take a joke ✅

@turrnut
Copy link

turrnut commented Apr 4, 2023

delete twitter LMAO

@ddvdozuki
Copy link

LGTM Ship it!

@okaits
Copy link

okaits commented Apr 6, 2023

LGTM, merge it

1 similar comment
@okaits

This comment was marked as duplicate.

Sign in to view
@5GVckyy
Copy link

5GVckyy commented Aug 27, 2023

LGTM

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

@umutbasal umutbasal umutbasal left review comments

@JakeyWasTaken JakeyWasTaken JakeyWasTaken left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet