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.

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

Replace most of the 'kagi search' site: entries with proper names and domains #132

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thibaultmol
Copy link
Contributor

I'll be honest, I made gpt do all of this (I just quickly checked it didn't change any things it shouldn't have).

I noticed that some of the ones that were changed are no longer using working websites but that's a check for later I guess

@nobodywasishere
Copy link
Member

The d field should stay as kagi.com

@thibaultmol
Copy link
Contributor Author

@NoahAndrews
Copy link
Contributor

NoahAndrews commented Oct 31, 2024

I apologize for my previous comment, I was bothered by what saw as a blind use of AI.

@thibaultmol
Copy link
Contributor Author

thibaultmol commented Oct 31, 2024

no worries, Figured GPT would be better at me at knowing what most of these websites were about (most of them I hadn't heard of till today).
tbf, they seemed right to me (correct like a couple manually) but some of the ones that are corrected in this PR actually just don't exist anymore. .... so i'm not sure if those should then be fully deleted or if those should stay (for people who use redirection systems? i dunno if those would even work then)

@NoahAndrews
Copy link
Contributor

NoahAndrews commented Oct 31, 2024

Yeah, it's a reasonable use (combined with at least a quick verification), I was just thrown by the domains being wrong.

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