Skip to content

Conversation

joshjohanning
Copy link
Owner

  • Add check for files tracked in Git but missing from filesystem
  • Update error message to distinguish between files and directories
  • Prevent crashes when staged files have been deleted locally
  • Add fs.existsSync() checks before fs.statSync() calls

@joshjohanning joshjohanning requested a review from Copilot October 2, 2025 18:46
Copy link

github-actions bot commented Oct 2, 2025

📋 Lint Results

⚡ ./gh-cli scripts

✅ No issues found.

🔧 ./scripts scripts

✅ No issues found.

Lint results updated at Thu Oct 2 19:09:55 UTC 2025

@joshjohanning joshjohanning requested review from Copilot and removed request for Copilot October 2, 2025 19:02
Copy link

@Copilot 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 improves the robustness of the lint-readme.js script by adding better error handling for missing files and enhancing error messages to distinguish between files, directories, and items that don't exist.

  • Adds filesystem existence checks before attempting to access file stats
  • Introduces a helper function to determine item types (file/directory/non-existent)
  • Updates error messages to be more specific about whether an item is a file or directory
  • Prevents crashes when Git-tracked files have been deleted locally

@joshjohanning joshjohanning requested a review from Copilot October 2, 2025 19:10
Copy link

@Copilot 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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

@joshjohanning joshjohanning merged commit 3c68761 into main Oct 2, 2025
7 checks passed
@joshjohanning joshjohanning deleted the update-linting branch October 2, 2025 19:12
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.

1 participant