54 votes
Accepted

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

It would be better to do this with a user style, e.g. using the Stylus extension, like this: .rep, .reputation, .reputation-score { visibility: hidden; } One ...
Ilmari Karonen's user avatar
29 votes

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

You could use querySelectorAll instead of getElementsByClassName. That way, you have a bit more flexibility in targeting ...
Joseph's user avatar
  • 25k
20 votes
Accepted

Where are my downvotes?

If this is actually your first SQL you're quite good in learning new languages. Some remarks: The Group By is not weird, it's the ...
dnoeth's user avatar
  • 1,172
16 votes

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

Update url UserScripts have the magical ability to update themselves if you provide a link to a raw version of your script as a UserScript header: Note: You actually have to increase the version, or ...
Quill's user avatar
  • 11.9k
14 votes
Accepted

Calculate questions per day on CodeGolf.SE

Your seconds_since_epoch function has a built-in Python equivalent, datetime.timestamp. Your namespaces would be cleaner if you ...
orlp's user avatar
  • 391
14 votes
Accepted

Badger - The tumbleweed detector

Conceptually, when you want to perform an action at a fixed rate, you want to use an ScheduledExecutorService. This class is designed to handle the use-case of ...
Tunaki's user avatar
  • 9,243
14 votes
Accepted

A wounded Zombie finder, or: how to find the almost dead undead?

Not utterly wrong, but q.PostTypeId = 1 and AND a.PostTypeId = 2 are superfluous. The only post types with a non-null ...
Glorfindel's user avatar
  • 1,089
13 votes
Accepted

Stack Exchange User Activity Score

TOP 1 When selecting values that correspond to Unique fields referenced by constants (OwnerUserId), you're only really ...
Quill's user avatar
  • 11.9k
13 votes

Stack Exchange rev​iew queue notifications

I don't see much benefit to the enum. The Notification.permission string (or the return value of requestPermission()) is already ...
CertainPerformance's user avatar
12 votes
Accepted

CodeReview question markdown downloader

PyCharm complains on this line: m = json.loads(data) If the above call to fetch_compressed_data fails, and the resulting ...
Carcigenicate's user avatar
11 votes
Accepted

Stack Exchange Chat Caret Pathfinder

There is still code-golf Here you have some golfed code: function(e){e.style.background = "";} ungolf it: ...
Downgoat's user avatar
  • 902
10 votes

CodeReview question markdown downloader

Just a couple stylistic points Function/Variable Naming Functions and variables should be in snake_case (PEP 8). ...
Linny's user avatar
  • 10.1k
8 votes

From Q to compiler in less than 30 seconds

I have a couple of suggestions which are not specifically about the code, but rather about the user experience. The first thing I did was copy-paste the script to a file, make it executable and tried ...
jacwah's user avatar
  • 2,661
8 votes
Accepted

Simple CodeReview Questions Notifications for OSX

Nitpicks Maximum line length should be 72 characters for docstrings. For your old variable, you may wan to use a set instead of ...
301_Moved_Permanently's user avatar
8 votes

CodeReview question markdown downloader

Improving HTTP communication scheme Instead of urllib.request/urllib.error use requests lib as well-known, elegant and simple ...
RomanPerekhrest's user avatar
7 votes

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

Instead of document.getElementsByClassName, you can use document.querySelectorAll(CSS selector). Edit(Reply to @200_success): ...
Константин Ван's user avatar
7 votes

Stack Exchange Chat Caret Pathfinder

One line variable declaration Please avoid declaring variables on the same line: var elements = {}, messages; It's also a great way to royally mess up scope ...
Quill's user avatar
  • 11.9k
7 votes
Accepted

From Q to compiler in less than 30 seconds

My answer's short because I don't see much wrong with it, but this is what I think: In fetch_compressed_data(), you define data ...
zondo's user avatar
  • 3,797
7 votes

Calculate questions per day on CodeGolf.SE

Code Style import requests, datetime, time Imports should not be on one line, as per PEP8 Your code doesn't meet the "too many characters on one line" rule of ...
Quill's user avatar
  • 11.9k
7 votes

Showing links to Contributor Breakdown on Stackoverflow Docs

You're using 'use strict'; in all functions except the top one. Why not declare it at the top of your file? You seem to have thought about readability. You're ...
Mast's user avatar
  • 13.4k
7 votes
Accepted

Python tool to assemble CodeReview posts from source files

PEP 8 I would review PEP 8. Some of your style choices are not considered best practice by the Python community. Some quick observations: Constants are given ...
Dair's user avatar
  • 5,864
7 votes
Accepted

Zombie killers ranking

As a heads up - we have very different formatting styles for our code. Feel free to ignore that difference and don't consider it a comment on your style (unless you prefer mine, in which case please ...
Dan Oberlam's user avatar
  • 7,804
6 votes

Stack Exchange User Activity Score

This is too much for a comment, adding to @Quill's answer: In your Select you use FROM Users, Comments, Posts which is an unconstraint cross join, i.e. every row ...
dnoeth's user avatar
  • 1,172
6 votes
Accepted

Show how close a user is to being unsung

I tried it on myself, elapsed time of 136 msec for userid 145459: https://data.stackexchange.com/codereview/query/1822723/how-close-am-i-to-being-unsung?opt.withExecutionPlan=true#executionPlan The ...
J_H's user avatar
  • 25k
5 votes

"How can I make this SEDE query better?" (bad title finder)

Let me address a possible DRY issue in the query in exchange for possible poorer performance. I'm in particular talking about this bit: ...
rene's user avatar
  • 305
5 votes

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

Since you're using ES6, you could replace your slice function with Array.from like this (implementing Josephs changes too); <...
Jivings's user avatar
  • 2,291
5 votes

You hate h̶a̶t̶s̶ reputation? Click here to get rid of it

Avoid using deprecated events Mutation Events (DOMNodeInserted) is deprecated and you should use Mutation Observers instead. Sample usage: ...
Bob's user avatar
  • 459
5 votes
Accepted

Adding links to tags on StackExchange sites

Since Stack Exchange already uses jQuery, I think it's a good idea to write userscripts using jQuery too. However, rewriting the whole code to use jQuery instead of native APIs would require a lot of ...
Michał Perłakowski's user avatar
5 votes
Accepted

SmokeDetector report notifier

A quote from another answer of mine: You don't use quotes or apostrophes for string literals consistently. Decide if you want to write string using either apostrophes or double quotes and use it ...
Michał Perłakowski's user avatar

Only top scored, non community-wiki answers of a minimum length are eligible