-
-
Notifications
You must be signed in to change notification settings - Fork 909
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
Server: Include the file path in error messages when running external files #2470
Server: Include the file path in error messages when running external files #2470
Conversation
Should hopefully fix sonic-pi-net#2336
😂 as I recall, pretty much the same solution I came up with on my machine also - just never got round to cleaning it up and creating a PR 🙂 I'll double check this, but pretty sure it will be good to go. Thanks! |
572acdf
to
bb447f3
Compare
Lovely work, thanks! |
Sorry, I had to revert this as it was buggy. Please do thoroughly test your code before submitting a pull request. Perhaps next time squash all your commits into a single change so it's easier to triage. Happy to consider a fixed version of this PR. |
@samaaron I understand :), I think I tested it a while ago and it seemed to work, but I admit I wasn't that thorough. |
Should hopefully fix #2336 🙂 Yet to be tested