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

Update VSPackage.resx #156

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Update VSPackage.resx #156

merged 1 commit into from
Jan 8, 2024

Conversation

engunneer
Copy link
Contributor

@CartBlanche
Copy link
Collaborator

@engunneer thanks for this fix.

Copy link
Collaborator

@CartBlanche CartBlanche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CartBlanche CartBlanche merged commit 6690823 into WildernessLabs:develop Jan 8, 2024
1 check passed
@engunneer
Copy link
Contributor Author

@CartBlanche, as an aside, I can't fully build because of references to Mono.Debugging. What else do I need to install to build this? There are other issues I considered looking in to, but got stuck there.

@CartBlanche
Copy link
Collaborator

CartBlanche commented Jan 8, 2024

@engunneer The dependency is on the Xamarin stack, which they now call ".NET Multiplatform App UI Development" (rolls off the tongue). Modify your VS installation to include that and you should be able to build the extension. We eventually want to remove that dependency and have it work similar to how the VSCode extension handles debugging. Do you have experience with VS Extension development?

@engunneer
Copy link
Contributor Author

No, not directly. I learn by trying to fix small issues, or at least by stepping through code to see how it ticks. The issue about not deploying to a connected f7 when targeting desktop seemed interesting to learn about. Thanks for the heads up on what dependency I was missing. I managed to not need it for regular meadow work, so haven't installed the whole workflow. I'm surprised VS and Google both didn't tell me that I needed it.

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.

2 participants