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

vscode-css-languageservice/test/pathCompletionFixtures/src/data/foo.asar fails the compilation #108016

Closed
PF4Public opened this issue Oct 3, 2020 · 8 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug css-less-scss Issues and items concerning CSS,Less,SCSS styling verified Verification succeeded
Milestone

Comments

@PF4Public
Copy link

PF4Public commented Oct 3, 2020

  • VSCode Version: latest
  • OS Version: linux

Steps to Reproduce:

  1. cd extensions/css-language-features/server/ && yarn install

Result:

$ cd server && yarn install
yarn install v1.22.4
warning ../../package.json: No license field
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
error https://registry.yarnpkg.com/vscode-css-languageservice/-/vscode-css-languageservice-4.3.4.tgz: Extracti
ng tar content of undefined failed, the file appears to be corrupt: "Invalid package /var/tmp/portage/app-edit
ors/vscode-9999/homedir/.cache/yarn/v6/npm-vscode-css-languageservice-4.3.4-8d8711b31f2b561f78c32e8c85a8584b0b
6dc43b-integrity/node_modules/vscode-css-languageservice/test/pathCompletionFixtures/src/data/foo.asar"

--skip-integrity-check does not help.

I think, there was no such error with versions 1.49.2 (and probably 1.49.3) and older.

Does this issue occur when all extensions are disabled?: N/A

@aeschli
Copy link
Contributor

aeschli commented Oct 12, 2020

You were right, microsoft/vscode-css-languageservice@a5de5fa was the cause.
Published vscode-css-languageservice@4.3.5

@PF4Public
Copy link
Author

Thanks @aeschli
But without bumping it in this repo, it will still pull in 4.3.4 and fail.

@aeschli
Copy link
Contributor

aeschli commented Oct 13, 2020

done: 362da0a

@aeschli aeschli added this to the October 2020 milestone Oct 13, 2020
@aeschli aeschli added css-less-scss Issues and items concerning CSS,Less,SCSS styling bug Issue identified by VS Code Team member as probable bug labels Oct 13, 2020
@PF4Public
Copy link
Author

Thanks @aeschli
Haven't you forgot ;)

"vscode-css-languageservice": "^4.3.4",

@aeschli
Copy link
Contributor

aeschli commented Oct 13, 2020

Thanks, will update.

aeschli added a commit that referenced this issue Oct 13, 2020
@PF4Public
Copy link
Author

Thanks @aeschli it compiles fine now.

I have inspected a build log after 362da0a, which should fail, but that action on GitHub in this repository finished successfully:

Installing dependencies in extensions/html-language-features...
$ yarn --frozen-lockfile
yarn install v1.22.5
warning ../package.json: No license field
[1/5] Validating package.json...
warning html-language-features@1.0.0: The engine "vscode" appears to be invalid.
[2/5] Resolving packages...
[3/5] Fetching packages...
warning vscode-extension-telemetry@0.1.1: The engine "vscode" appears to be invalid.
warning vscode-languageclient@7.0.0-next.5.1: The engine "vscode" appears to be invalid.
[4/5] Linking dependencies...
[5/5] Building fresh packages...
$ cd server && yarn install
yarn install v1.22.5
warning ../../package.json: No license field
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
info fsevents@2.1.3: The platform "linux" is incompatible with this module.
info "fsevents@2.1.3" is an optional dependency and failed compatibility check. Excluding it from installation.
[4/5] Linking dependencies...
[5/5] Building fresh packages...

I assume, that because of that, this issue was not caught immediately by GitHub actions.

How could this be so that on my system it fails due to .npmignore, but on GitHub actions this never happens?

jmannanc pushed a commit to jmannanc/vscode that referenced this issue Oct 26, 2020
@aeschli
Copy link
Contributor

aeschli commented Oct 29, 2020

To verify check that the published vscode-css-languageservice module (https://unpkg.com/browse/vscode-css-languageservice/) does not contain the test folder

@PF4Public
Copy link
Author

does not contain the test folder

True, but https://unpkg.com/browse/vscode-css-languageservice@4.3.4/ has it.

@roblourens roblourens added the verified Verification succeeded label Oct 29, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug css-less-scss Issues and items concerning CSS,Less,SCSS styling verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants