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 extension by upgrading vscode #27

Closed
wants to merge 2 commits into from

Conversation

searls
Copy link

@searls searls commented Mar 15, 2023

I was getting a lot of null/undefined errors when trying to use the extension (after months of happily using it without issue).

I pulled the code down and I wasn't able to do a clean yarn install because the typings file being downloaded in the postinstall script was 404'ing. So the only change I made was to run yarn upgrade vscode and that not only fixed the install issue, but also appears to have fixed the extension for me.

4 years is a long time. Please let me know if the project is still maintained and you're willing and able to publish this patch to the vscode store so the extension keeps working. If not, I'm happy to talk about taking it on under the @testdouble org.

searls added 2 commits March 15, 2023 09:28
Also:

* Change command name prefix so it won't conflict if both extensions are installed
* Use yarn consistently instead of mix-and-matching npm + yarn
@searls searls deleted the branch alternate-file:master March 23, 2023 22:08
@searls searls closed this Mar 23, 2023
@searls searls deleted the master branch March 23, 2023 22:08
@searls searls restored the master branch March 24, 2023 00:17
@searls
Copy link
Author

searls commented Mar 24, 2023

Assuming I won't hear back, I released a fix as this extension for now https://marketplace.visualstudio.com/items?itemName=testdouble.vscode-alternate-alternate-file

@searls searls reopened this Mar 24, 2023
@searls searls closed this Mar 24, 2023
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