Skip to content

fix: include LICENSE in VSIX package#959

Merged
HighCommander4 merged 1 commit into
clangd:masterfrom
stellhub:stellhub/include-license-in-vsix
May 12, 2026
Merged

fix: include LICENSE in VSIX package#959
HighCommander4 merged 1 commit into
clangd:masterfrom
stellhub:stellhub/include-license-in-vsix

Conversation

@stellhub
Copy link
Copy Markdown
Contributor

Summary

  • allow the root LICENSE file through .vscodeignore
  • ensure the packaged VSIX includes the extension license file

Closes #958

Testing

  • npm test
  • npm run package
  • verified the generated VSIX now contains LICENSE.txt

@stellhub
Copy link
Copy Markdown
Contributor Author

Thanks for taking a look. This PR comes from a fork, and GitHub Actions is currently blocked with an action_required status before any jobs can start. I have already run the relevant checks locally:\n- npm test\n- npm run package\n- verified the generated VSIX now contains LICENSE.txt\n\nCould a maintainer please approve and run the workflows for this PR?

@HighCommander4 HighCommander4 merged commit 4be7f90 into clangd:master May 12, 2026
1 check passed
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.

License file is not packaged

2 participants