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

Fix TailwindCSS Intellisense extension #817

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

FluxCapacitor2
Copy link

Description

This PR updates the TailwindCSS Intellisense (GitHub repo) extension entry in extensions.json to fix automated publishing to Open VSX.

Automated publishing stopped working after version 0.10.2, so I suspect the problem was initially caused by this commit.

This includes two changes:

  1. I changed the extension package's root directory to packages/vscode-tailwindcss. This is where the source for the VS Code extension lives.
  2. I switched from npm to pnpm. This is required because, as of this commit, the repository relies on the pnpm-specific workspace feature.

This works on my machine when testing with this command:

EXTENSIONS=bradlc.vscode-tailwindcss node publish-extensions.js

This generates an artifact at /tmp/artifacts/bradlc.vscode-tailwindcss.vsix, which I was able to install and use in VSCodium.

I looked at the GitHub Actions workflow file, and I couldn't see a step that installed pnpm, so I'm using it here via npx. The -y option disables prompting the user before automatically installing the package if it isn't found locally.

Signed-off-by: FluxCapacitor2 <31071265+FluxCapacitor2@users.noreply.github.com>
Signed-off-by: FluxCapacitor2 <31071265+FluxCapacitor2@users.noreply.github.com>
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