TE
科技回声
首页24小时热榜最新最佳问答展示工作
GitHubTwitter
首页

科技回声

基于 Next.js 构建的科技新闻平台,提供全球科技新闻和讨论内容。

GitHubTwitter

首页

首页最新最佳问答展示工作

资源链接

HackerNews API原版 HackerNewsNext.js

© 2025 科技回声. 版权所有。

Expanding Context in Diffs

112 点作者 nickpresta超过 11 年前

15 条评论

yuvadam超过 11 年前
I hoped they would fix line wrapping - which should be a relatively simply task - by now.<p>This is a very needed feature for projects that work a lot with text files (such as markdown) that do not automatically break on set widths (e.g. [1])<p>I&#x27;ve already contacted Github support with this request months ago, but no one&#x27;s stepped up with a fix. Maybe this can get escalated somehow? Heck, I&#x27;d even submit a pull request if it was possible for github&#x2F;github.<p>[1] - <a href="https://github.com/cryptoparty/handbook/pull/72/files" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;cryptoparty&#x2F;handbook&#x2F;pull&#x2F;72&#x2F;files</a>
评论 #6834335 未加载
评论 #6842476 未加载
评论 #6834908 未加载
评论 #6835863 未加载
jakub_g超过 11 年前
Since we&#x27;re touching code reviews, I&#x27;ll plug my userscript here. It&#x27;s useful for me for code reviews of big commits: you can fold the files, mark them as reviewed etc. and all of this is persisted in the local storage. You can install it from [1] if you know userscripts or read [2] otherwise.<p>[1] <a href="http://userscripts.org/scripts/show/153049" rel="nofollow">http:&#x2F;&#x2F;userscripts.org&#x2F;scripts&#x2F;show&#x2F;153049</a><p>[2] <a href="https://github.com/jakub-g/gh-code-review-assistant" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;jakub-g&#x2F;gh-code-review-assistant</a>
obiterdictum超过 11 年前
Worth noting that BitBucket had this as long as I can remember.
ataggart超过 11 年前
Side-by-side diffs would be superior.
评论 #6836541 未加载
评论 #6833984 未加载
评论 #6835022 未加载
shurcooL超过 11 年前
&gt; and the feature is available anywhere we render diffs<p>No, it&#x27;s not available on gist.github.com.<p>Sure, they may (or may not) port it over after some time, but I just don&#x27;t get why they can&#x27;t use the same code in the first place. Why is GitHub reinventing the wheel in for its very own sub-site?<p>(Other than that, this is absolutely great!)
philfreo超过 11 年前
Nice! I&#x27;ve been wanting this feature for a long time:<p><a href="http://philfreo.com/blog/the-problem-with-pull-requests-code-reviews/" rel="nofollow">http:&#x2F;&#x2F;philfreo.com&#x2F;blog&#x2F;the-problem-with-pull-requests-code...</a>
mduan超过 11 年前
Since we&#x27;re on the subject of Github&#x27;s diff viewer, thought I&#x27;d throw in a plug for a chrome extension I wrote recently to improve it:<p><a href="https://chrome.google.com/webstore/detail/github-enhancement-suite/ahamcncifjblaomhphpfpopppadboiin" rel="nofollow">https:&#x2F;&#x2F;chrome.google.com&#x2F;webstore&#x2F;detail&#x2F;github-enhancement...</a><p>Has side-by-side + word wrap + context (somewhat nicer looking than Github&#x27;s, although could be biased)<p>Largely inspired by Phabricator&#x27;s diff viewer.
DanielRibeiro超过 11 年前
Cool! Now the only thing missing is to integrate this[1] to natively support syntax highlight on diffs ;)<p>[1] <a href="https://github.com/danielribeiro/github-diff-highlight-extension" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;danielribeiro&#x2F;github-diff-highlight-exten...</a>
paddy_m超过 11 年前
I was just looking for this over the weekend, and found it this morning.<p>When I talk about code to collaborators in email or chat, I always try to link to the file on github rather than just giving a path. I much prefer to recieve feedback that way. Maybe i&#x27;m not good enough with my text editor, but it is just a lot faster to get to the file that way.
bencoder超过 11 年前
Been needing this badly for reviewing changes to legacy code which tends to require a much larger context for understanding of a diff.<p>Hope it shows up in GH enterprise soon!
评论 #6834427 未加载
bcl超过 11 年前
When working locally you can do something similar with -U&lt;n&gt; and I&#x27;ve also found using --patience can sometimes generate cleaner patches.
alxndr超过 11 年前
Fingers crossed for &quot;default to ignore whitespace in <i>all</i> diffs&quot; to be the next feature they roll out...
staticshock超过 11 年前
phabricator has had this feature forever. Here&#x27;s an example diff: <a href="https://secure.phabricator.com/rP1b026fa6291fd86ed6a2e598e6a013b5b6124532" rel="nofollow">https:&#x2F;&#x2F;secure.phabricator.com&#x2F;rP1b026fa6291fd86ed6a2e598e6a...</a>
johnny_reilly超过 11 年前
I love these guys
crassus超过 11 年前
If any githubbers are reading, it would be great to be able to filter my organization&#x27;s timeline events out of my personal timeline events. It&#x27;s been bugging me for years. Thanks.
评论 #6836570 未加载