Skip to content

Conversation

@mcsf
Copy link
Contributor

@mcsf mcsf commented Nov 12, 2025

Testing #232

@mcsf mcsf force-pushed the test/docs-workflow-should-pass branch from 6cc5f87 to d1af25e Compare November 12, 2025 16:46
@mcsf mcsf changed the base branch from add/ci-check-docs-manifest to trunk November 12, 2025 16:47
@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props mcsf.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 1.17%. Comparing base (ac3055a) to head (e750832).

Additional details and impacted files
@@           Coverage Diff           @@
##             trunk    #236   +/-   ##
=======================================
  Coverage     1.17%   1.17%           
  Complexity    4416    4416           
=======================================
  Files          261     261           
  Lines        27017   27017           
=======================================
  Hits           318     318           
  Misses       26699   26699           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcsf
Copy link
Contributor Author

mcsf commented Nov 12, 2025

Nice, it works. I first pushed a change to the contents of a docs file, which triggered the workflow but caused no failure, as expected. I then pushed a file rename, which definitely requires a new manifest, and the workflow correctly failed:

Screenshot 2025-11-12 at 16 51 17 Screenshot 2025-11-12 at 16 51 25

The work here is done. Closing.

@mcsf mcsf closed this Nov 12, 2025
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