Skip to content

Update README and templates - #1214

Closed
dotext wants to merge 13 commits into
AdguardTeam:masterfrom
dotext:patch-1
Closed

dotext wants to merge 13 commits into
AdguardTeam:masterfrom
dotext:patch-1

Conversation

@dotext

@dotext dotext commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Changes

README.md

  • Rearrange layout to match other recent AG repos for consistency
  • Update Website link to match other AG repos
  • Update Twitter to X
  • Remove visual bug with <a href showing space and underline after SVG image
  • Add web reporting tool link as an alternative to match other AG repos
  • Add contribution section
  • Minor revisions (typos, grammar, etc.) made

01-bug-report.yaml

  • Include AdGuard Mail Tracking Protection filter
  • Minor version numbering updates to keep it in line with recent releases
  • Update reporting tool link to match same as config.yml (https://link.adtidy.org/forward.html?action=report&app=home&from=github)
  • Update filter lists to match what is currently available (except language-specific)
  • Minor revisions (typos, grammar, etc.) made
  • Capitalisation fixes for all AG repo cohesion

02-feature-request.yaml

  • Capitalisation fixes for all AG repo cohesion

03-question.yaml

  • Add required answer for using the latest version

config.yml

  • Add config.yml to match other AG repos
  • Update knowledgebase to "AdGuard Knowledge Base"

Context

I've noticed that the README.md files are not unified in layout, need updated links and to resolve a few visual issues. The GitHub templates for Issues were also with outdated filter lists and with minor opportunities for some tweaks to bring it up to date with recent releases and fix a few things here and there.

Since working with others on the other PRs for updating the README.md and template files, there are a few more things that should be applied to the templates for the sake of cohesion. These changes have already been applied to this existing PR, but PRs for this second revision have been made if already merged before these changes:

Also added further repos that could use the same update treatment:

@neshumov
neshumov removed the request for review from vityevato September 29, 2026 11:36
@zzebrum
zzebrum requested review from dmitriivqa39 and removed request for fix-parrot October 5, 2026 17:34
@dmitriivqa39

Copy link
Copy Markdown
Contributor

Hi @dotext,
Thank you for your pull request! We follow a specific internal workflow for handling PRs, so I have incorporated all your fixes and merged them into our master branch. I also made sure to add you as a co-author on the commit.
You can view the commit here: efebc2a
Thanks again for your valuable contribution!

@dotext

dotext commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @dmitriivqa39, I appreciate you taking this on board. I'm not sure why the co-author didn't seem to work properly (it made my email address 17621430+dotext@://github.com instead of my one I use for GitHub), but that's okay!

@dmitriivqa39

Copy link
Copy Markdown
Contributor

@dotext
t appears that your real email address is hidden and a placeholder was used instead. If your actual email is the one you used to send logs for AdGuard Mini for Mac, I will use it for future PRs.

@dotext

dotext commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Oops, sorry about that @dmitriivqa39, I'm still new to actively contributing, as you can tell. I've been a lurker for too long. 😅

I've now resolved that, there should be a public email now on my profile that I will primarily use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants