Diamond
342 issues found
Potential edge case
Documentation issue
Code quality/style
Performance issue
Accidentally committed code
Security issue
Logic bug
1482
PRs reviewed
281
Issues led to fixes
3.5%
Comments downvoted
server.js
Javascript
+48
3 + const port = 3000;
Unresolved thread: line +3
Diamond1h ago
Logic bug
Change the port from 3000 to 8000 to match the exported port in the tests. The CI test is failing because it expects the server to run on port 8000.
Suggested change
3 - const port = 3000;
3 + const port = 8000;
Commit suggestion
Filters
Dialogs do not need explicit onClose handlers.
Don't leave comments about naming changes that could be considered nits