search

ObliviousHarmony / vscode-php-codesniffer

A VS Code extension for integrating PHP_CodeSniffer.
https://marketplace.visualstudio.com/items?itemName=obliviousharmony.vscode-php-codesniffer
Other
38 stars 2 forks source link

Separate Document Change & Save Handling #84

Closed ObliviousHarmony closed 10 months ago

ObliviousHarmony commented 10 months ago

All Submissions:

Changes proposed in this Pull Request:

Previously there was a problem where document saves would squash the debounce for updates with a different event. There isn't really a reason to debounce the save handler and so I've separated the two handlers.

How to test the changes in this Pull Request:

  1. Make sure linting on save works.
  2. Make sure linting on change works.
  3. Make sure linting on open works.