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

Installing this extension causes the Microsoft Python extension to crash #18

Closed
DonJayamanne opened this issue Dec 16, 2017 · 5 comments

Comments

@DonJayamanne
Copy link

Looks like there's something in this extension that causes the Microsoft Python Extension to crash.

See here for confirmation and further info:
microsoft/vscode-python#432 (comment)

Would like to resolve this, unfortunately the source is not available. Please let us know how we can help.

@chris-lauren
Copy link
Contributor

chris-lauren commented Dec 16, 2017 via email

@DonJayamanne
Copy link
Author

Sure thing, just let me know how I can help. For starters I'd love to try and help with this issue.

@ghost
Copy link

ghost commented Dec 16, 2017

The errors that I receive which I believe cause the Microsoft Python Extension to crash are

{
	"activationTimes": {
		"startup": false,
		"codeLoadingTime": 381,
		"activateCallTime": 0,
		"activateResolvedTime": 1889,
		"activationEvent": "onLanguage:python"
	},
	"runtimeErrors": [
		{
			"name": "TypeError",
			"message": "Cannot read property 'log' of undefined"
		},
		{
			"name": "TypeError",
			"message": "Cannot read property 'log' of undefined"
		},
		{
			"name": "TypeError",
			"message": "Cannot read property 'log' of undefined"
		},
		{
			"name": "TypeError",
			"message": "Cannot read property 'log' of undefined"
		}
	]
}

@DonJayamanne
Copy link
Author

Looks like this issue is back. We have updated reflect-metadata to the latest version in the Python extension.
Please could you also update your extension to the latest version of reflect-metadata.
Here's the link with details on the issue and the fix (fix = update reflect metadata to latest version) inversify/InversifyJS#737

@linmajia
Copy link
Contributor

Fixed in 0.1.6 last December.

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

No branches or pull requests

3 participants