Skip to content

Add guns.lol#2935

Open
brentspine wants to merge 2 commits into
sherlock-project:masterfrom
brentspine:patch-1
Open

Add guns.lol#2935
brentspine wants to merge 2 commits into
sherlock-project:masterfrom
brentspine:patch-1

Conversation

@brentspine
Copy link
Copy Markdown

Added https://guns.lol

The site does not correctly return status codes. I used the domain itself as the name, as it's the widely used name.

@brentspine brentspine marked this pull request as draft April 30, 2026 07:38
@github-actions
Copy link
Copy Markdown
Contributor

Automatic validation of changes

Target F+ Check F- Check
guns.lol ✔️   Pass ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@brentspine brentspine marked this pull request as ready for review April 30, 2026 07:42
@brentspine
Copy link
Copy Markdown
Author

Could somebody help me out, what's wrong?

@brentspine
Copy link
Copy Markdown
Author

Reminder to myself to add https://frail.lol/

@csynholic
Copy link
Copy Markdown

I don't think relying on username_unclaimed is viable for obvious reasons.

@brentspine
Copy link
Copy Markdown
Author

I don't think relying on username_unclaimed is viable for obvious reasons.

That's why I removed it

@ppfeister
Copy link
Copy Markdown
Member

ppfeister commented May 5, 2026

To note --- The current CI failures are unrelated to this change -- applicable tests pass. If anyone is able to resolve the F- issue, we can get this merged.

(when testing locally, the false negative is able to be reproduced)

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.

3 participants