Skip to content

Conversation

dogadogan
Copy link
Contributor

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes
  • If data sources for any opportunity has been updated/added, please update the wiki for same opportunity.

Related Issues

Thanks for contributing!

Copy link

github-actions bot commented Sep 18, 2025

This PR will trigger a patch release when merged.

@dianaapredaa
Copy link
Contributor

Noticed that you placed the preflight readability check and the readability audit in the same folder. I think they should be stored separately to match the structure used for the other audits and preflight checks—each audit in its own folder, and all preflight handlers within the preflight folder.

'Use AI-generated suggestions as a starting point for improvements.',
],
},
tags: ['Content Quality', 'User Experience', 'SEO', 'Traffic Acquisition'],
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure all of these tags are relevant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated to ['Engagement', 'Accessibility']

src/index.js Outdated
'guidance:missing-alt-text': missingAltTextGuidance,
'guidance:readability': readabilityGuidance,
'guidance:readability': unifiedReadabilityGuidance, // unified for both preflight and opportunities
'readability-opportunities': readabilityOpportunities, // for opportunities
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the opportunity name should be only readability.

@dogadogan dogadogan requested a review from duynguyen October 13, 2025 13:31
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.

3 participants