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

Hyperlint Automation: Meta Description Fixes #19357

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

hyperlint-ai[bot]
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot commented Jan 22, 2025


🪄 Experimental Hyperlint Feature - Read more 👇

This pull request is part of a series of Hyperlint automations to ease the burden of maintaining documentation.

We'll be introducing this automation, and others, over the coming months. These automations will always open pull requests(or issues) prior to making changes.

If this is not the change you want, please close this pull request.

⚠️ Important Note:
This feature may introduce erroneous changes (although we work really hard to make sure that doesn't happen). Please review the changes before merging.

❌ Limitations:
At this time, our bot can't respond to pull requests or issues.

📩 Need Help?
Please reach out to support with any requests, questions, or concerns.

@hyperlint-ai hyperlint-ai bot requested review from zeinjaber, ngayerie and a team as code owners January 22, 2025 16:56
@bllchmbrs
Copy link
Contributor

bllchmbrs commented Jan 22, 2025

@kodster28 @pedrosousa , meta descriptions! I believe this is the right format, let me know if not! Ignore the empty PR body, that will be fixed.

Copy link

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: f06f01d
Status: ✅  Deploy successful!
Preview URL: https://da9ce7d3.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://hyperlint-scan-meta-descript-ynxj.cloudflare-docs-7ou.pages.dev

View logs

@kodster28
Copy link
Contributor

kodster28 commented Jan 22, 2025

Howdy @bllchmbrs, lgtm.

Is there any way we could batch these to cover the whole of a folder at a time? So like one for all of /support, then /rules, then /dns etc.

If not, that's totally fine... just thinking that process-wise the meta description doesn't affect anything customer-facing + if it's adding something net new and less risky, so reviewing like 4000 PRs for all our pages might be a touch cumbersome.

There might be a similar thought / tradeoff / balance between these other types of things.... and it might vary for the type of ask / whether it's already a standard practice.

  • Meta descriptions --> For us, not really used extensively, should prioritize for scale / adding a base level of them
  • Alt text --> Generally there, might make sense to run over a folder to suggest improvements... or just gradually at one-off examples.
  • Anchor links --> Generally there, gradually heal.

@kodster28 kodster28 merged commit f564976 into production Jan 23, 2025
11 checks passed
@kodster28 kodster28 deleted the hyperlint/scan/meta-description/4e539e6 branch January 23, 2025 14:03
@workers-devprod workers-devprod added the contribution [Holopin] Recognizes a docs contribution, big or small label Jan 23, 2025
Copy link

holopin-bot bot commented Jan 23, 2025

Congratulations @Hyperlint-AI[bot], the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cm69el9ll35750clcx8hoi8q0

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution [Holopin] Recognizes a docs contribution, big or small product:support size/xs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants