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

Add support for devcontainer locally and remotely #83

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RadWolfie
Copy link
Member

I had not thoroughly tested this template for visual studio code. There could be some unnecessary extension, intellisense issues, etc.

However, this template had been my goto devcontainer for other nxdk relative projects. Feel free to find any faults and note them here. And feel free to create codespace, from GitHub, to try them out from the web browser. Please remember to delete your codespace creation after being done with it.

@RadWolfie
Copy link
Member Author

I recently made an discovery for intelliSenseMode need windows- prefix in order to resolve L"string" misunderstanding wide char is not 4 bytes but actually 2 bytes long.

Other than that, the current issue is XBAPI/NTAPI/CDECL/FASTCALL/DECLSPEC_NORETURN are producing false positive errors. Unless they were replaced with __stdcall and the like. I'm not sure about DECLSPEC_NORETURN's issue to resolve this.

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