Skip to content

Conversation

@coliff
Copy link
Member

@coliff coliff commented Jun 20, 2025

This pull request introduces a new feature to the vscode-htmlhint extension, allowing users to skip linting files ignored by .gitignore. It also includes updates to handle .gitignore files, improve diagnostics, and update dependencies. Below is a summary of the most important changes:

New Feature: .gitignore Integration

  • Added a new configuration option htmlhint.ignoreGitignore to skip linting files ignored by .gitignore. This feature can be enabled in VS Code settings. [1] [2] [3] [4]
  • Implemented logic to parse .gitignore files and determine whether files should be ignored during validation. This includes caching for performance and clearing the cache when settings or .gitignore files change. [1] [2] [3] [4] [5]

Diagnostics and Debugging Enhancements

  • Added debug logging for code actions and .gitignore-related operations to improve traceability. [1] [2]
  • Updated validation logic to clear diagnostics for ignored files, ensuring no unnecessary warnings are displayed.

Dependency and Configuration Updates

  • Updated package.json and package-lock.json to include the ignore library for .gitignore handling. [1] [2]
  • Bumped the extension version to 1.11.0 and updated the changelog to document the new feature. [1] [2]

File Watcher Enhancements

  • Added a file system watcher for .gitignore files to trigger revalidation when they are modified. [1] [2]

General Codebase Improvements

  • Updated .cursor/rules/general.mdc with guidelines for maintaining alphabetical order in rules, attributes, and lists.

gemini-code-assist[bot]

This comment was marked as resolved.

@coliff coliff changed the title feat: add .gitignore option feat: add .gitignore option Jun 20, 2025
@coliff coliff merged commit 826994c into main Jun 20, 2025
10 checks passed
@coliff coliff deleted the dev/coliff/add-git-ignore-feature branch June 20, 2025 04:24
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