fix(ci): radar sprawdza, czy kandydat w ogole jest katalogiem - #168
Merged
Merged
Conversation
The run of 2026-08-17 queued a database client and a collection of n8n workflow templates. Both passed because their README contains the word "email" somewhere, which is true of most repositories and says nothing about whether the project is a curated list. Two gates instead. A candidate must look like a directory - at least 40 entry-shaped lines, `- [name](https://...)` - and must carry a mail or email *heading*, not the word loose in the prose. The matched heading is now what the queued issue reports, so a reviewer sees which section the entry would go in. Checked against real READMEs: free-for-dev 1287 entries and a heading, passes. Chat2DB 0 entries, rejected. awesome-n8n-templates 2 entries, rejected. awesome-selfhosted has 1261 entries and passes this gate, and is still caught by the eligibility gate above it, which is the right order.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bieg z 17.08 zakolejkował klienta bazodanowego i zbiór szablonów n8n. Oba przeszły, bo ich README zawiera gdzieś słowo
email— co jest prawdą o większości repozytoriów i nie mówi nic o tym, czy projekt jest listą kuratorską.Radar sprawdzał, jakiego rodzaju jest lista, ale nie sprawdzał, czy to w ogóle jest lista.
Dwie bramki zamiast jednej
- [nazwa](https://...). Katalog wygląda jak katalog.Sprawdzone na prawdziwych README
ripienaar/free-for-devOtterMind/Chat2DBenescingoz/awesome-n8n-templatesawesome-selfhostedOstatni wiersz jest istotny: kolejność bramek jest właściwa — najpierw „czy to lista", potem „czy przyjmuje usługi hostowane".
Issues #166 i #167 zamknięte z uzasadnieniem.