-
Notifications
You must be signed in to change notification settings - Fork 8
Troubleshooting #241
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
Troubleshooting #241
Conversation
|
@hardy-ethan I would value your feedback if you can take a look. You created some of the issues. |
|
The spec from this PR is available at https://preview.sesh.rs/previews/eecs280staff/tutorials/241/. (Available until Wed Jan 21 2026.) |
|
sure!
…On Fri, Dec 19, 2025 at 10:21 AM Andrew DeOrio ***@***.***> wrote:
*awdeorio* left a comment (eecs280staff/tutorials#241)
<#241 (comment)>
@hardy-ethan <https://github.com/hardy-ethan> I would value your feedback
if you can take a look. You created some of the issues.
—
Reply to this email directly, view it on GitHub
<#241 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYQTEUNRU5LBLXHWRJJI6CT4CQJW3AVCNFSM6AAAAACPRU5YF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMNZVGQ3DSOBSGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
jamesjuett
left a comment
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.
Left a typo fix commit and a few minor suggestions.
|
I'll reply to Prof. Juett's comments for the other two issues, but for here: #204I'm thinking that in the case where the user has VSCode installed in the Downloads folder, they probably had it installed from a previous class or project, and might not remember that it's in the Downloads folder. It might be easier for them to look for the error message that VSCode shows:
Maybe like (
Probably should've included that error text in my original issue... |
|
Thanks for the feedback. I think I got it all. Take another look. |
For #204 I only put the bolding to show that it was an addition, but if you think it looks good in the text feel free to keep 😂 Planning on testing the two other issues on a Windows install if I can get one running |
jkloosterman
left a comment
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.
Not able to test myself but looks reasonable
jamesjuett
left a comment
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.
LGTM.
Done! Thanks for testing this. 1bebad3 |
Add a few troubleshooting items suggested in 3 issues.
Closes #204, closes #209, closes #231