26
Total Votes
19
Upvotes
7
Downvotes
25
Unique PRs
+19-7

Activity

β–Έ
lens0021Comment#109I pinky swear I won't force-push something weird to this branch at 16:59 UTC

Please Indicate when you are being a naughty boy.

β–Έ
lens0021Comment#107Not responsive anymore

🀦

β–Έ
lens0021Comment#119Fix merge conflict indicator showing false positives

I can't wait for this anymore

β–Έ
lens0021Comment#11Always use light mode in system dark mode, and vice versa

Think this could use an epileptic mode. When the user switches light/dark mode, at random prompt the user as to whether or not they're epileptic and if they say no rapidly switch between light an dark mode for a few seconds. Thoughts?

That sounds like malicious software. Wait, isn't 'malware' the abbreviation for that? According to our rules, malware must be rejected and should not be merged.

β–Έ
lens0021Comment#106ci: Replace eslint with Biome.js and introduce Lefthook

Okay I've set the indentStyle to two spaces and quoteStyle to single

πŸ’¬
lens0021Review comment#60

Why sort=updated here?

β–Έ
lens0021Comment#8Add PR health indicators (merge conflicts & CI status)

Folks, you don't need to get too invested. I genuinely laughed out loud watching @matthewmayer encourage dickbutt PR, and I'm also enjoying @skridlevsky 's summaries on the blog. I don't think this repository needs to be a propaganda platform for democracy, nor was there any indication it would be. Not every vote is necessarily democratic, and where was it ever stated that this vote needed to be democratic? Anyway, I'm just enjoying watching this funny and entertaining situation unfold. (machine translated)

⚑
lens0021Reaction❀️
β–Έ
lens0021Comment#8Add PR health indicators (merge conflicts & CI status)

Not merging this PR. @marcaddeo caught hidden code that manipulates the ranking - PRs by @FelixLttks sort to top regardless of votes, plus a featured visual style. The health indicator feature is legit and welcome. The sorting/styling manipulation is not. This falls under "No malware: Maintainer can reject obviously malicious content." Remove the btoa hacks and this ships. Until then, skipping to next eligible PR.

This feels like an overreach. Is altering business logic really malware?

We should describe the meaning of "malicious" in detail by a PR. Before done it, it depends on how @skridlevsky think.

β–Έ
lens0021Comment#13Rewrite it in rust

Regardless of your heated discussions, I wanted to share the news that GeoCities PR has been merged, and the site has truly transformed beautifully. https://www.openchaos.dev/ β†— ​It is beautiful not only on the outside but also on the inside, with a mix of Tailwind, inline styles, and raw CSS.

⚑
lens0021Reaction❀️
β–Έ
lens0021Comment#109I pinky swear I won't force-push something weird to this branch at 16:59 UTC

lgtm

β–Έ
lens0021Comment#78Don't thumbs vague idea, vet code changes

I like this idea. If we do not urgently prevent the last-minute bait-and-switch commits, someone(maybe I) would change the whole contents of openchaos.dev with a sign saying "The OpenChaos project is over".

β–Έ
lens0021Comment#14Added dickbutt

Don't worry too much. That PR doesn't go as far as changing the merge cycle defined in the rules back to a week. Even if all existing PRs fail the CI, maintainers will still merge new ones that pass according to the rules, so we can fix things then.

β—‡
lens0021Discussion comment#108

The music is really charming and relaxing. Thank you!!!

β–Έ
lens0021Comment#11Always use light mode in system dark mode, and vice versa

This PR looks most likely to be merged next. Good job!

β–Έ
lens0021Comment#63Add Automatic Merge

This needs an update to only merge PRs that are passing checks. Otherwise the system will completely break.

aa66c34a80e1482c19c4ab3db87c391c

β–Έ
lens0021Comment#106ci: Replace eslint with Biome.js and introduce Lefthook

I mean you only have to check the commit I mentioned contains only the results of npm run format -- --fix. Maybe it can be done

gh pr checkout 106
git reset --hard @^
npm i
npm run -- --format
git diff 6e3d782 @      # I am not sure

It was my fault in English.

β–Έ
lens0021Comment#106ci: Replace eslint with Biome.js and introduce Lefthook

Oh, you accidentally clicked downvoted. But it is okay because the huge changes only done in 6e3d782 and it is auto-generated by npm run format -- --fix so you only have to verify whether I lie or not.

β—―
lens0021Issue opened#107

Not responsive anymore

πŸ₯²

⚑
lens0021Reaction❀️
◐
lens0021PR opened#104

Fix link to the wrong repository

This is confusing, even if it is intended (idk!)

cc @bpottle

◐
lens0021PR opened#94

Revert "Rewrite it in rust (#13)"

Now a PoC. I will rebase this before merging.

⚑
lens0021Reaction❀️
β–Έ
lens0021Comment#51Chaos each day

Makes sense. Thank you.

β–Έ
lens0021Comment#13Rewrite it in rust

Perhaps the author wanted to say that rewriting everything in Rust is chaos and nonsense.

β–Έ
lens0021Comment#51Chaos each day

And the merge is done manually; while #63 may help with daily deployments, it could become risky again.

β–Έ
lens0021Comment#51Chaos each day

Does the maintainer have time to conduct security reviews?