Skip to content

Conversation

@alaahong
Copy link
Member

Purpose of the pull request

chore: add markdown lint check for markdown files on push and pull request
Related: #423

What's changed?

.github/workflows/markdownlint.yml

Checklist

  • I have read the CONTRIBUTING guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds automated markdown linting to the FastExcel project by introducing a GitHub Actions workflow that checks markdown files for formatting issues on push and pull request events.

  • Adds markdownlint validation for markdown files using markdownlint-cli2
  • Creates PR comments with detailed lint results when validation fails
  • Includes file contents in lint failure comments for easy review

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

alaahong and others added 5 commits August 13, 2025 20:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@delei delei merged commit 672afa2 into apache:main Aug 13, 2025
5 checks passed
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