Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Warn for old Stylelint regardless of language ID #340

Merged
merged 1 commit into from
Nov 20, 2021

Conversation

adalinesimonian
Copy link
Member

Closes #333

Does not warn if Stylelint was resolved globally, only locally.

Does not warn if Stylelint was resolved globally, only locally.
@adalinesimonian adalinesimonian added this to the v1.3.0 milestone Nov 19, 2021
@adalinesimonian adalinesimonian self-assigned this Nov 19, 2021
@adalinesimonian adalinesimonian added this to In progress in v1.3 via automation Nov 19, 2021
v1.3 automation moved this from In progress to Reviewer approved Nov 20, 2021
@adalinesimonian adalinesimonian merged commit e0568ee into v1.3 Nov 20, 2021
@adalinesimonian adalinesimonian deleted the warn-all-languages branch November 20, 2021 01:33
v1.3 automation moved this from Reviewer approved to Done Nov 20, 2021
ota-meshi added a commit that referenced this pull request Nov 2, 2023
* feat: Allow disposing server, add first LSP test (#326)

* feat: Allow disposing server, add first LSP test

* test: close connection before finishing

* refactor: use stylelint namespace for custom event

* feat: Add restart server command (#339)

Closes #332

* test: add code actions on save end-to-end test (#341)

* test: add code actions on save end-to-end test

* test:test auto-fix on save after problem actions

Quick hack to hopefully stabilize tests on CI for now

* feat: Warn for old Stylelint regardless of lang ID (#340)

Does not warn if Stylelint was resolved globally, only locally.

* test: await functions that may return promises

* fix: de-nest globs used for watching config files (#356)

* fix: de-nest globs used for watching config files

* test: update extension tests

* docs: update changelog

* fix: ts error (#471)

* feat: Report ranges for diagnostics (#358)

* feat: Report ranges for diagnostics

* update snap

* update changelog

* update stylelint

* Fixes server/modules/formatter tests (#450)

Fix copied from original PR

https://github.com/stylelint/vscode-stylelint/pull/426/files#diff-f2680deb1906c0920d4e4b4f73879a32a741ed4288aabb2a218a2eef48f38211

* update snapshot

* update snapshot

* fix

* fix: test cases

---------

Co-authored-by: yosuke ota <otameshiyo23@gmail.com>
Co-authored-by: Chema Balsas <jbalsas@gmail.com>

---------

Co-authored-by: Adaline Valentina Simonian <adalinesimonian@gmail.com>
Co-authored-by: Chema Balsas <jbalsas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
v1.3
Done
Development

Successfully merging this pull request may close these issues.

Show old Stylelint warning regardless of if document is configured to be linted
2 participants