Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update sites.yml #852

Merged
merged 5 commits into from May 31, 2022
Merged

Update sites.yml #852

merged 5 commits into from May 31, 2022

Conversation

ghost
Copy link

@ghost ghost commented May 28, 2022

This PR is:

  • Adding a new domain
  • Updating existing domain size
  • Changing domain name
  • Removing existing domain from list
  • Website code changes (512kb.club site)
  • Other not listed
  • I used the uncompressed size of the site
  • I have included a link to the GTMetrix report
  • The domain is in the correct alphabetical order
  • This site is not a ultra lightweight site
  • The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

  • Check to confirm
- domain: catdrout.xyz
  url: https://catdrout.xyz
  size: 313
  last_checked: 2022-05-28

GTMetrix Report: https://gtmetrix.com/reports/catdrout.xyz/WD0Kk8lh/

Copy link
Collaborator

@garritfra garritfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you fix the alphabetical ordering? Thanks!

@ghost
Copy link
Author

ghost commented May 29, 2022

Fixed...

@garritfra
Copy link
Collaborator

It seems like you updated the size as well. Do you have a new report as a proof?🙂

@ghost
Copy link
Author

ghost commented May 29, 2022

give me 24 hours i need to fix something real quick

@ghost
Copy link
Author

ghost commented May 29, 2022


Merge

@garritfra
Copy link
Collaborator

I don't know if the results of Pingdom are on par with what we're using. Let's stick to GTMetrix, to keep it fair with everyone else.

@kmutahar
Copy link
Collaborator

Just a heads up GTMetrix is what we officially use, however if the site can not be scanned an alternative is used on a case by case.

@ghost
Copy link
Author

ghost commented May 30, 2022

https://gtmetrix.com/reports/catdrout.xyz/2PVmEzlH/

Page size 313 kb uncompressed

@garritfra garritfra merged commit 85ec634 into kevquirk:master May 31, 2022
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.

None yet

2 participants