LLocalSearch icon indicating copy to clipboard operation
LLocalSearch copied to clipboard

Published 20 hours ago • verified publisher icon nilsherzig

Update README architecture link

Open suparious opened this issue 10 months ago • 2 comments

As the link changed from a issue to a discussion I also wanted to take the opportunity to update the language, as this is not infrastructure, it is architecture.

suparious avatar Apr 13 '24 21:04 suparious

Oh thanks, you're right about that. May I ask you something? Are there any benefit of opening a pull request besides fixing the link? I get a lot of single line change pull requests

nilsherzig avatar Apr 16 '24 14:04 nilsherzig

@nilsherzig - For me the benefit is with the new users coming to the project and possibly getting put off by finding documentation links that lead to 404 errors, and language that doesn't make logical sense.

I guess the language change was pedantic, but important, as infrastructure and architecture are different specializations / disciplines, and some people are not familiar with both.

Also, seeing your response and question to my simple PR, gives me confidence that this project would be something that I would want to spend time on and make a real contribution.

suparious avatar Apr 18 '24 08:04 suparious