-
Notifications
You must be signed in to change notification settings - Fork 18
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
Added Support for Code Runner #6
base: main
Are you sure you want to change the base?
Conversation
Tested on Friends Macbook and it works there too 🎉. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great addition, and I'm glad you found easy-masm helpful :)
I've just added some suggested changes.
PS. I'm thrilled to see people around the world using easy-masm. Please feel free to send this repo to your colleagues as well as your professor. My university now incorporates easy-masm into the curriculum of our assembly class, and yours could too. It makes life much better on the future students who take this class.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may be a good idea to adjust the instructions slightly to make them apply to a single workspace, rather than the entire User profile. The reasoning is that EasyMASM only exists in a single folder.
This can be achieved my simply adding a step to switch from the User
tab to the Workspace
tab in the extension settings.
I don't use code runner or do much vscode customization stuff so I don't know much about this... but yea I think making it workspace specific would probably be better than user specific. I wonder if there's a way to automatically handle this by adding a |
Unfortunately, One solution would be to build a small program to detect the user's platform and run the right script? |
Hi there, thanks for you response, added the changes. |
Added support for coderunner. I currently only have a linux machine but I'll try adding support for MacOs and Windows. If you find anything wrong in the PR please make sure to let me know about it.
Btw if the mod is reading this: Thank you so much for this repo, I am on linux and my university only allows MASM, so this is a great help. Thanks alot.