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

Enable arm64 vsix #990

Merged
merged 1 commit into from
Mar 15, 2023
Merged

Enable arm64 vsix #990

merged 1 commit into from
Mar 15, 2023

Conversation

GrahamTheCoder
Copy link
Member

@GrahamTheCoder GrahamTheCoder commented Mar 13, 2023

Potentially fixes #989

This should make the extension installable on VS2022 on arm. I don't have a device to test it with, and even if it installs it may not work. Any feedback would be appreciated

ICSharpCode.CodeConverter.VsExtension.zip

@christophwille
Copy link
Member

I have an ARM device - do we have a sort-of go-to standard test project for testing conversion?

@andriysavin
Copy link

Just tried on Windows on Arm in Parallels VM on Apple Silicon, works with no problems

@GrahamTheCoder GrahamTheCoder merged commit 0790f4e into master Mar 15, 2023
@GrahamTheCoder GrahamTheCoder deleted the arm64 branch March 15, 2023 09:30
@GrahamTheCoder
Copy link
Member Author

@christophwille You could use the solution that the automated tests convert. https://github.com/icsharpcode/CodeConverter/tree/master/Tests/TestData/MultiFileCharacterization/SourceFiles
It'd also be interesting to run the automated tests in arm64 visual studio. I hadn't thought of the fact that the github actions may be runnable on different platforms

@christophwille
Copy link
Member

Yes, running the tests on arm64 would be useful.

@christophwille
Copy link
Member

Gave it a spin on 17.5.2 on Surface X Windows 11 Moment 2 (latest everything basically) - works.

@GrahamTheCoder
Copy link
Member Author

Great, thanks!

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.

Support ARM64 version of the extension
3 participants