AdguardTeam / AdguardTeam/DeadDomainsLinter

Add comment leftovers removing

Aperta
#9 0 commenti 2 reazioni 1 assegnatario Rivendicata da @maximtop Vedi su GitHub
Priority: P4
Lingua principale
JavaScript
Stelle
35
Fork
3
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In general not all comments should be removed because they also used for description of groups or rules purposes.

But simple comments may be removed.

For example selected `! https://github.com/AdguardTeam/repo_name/` comments can be removed quite harmlessly
![image](https://github.com/AdguardTeam/DeadDomainsLinter/assets/8361299/79dd6e01-6b68-43d6-bced-fcb73aa5d5a2)

Criteria for removal:
* the rule(s) [+ `HINT`] to be deleted is accompanied by a comment of the following form
`! https://github.com/AdguardTeam/repo_name/`
* the line following the rule(s) to be deleted is a comment of the form
`! https://github.com/AdguardTeam/repo_name/`

Some rules may be accompanied by a few comments (not multiline comments - we will handle them manually).
For example:

```
! https://github.com/AdguardTeam/CoreLibs/issues/1763
! Website checks Content-MD5 in response headers and if it's empty does not display a content
@@||devfaq.ru/*/search?*&method=HEAD$xmlhttprequest,document
! https://github.com/AdguardTeam/CoreLibs/issues/1744
!+ PLATFORM(windows, mac, android)
@@||bing.com/search?showselans=$document,subdocument,important
! comment line 1
! comment line 2
@@||jetpack.io^$elemhide,jsinject,extension,content
! https://github.com/AdguardTeam/CoreLibs/issues/1830
! TODO: check the bug status 14.12.23
!+ PLATFORM(windows, mac, android)
@@||unicode.org^$content
! https://github.com/AdguardTeam/CoreLibs/issues/123
```

* if `bing.com` is dead, `! https://github.com/AdguardTeam/repo_name/` + `HINT` can be removed.
* if `unicode.org` is dead, `! https://github.com/AdguardTeam/repo_name/` + any text comment + `HINT` can be removed.
* if `jetpack.io` is dead, don't remove comments

```
!
! This section contains the list of third-party advertising networks domains.
! Note, that we only put rules that block full domains here and not URL parts (there's `general_url.txt` for that). Also, it must be domains that are used in a third-party context.
! The rules with hints are at the end of file.

! Good: ||doubleclick.net^$third-party
! Bad: /banner.jpg (should be in general_url.txt)
! Bad: ||adssubdomain.legitwebsite.com^ (should be in adservers_firstparty.txt)
!
!
!
||venturepeasant.com^
||riweeboo.com^
||8trd.online^
// another rules or the end of file
```

if `venturepeasant.com`, `riweeboo.com`, `8trd.online` are dead - do not remove comments.

```
! SECTION: KillAdBlock
! https://github.com/AdguardTeam/AdguardFilters/issues/82559
zuketcreation.net#@##adMid1
zuketcreation.net#@##ad_feature
zuketcreation.net#@##adtagfooter
zuketcreation.net#@##docmainad
zuketcreation.net#@##myAd
zuketcreation.net#@##travel_ad
! '#$#.pub_300x250.pub_300x250m.pub_728x90.text-ad.textAd.text_ad.text_ads.text-ads.text-ad-links { display: block !important; }'
zuketcreation.net#$#.pub_300x250.pub_300x250m.pub_728x90.text-ad.textAd.text_ad.text_ads.text-ads.text-ad-links { display: block !important; }
!#if (adguard_ext_safari || adguard_app_ios || adguard_ext_android_cb)
! exclusions for platforms without CSS rules support
tapewithadblock.org#@#.pub_300x250
tapewithadblock.org#@#.pub_300x250m
tapewithadblock.org#@#.pub_728x90
tapewithadblock.org#@#.text-ad
tapewithadblock.org#@#.text-ad-links
tapewithadblock.org#@#.text-ads
tapewithadblock.org#@#.textAd
tapewithadblock.org#@#.text_ad
tapewithadblock.org#@#.text_ads
tapewithadblock.org#@#.textad
tapewithadblock.org#@#.TextAd
! https://github.com/AdguardTeam/AdguardFilters/issues/67366
articlix.com,fantastic-film.online,joinfaucet.com#@#.textad
articlix.com,fantastic-film.online,joinfaucet.com#@#.TextAd
!#endif
! NOTE: KillAdBlock end ⬆️
! !SECTION: KillAdBlock
```

In this case, if domain of the section are dead, comments should not be removed. We will handle them manually, because such groups may be reused many times. That's why the criteria for removal is so limited.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.