4 ms·
There is a special place in hell for anyone doing this. I'm going to watch this repo and blacklist pull requests from anyone who forks it :-)
by sheraz 11y ago
There is a special place in hell for anyone doing this. I'm going to watch this repo and blacklist pull requests from anyone who forks it :-)
- creshal 11y agoThey share the place with coding blogs that use instead of spaces for code snippets.
- minikomi 11y agoOr “real” quotes in code examples..
- scintill76 11y agoOr en-dashes instead of double hypens for command line flags... I can feel my blood pressure rising just thinking about it.
- zaptheimpaler 11y agoDamn you Google docs! they also do this :(
- TorKlingberg 11y agoI don't think people do this intentionally. Either the code snippet has passed through MS Word (why?) or their blog tool is being "helpful".
- harryc2011 11y agoIt happens if you paste the code into an Outlook email (which used Word as the rendering engine IIRC)
- jff 11y agoI once had to do a team project with another student who did all his coding in Wordpad, god knows why. His indentation was more or less random. I wanted to murder him.
- nommm-nommm 11y agoEmailed code snippets have this problem when emailed using some email clients (Outlook, looking at you)
- jimbokun 11y agoHah, I was giving a presentation where I was running little queries as part of a demo. I had copied the queries into the presenter notes in PowerPoint, then pasted them one at a time into a web app to run them. Couldn't figure out why one of them wasn't working, and it was actually an audience member who figured out PowerPoint had turned a quote character into a "smart quote".
- ciupicri 11y agoI've had to use a couple of times in order to get a proper indentation. I know there's <PRE> and <CODE>, but they aren't allowed on every blog, so I had to improvise.
- ygra 11y ago> blacklist pull requests from anyone who forks it Why, if I may ask? If they introduce compile errors in your project, those should be caught by the CI build and test run, shouldn't they? In any case, accepting a code change just by looking at the diff and without even trying it out sounds like not the best course of action to me anyway.
- sheraz 11y agoYes. A good CI will blow up any malicious pull requests provided you are using a compiled language. Or if interpreted, you need sufficient code coverage that your tests will blow up