Skip to content

[internal test] Test new PR automated validation #2557

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pwizla
Copy link
Collaborator

@pwizla pwizla commented Jun 24, 2025

No description provided.

@pwizla pwizla requested a review from meganelacheny as a code owner June 24, 2025 14:52
Copy link

vercel bot commented Jun 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2025 3:51pm

Copy link

❌ Documentation Style Review

Validation script failed to run. Please check the GitHub Action logs.

This might be due to missing files or configuration issues.

Copy link

🎯 Strapi Documentation Style Review

Based on Strapi's 12 Rules of Technical Writing

📊 Summary

  • Files checked: 1
  • Total issues: 2
  • Critical errors: 2 🚨
  • Warnings: 0 ⚠️
  • Suggestions: 0 💡

🚨 Critical Issues (must fix)

docs/new-file.md:2

  • CRITICAL: Never use "easy" - it can discourage readers (Rule 6)
  • 💡 Remove subjective difficulty assessment and provide clear instructions instead

docs/new-file.md:2

  • CRITICAL: Never use "simple" - it can discourage readers (Rule 6)
  • 💡 Remove subjective difficulty assessment and provide clear instructions instead

🤖 Automated review based on Strapi's 12 Rules of Technical Writing

Copy link

🎯 Strapi Documentation Style Review

Based on Strapi's 12 Rules of Technical Writing

📊 Summary

  • Files checked: 1
  • Total issues: 4
  • Critical errors: 1 🚨
  • Warnings: 2 ⚠️
  • Suggestions: 1 💡

🚨 Critical Issues (must fix)

docs/whats-new.md:27

  • CRITICAL: Never use "hard" - it can discourage readers (Rule 6)
  • 💡 Remove subjective difficulty assessment and provide clear instructions instead

⚠️ Warnings (should address)

docs/whats-new.md:21 - Long enumeration detected (3 items) - use bullet list instead (Rule 8)
docs/whats-new.md:25 - Long enumeration detected (3 items) - use bullet list instead (Rule 8)

💡 Suggestions (improvements)

docs/whats-new.md:35 - Too many pronouns (5) - avoid "you/we" in technical docs (Rule 11)


🤖 Automated review based on Strapi's 12 Rules of Technical Writing
🔧 Manually triggered for PR #2557

Copy link

🎯 Strapi Documentation Style Review

Based on Strapi's 12 Rules of Technical Writing

📊 Summary

  • Files checked: 1
  • Total issues: 2
  • Critical errors: 2 🚨
  • Warnings: 0 ⚠️
  • Suggestions: 0 💡

🚨 Critical Issues (must fix)

docs/new-file.md:2

  • CRITICAL: Never use "easy" - it can discourage readers (Rule 6)
  • 💡 Remove subjective difficulty assessment and provide clear instructions instead

docs/new-file.md:2

  • CRITICAL: Never use "simple" - it can discourage readers (Rule 6)
  • 💡 Remove subjective difficulty assessment and provide clear instructions instead

🤖 Automated review based on Strapi's 12 Rules of Technical Writing
🔧 Manually triggered for PR #2557

@pwizla pwizla self-assigned this Jun 25, 2025
@pwizla pwizla added the flag: don't merge This PR should not be merged at the moment label Jun 25, 2025
@pwizla pwizla force-pushed the main branch 2 times, most recently from 25723df to 63466f0 Compare June 27, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flag: don't merge This PR should not be merged at the moment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant