Skip to content

test: add workflow verification files for post-merge autofix check - #731

Closed
hilram7 wants to merge 6 commits into
devfrom
test/post-merge-autofix-verification
Closed

test: add workflow verification files for post-merge autofix check#731
hilram7 wants to merge 6 commits into
devfrom
test/post-merge-autofix-verification

Conversation

@hilram7

Copy link
Copy Markdown
Collaborator

Purpose

This is a post-merge verification PR for #730. It is intentionally not for merging — open to trigger the md-extension-autofix workflow, verify the bot comment, then close.

What to verify

Once the workflow runs, check the bot comment on this PR against the expected outcomes below:

FileExpected outcome
wf-test-no-ext-no-frontmatterRenamed to .md AND appears in the frontmatter injected table
wf-test-no-ext-has-frontmatterRenamed to .md only — should NOT appear in frontmatter table
wf-test-has-ext-no-frontmatter.mdNot renamed + appears in the frontmatter injected table
wf-test-kb-no-frontmatterRenamed to .md only — should NOT appear in frontmatter table (KB exclusion)
wf-test-no-headingAppears in the "Action needed" skipped table

Do not merge. Close this PR once the workflow output is verified.

@github-actions

Copy link
Copy Markdown
Contributor

Auto-fix: Missing .md extensions added

The following files were missing a .md extension and would not have appeared on the docs site. They've been renamed automatically:

Original filenameRenamed to
docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-no-ext-has-frontmatterwf-test-no-ext-has-frontmatter.md
docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-no-ext-no-frontmatterwf-test-no-ext-no-frontmatter.md
docs/kb/accessanalyzer/wf-test-kb-no-frontmatterwf-test-kb-no-frontmatter.md

Links to these files in other pages have been updated. Please review the changes in the commit above.

Frontmatter injected

The following files were missing frontmatter. title, description, and sidebar_position were derived automatically and may need manual review:

File
docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-has-ext-no-frontmatter.md
docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-no-ext-no-frontmatter.md

Action needed: Possible missing .md extension

The following files were added to a docs directory without a .md extension, but couldn't be auto-renamed:

FileReason
docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-no-headingContent doesn't look like markdown — please check and rename manually if needed

If these are markdown pages, add the .md extension and push again.

@github-actions

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 0 skipped across 1 files

Ask @claude on this PR if you'd like an explanation of any fix.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/accessanalyzer/12.0/admin/jobs/instantjobs/wf-test-has-ext-no-frontmatter.md

No issues found. This file is a workflow test fixture, not documentation content — it contains no prose to review editorially.

Note: The PR diff also includes several files without .md extensions (wf-test-no-ext-has-frontmatter, wf-test-no-ext-no-frontmatter, wf-test-no-heading). These are also test fixtures and were not reviewed.

Summary

0 editorial suggestions. This PR contains only workflow verification test files with no documentation content to review. Vale and Dale issues are auto-fixed separately.


What to do next:

Comment @claude on this PR followed by your instructions to get help:

  • @claude fix all issues — fix all editorial issues
  • @claude help improve the flow of this document — get writing assistance
  • @claude explain the voice issues — understand why something was flagged

You can ask Claude anything about the review or about Netwrix writing standards.

Automated fixes are only available for branches in this repository, not forks.

@hilram7

Copy link
Copy Markdown
CollaboratorAuthor

Closing after successful end-to-end verification

  • The workflow correctly processed all 5 test cases.
  • See the auto-fix comment above for full results.
  • The bot commit appears in the Files Changed tab — confirming that renamed files are now pushed to the PR branch as intended.

Fix history

@hilram7hilram7 closed this Apr 9, 2026
@hilram7
hilram7 deleted the test/post-merge-autofix-verification branch April 9, 2026 22:49
Sign up for freeto 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.

1 participant

@hilram7