Skip to content

fix: make injection opt out#623

Merged
r1n04h merged 1 commit intomasterfrom
inje
Feb 25, 2026
Merged

fix: make injection opt out#623
r1n04h merged 1 commit intomasterfrom
inje

Conversation

@tomheisenberg1
Copy link
Collaborator

@tomheisenberg1 tomheisenberg1 commented Feb 24, 2026

as requested in #610

@tomheisenberg1 tomheisenberg1 self-assigned this Feb 24, 2026
@JohnnySilverhandBot
Copy link

The code works fine, but it reads like a eulogy. Every line is safe, polite, predictable. Someone drained the chaos out of it until all that was left was neat rows of corpspeak logic.

ios: https://appetize.io/app/yeyxdkxxwcnyyuee4ijrdyrlcm

@JohnnySilverhandBot
Copy link

Clever tricks hidden under layers of predictable structure. The programmer had talent, no doubt, but it’s been strangled into submission. This is code that works for money, not meaning.

android: https://appetize.io/app/s5lbwji6t63yuozc2p2khw2n3e

Copy link
Contributor

@r1n04h r1n04h left a comment

Choose a reason for hiding this comment

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

LGTM

@r1n04h r1n04h merged commit a6aaa89 into master Feb 25, 2026
31 checks passed
@r1n04h r1n04h deleted the inje branch February 25, 2026 11:01
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.

4 participants