-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Revamp debugging section #3224
Revamp debugging section #3224
Conversation
- Most people leave the logs out so add a section for them in the issue template. - Remove the VS Code logs because those get sent to stdout now and will show up in our logs. - Separate browser console and network.
Codecov Report
@@ Coverage Diff @@
## main #3224 +/- ##
=======================================
Coverage 46.81% 46.81%
=======================================
Files 23 23
Lines 1194 1194
Branches 237 237
=======================================
Hits 559 559
Misses 451 451
Partials 184 184 Continue to review full report at Codecov.
|
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.
Looks good, but I'm confused about the minor inconsistency with ~/.local/share/code-server/logs
@@ -330,8 +330,8 @@ Once this is done, replicate the issue you're having then collect logging | |||
information from the following places: | |||
|
|||
1. The most recent files from `~/.local/share/code-server/coder-logs`. | |||
2. The most recently created directory in the `~/.local/share/code-server/logs` directory. |
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.
Line removed here, but still present in the issue template;
Once this is done, replicate the issue you're having then collect logging | ||
information from the following places: | ||
|
||
1. The most recent files from ~/.local/share/code-server/coder-logs. |
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.
Line added here, but removed from the debugging FAQ;
Ah yeah it looks a bit confusing because we have the unfortunate names of
|
Oh lordy - I can't believe I missed the |
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.
Looks good!
template.
show up in our logs.