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: use extension name as diagnostic source #297

Merged
merged 3 commits into from
Feb 9, 2021
Merged

fix: use extension name as diagnostic source #297

merged 3 commits into from
Feb 9, 2021

Conversation

bd82
Copy link
Member

@bd82 bd82 commented Feb 7, 2021

No description provided.

@bd82 bd82 requested a review from tal-sapan February 7, 2021 07:33
@bd82
Copy link
Member Author

bd82 commented Feb 7, 2021

image

@bd82 bd82 requested a review from sapirpol February 7, 2021 08:00
@bd82
Copy link
Member Author

bd82 commented Feb 7, 2021

I have random failures in the VSCode integration tests.

@tal-sapan any thoughts?

@tal-sapan
Copy link
Contributor

I have random failures in the VSCode integration tests.

@tal-sapan any thoughts?

No idea where that error is coming from. Are we creating it? Is it coming from another extension?

@bd82 bd82 merged commit f6d7dc4 into master Feb 9, 2021
@bd82 bd82 deleted the source_prop branch February 9, 2021 12:37
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.

None yet

2 participants