AdguardTeam / AdguardTeam/DeadDomainsLinter

Add comment leftovers removing

Aberta
#9 0 comentários 2 reações 1 responsável Reivindicada por @maximtop Ver no GitHub
Priority: P4
Linguagem predominante
JavaScript
Estrelas
35
Forks
3
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

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.

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.