Skip to content

Fix markdown-fix workflow to commit partial fixes when errors remain#291

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/fix-markdown-errors-in-pr-290
Closed

Fix markdown-fix workflow to commit partial fixes when errors remain#291
Copilot wants to merge 2 commits intomainfrom
copilot/fix-markdown-errors-in-pr-290

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 6, 2026

The @phlexbot format command in PR #290 failed because markdownlint-cli2-action exits with code 1 when errors are found, terminating the workflow before the commit step runs.

Changes

Details

- name: Run markdownlint
  uses: DavidAnson/markdownlint-cli2-action@07035fd053f7be764496c0f8d8f9f41f98305101
  with:
    globs: |
      ${{ env.local_checkout_path }}/**/*.md
      !${{ env.local_checkout_path }}/**/CHANGELOG.md
    fix: true
  continue-on-error: true  # Allows commit step to run even when errors remain

This matches the existing pattern in markdown-check.yaml line 84. The workflow now commits whatever fixes markdownlint can make automatically, even when some errors require manual intervention.

The analysis document provides the exact fixes needed for PR #290's remaining errors (MD040: missing code fence language, MD060: table formatting).

Original prompt

A @phlexbot format command to fix markdown errors in PR 290 was unsuccessful, per https://github.com/Framework-R-D/phlex/actions/runs/21736866948/job/62703615091. Please investigate and fix if possible.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix markdown errors from PR 290 Fix markdown-fix workflow to commit partial fixes when errors remain Feb 6, 2026
Copilot AI requested a review from greenc-FNAL February 6, 2026 03:05
@greenc-FNAL greenc-FNAL closed this Feb 6, 2026
@greenc-FNAL greenc-FNAL deleted the copilot/fix-markdown-errors-in-pr-290 branch March 10, 2026 16:41
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