sublimelsp / LSP-tailwindcss

Tailwind css support for Sublime's LSP plugin
MIT License
54 stars 5 forks source link

Switch to npm package #38

Closed predragnikolic closed 2 years ago

predragnikolic commented 2 years ago

It looks like tailwindcss language server is on npm :rocket:

Now we can rely on dependabot to create PR as soon as it detects new versions on npm.

rchl commented 2 years ago

Can also add dependabot configuration.

predragnikolic commented 2 years ago

Will wait with merging until this is tested on windows.

I do not have Windows, so I would have to setup a WM and install windows on it... (or hope that someone who uses windows now can test this :) )

rchl commented 2 years ago

I have Windows but won't be able to test until Monday at least.

predragnikolic commented 2 years ago

No worry, I already started downloading Windows 11. I will set it up in the following days.

Ultra-Instinct-05 commented 2 years ago

I am trying to test this on Windows 11. I cloned the repo and fetched the changes of this PR. However, I get this error in the console

reloading python 3.3 plugin LSP-tailwindcss.plugin
Traceback (most recent call last):
  File "C:\Sublime Text\Lib\python33\sublime_plugin.py", line 308, in reload_plugin
    m = importlib.import_module(modulename)
  File "./python3.3/importlib/__init__.py", line 90, in import_module
  File "<frozen importlib._bootstrap>", line 1584, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1565, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1529, in _find_and_load_unlocked
ImportError: No module named 'LSP-tailwindcss.plugin'
predragnikolic commented 2 years ago

VirtualBox_w11_21_04_2022_11_01_46

Tested in a Windows 11 VM, and it works :)

predragnikolic commented 2 years ago

@Ultra-Instinct-05 I do not know how to reproduce the issue you have experienced :) I will merge this and create a release.


If anyone reports any sort of issues, I will revert this.

Ultra-Instinct-05 commented 2 years ago

@predragnikolic After pulling in changes from master, it works as expected. Not sure what the previous issue was but can be ignored now. Thanks !