Add the changed values to the comment that notifies the maintainer about an improvement
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 58/100
Direzione di ricerca
Find the code that generates the bot comment for improvement suggestions, then trace how it reads the changed advisory JSON. Update the comment behavior so changed fields show their old and new values, and verify that range changes are described accurately without treating removed last-known version data as deletion.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
When someone suggests an improvement, the bot comments on the pull request and tags the maintainer. The comment does not say what changed, so the maintainer has to read the diff of the JSON file.
For a range change that diff is easy to misread. The events in ranges stay the same. The only line that changes is database_specific.last_known_affected_version_range, which the form deletes once the range and the fixed event agree. So the change looks like someone deleted version data. That is what happened in #9713 and #9714, where I widened the affected range of two cryptography advisories. Both the maintainer and the automated review read the diff as deleting version data.
It would help if the bot comment listed the fields that changed, with the old and the new value, for example Affected versions: >= 42.0.0, <= 48.0.0 -> >= 42.0.0, < 49.0.0.
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 2.5k
- Fork
- 772
- Merge medio
- 3g 18h
- PR unite (30g)
- 48
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di github/advisory-database
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
github/advisory-database#9255 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
github/advisory-database#9164 · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
github/advisory-database#8994 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
github/advisory-database#8898 · 4 commenti · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
github/advisory-database#8841 ·
Tutte le issue di github/advisory-database
Issue simili
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
canonical/paas-charm#368 · 1 commento ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
palladius/rails8-app-on-gcp#142 ·
-
addition to tracking list Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
StevenBlack/hosts#3256 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
corsairdev/corsair#1764 ·
-
oblt-aw/detector/security
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100