Page MenuHomePhabricator
Feed Advanced Search

Dec 16 2016

epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 2:53 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 2:52 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 2:49 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 2:19 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 1:46 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 16 2016, 12:28 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential

Dec 15 2016

epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 15 2016, 7:24 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential

Dec 14 2016

epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 10:02 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 9:38 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 4:05 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a comment to T11114: Move Differential to EditEngine.

Broadly, DifferentialCustomField drives a very large amount of stuff in Differential. This was the first generation of custom fields, and is the most complex implementation.

Dec 14 2016, 3:26 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 2:42 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley updated the task description for T11114: Move Differential to EditEngine.
Dec 14 2016, 2:32 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 2:20 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley updated the task description for T11114: Move Differential to EditEngine.
Dec 14 2016, 2:05 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley updated the task description for T11114: Move Differential to EditEngine.
Dec 14 2016, 1:26 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley updated the task description for T11114: Move Differential to EditEngine.
Dec 14 2016, 1:00 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 14 2016, 1:17 AM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential

Dec 13 2016

epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 13 2016, 11:29 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 13 2016, 11:10 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 13 2016, 9:22 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley added a revision to T11114: Move Differential to EditEngine: Restricted Differential Revision.
Dec 13 2016, 8:38 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
epriestley moved T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential from Backlog to v3 (Infrastructure) on the Differential board.
Dec 13 2016, 4:29 PM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential
epriestley moved T11050: Allow resigning from a review you are not directly responsible for from Backlog to v3 (Infrastructure) on the Differential board.
Dec 13 2016, 4:20 PM · Prioritized, Restricted Project, Differential, Feature Request
epriestley moved T11114: Move Differential to EditEngine from Backlog to v3 (Infrastructure) on the Differential board.
Dec 13 2016, 4:20 PM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential

Dec 5 2016

epriestley closed T11044: Support partitioning application databases across multiple database hosts as Resolved.

This has been running cleanly in production for roughly two weeks, and appears stable. secure001 stopped writing Files data at F1943597 and we're now at F2078289 on secure003. We saw a couple of minor setup issues (mostly: exception messages not being tailored enough) but no fundamental issues.

Dec 5 2016, 10:54 PM · Clusters, Prioritized
epriestley added a comment to T550: Build an SSH conduit client.

I'm planning to support this in the arc experimental branch, although initial support won't be optimized. We can look at ControlPath and/or adding batching to the protocol layer to improve performance once it works.

Dec 5 2016, 7:18 PM · Prioritized, Arcanist, Restricted Project, Restricted Project, Conduit
epriestley moved T550: Build an SSH conduit client from Backlog to The Queue on the Prioritized board.
Dec 5 2016, 7:07 PM · Prioritized, Arcanist, Restricted Project, Restricted Project, Conduit
epriestley added projects to T550: Build an SSH conduit client: Arcanist, Prioritized.
Dec 5 2016, 7:06 PM · Prioritized, Arcanist, Restricted Project, Restricted Project, Conduit

Nov 29 2016

epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

This one is fine, I'm still going to fiddle with "in" too.

Nov 29 2016, 8:36 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
aklapper added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

In https://phabricator.wikimedia.org/T76732#2803813 ksmith brings up that

In the search bar in the toolbar at the top of the screen, searching for "Maps" brings up Maps as the third option. Searching for "Discovery" brings up Discovery as the second option.
When editing a task and entering tags, autocomplete does bring up Maps and Discovery as the first items.

but that could be a separate (unprioritized) task I guess.

Nov 29 2016, 8:35 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

Feel free to file a new task when you encounter actual problems with real text and we can look at adjusting the limit, adding an additional block-level difference phase, rewriting PhutilEditDistanceMatrix in C (see T2312), or other approaches. We can make a better decision based on concrete examples of where the current solution is falling short than by guessing where issues might arise.

Nov 29 2016, 12:09 PM · Prioritized, Transactions
dseifert added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

@epriestley thank you very much for the fast update and fix. It is working now with my example, and should be much better with a lot of pages.

Nov 29 2016, 6:16 AM · Prioritized, Transactions

Nov 23 2016

epriestley added a comment to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`.

After D16933, the behavior of arc browse <branch> on the experimental branch is:

Nov 23 2016, 8:07 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 7:50 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T5267: Localize Phabricator: Restricted Differential Revision.
Nov 23 2016, 7:09 PM · Prioritized, Localization
epriestley added a comment to T5267: Localize Phabricator.

Sure, not a problem.

Nov 23 2016, 7:03 PM · Prioritized, Localization
Nikerabbit added a comment to T5267: Localize Phabricator.

We now export a frequency.json, with string keys in a list. Since we haven't actually collected frequency data, the order is currently (roughly) random:

https://secure.phabricator.com/source/translatewiki/browse/master/projects/phabricator/maniphest/frequency.json;00108900457eda804c9c786dd76ff96620a05195

Nov 23 2016, 6:54 PM · Prioritized, Localization
epriestley added a comment to T5267: Localize Phabricator.

We now export strings into projects/<project>/<group>/en.json, etc. For example, here are strings which appear ONLY in Maniphest:

Nov 23 2016, 6:46 PM · Prioritized, Localization
epriestley added a revision to T5267: Localize Phabricator: Restricted Differential Revision.
Nov 23 2016, 6:41 PM · Prioritized, Localization
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 23 2016, 6:31 PM · Clusters, Prioritized
epriestley added a comment to T5267: Localize Phabricator.

Okay, I believe we already produce a format which is compatible with that definition:

Nov 23 2016, 6:16 PM · Prioritized, Localization
Nikerabbit added a comment to T5267: Localize Phabricator.
  1. Proto-english strings.
  2. Nested English translations.
  3. Variable types.
  4. Usage frequency information ("this string has frequency 0.9 out of 1.0", or "this string is part of the 'most common 10% of strings' group").
  5. Application groupings ("this string is part of Maniphest").
  6. Places in the code where the string appears ("this string appears at x.php line 234").
  7. Additional human-readable context information, like "qqq" (very rarely available).
Nov 23 2016, 5:59 PM · Prioritized, Localization
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 5:49 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a comment to T5267: Localize Phabricator.

Maybe it would be simplest if you just give me an example of exactly what you want us to emit?

Nov 23 2016, 4:56 PM · Prioritized, Localization
Nikerabbit added a comment to T5267: Localize Phabricator.

What works out of the box is JSON files with string key-value pairs for source text. For documentation the qqq.json that is "just-another-language" is what we use most often. The strings of that file are parsed as wikitext using MediaWiki syntax and displayed automatically to translators.

Nov 23 2016, 4:44 PM · Prioritized, Localization
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 4:31 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 4:29 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a comment to T5267: Localize Phabricator.

That sounds fine to me. Do you have any preference on what format you'd like the proto-english + variable type annotations in? Maybe something like this, since it sounds like you're doing some level of custom stuff on your side anyway?

Nov 23 2016, 4:01 PM · Prioritized, Localization
Nikerabbit added a comment to T5267: Localize Phabricator.

Okay, I have been thinking this for a bit. I don't have a perfect solution, but I have the following proposal.

Nov 23 2016, 3:40 PM · Prioritized, Localization
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 3:35 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 3:20 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 23 2016, 12:47 PM · Clusters, Prioritized
chad added a comment to T11355: Build a flexible revision lookup API in Arcanist.

frowncat

Nov 23 2016, 2:57 AM · Restricted Project, Prioritized, Arcanist
epriestley added a comment to T11355: Build a flexible revision lookup API in Arcanist.

I'm sure if I keep digging we'll be out of this hole in no time.

Nov 23 2016, 2:41 AM · Restricted Project, Prioritized, Arcanist
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 1:19 AM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley added a revision to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Restricted Differential Revision.
Nov 23 2016, 12:49 AM · Arcanist, Prioritized, Restricted Project, Feature Request

Nov 22 2016

epriestley moved T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD` from Backlog to The Queue on the Prioritized board.
Nov 22 2016, 10:38 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley moved T4706: Maybe make `arc land D123` go look up the branch? from Backlog to The Queue on the Prioritized board.
Nov 22 2016, 10:38 PM · Prioritized, Restricted Project, Arcanist
epriestley moved T11355: Build a flexible revision lookup API in Arcanist from Backlog to The Queue on the Prioritized board.
Nov 22 2016, 10:38 PM · Restricted Project, Prioritized, Arcanist
epriestley added a comment to T11355: Build a flexible revision lookup API in Arcanist.

I'm not sure if D16857 is even close to the right level of generality or correct approach and the answer has yet to come to me in a dream, so I'm going to push forward on top of it and see how far I get. If too much stuff stacks up I'll maybe shove this stuff into an experimental branch for now until I decide whether I'm throwing it all away or not.

Nov 22 2016, 10:35 PM · Restricted Project, Prioritized, Arcanist
epriestley closed T11845: Media with "autoplay" should possibly not autoplay in news feed as Resolved by committing Restricted Diffusion Commit.
Nov 22 2016, 10:20 PM · Prioritized, Remarkup
epriestley added a revision to T11845: Media with "autoplay" should possibly not autoplay in news feed: Restricted Differential Revision.
Nov 22 2016, 10:17 PM · Prioritized, Remarkup
epriestley moved T11044: Support partitioning application databases across multiple database hosts from The Queue to Paused on the Prioritized board.

This seems to be working now. I'm going to let it sit in production for a while and see if any issues crop up before considering it resolved, but it seems like everything is working smoothly.

Nov 22 2016, 7:55 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

I configured 003 to replicate to 004:

Nov 22 2016, 7:26 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

Okay, we're headed back into readonly mode shortly to set up replication. I'm going to verify D16916 along the way so there may be some "partitions disagree about life" errors.

Nov 22 2016, 7:00 PM · Clusters, Prioritized
epriestley closed T11672: Evaluate persistent connections from HTTP contexts, a subtask of T11044: Support partitioning application databases across multiple database hosts, as Resolved.
Nov 22 2016, 6:57 PM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 6:52 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

Here's what I've done so far:

Nov 22 2016, 6:46 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

I'm partitioning secure.phabricator.com now. Things will drop into read-only mode for a bit.

Nov 22 2016, 6:03 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.
  • T11908 is a followup for executing queries for multiple applications on a single connection. I believe the pathway for that is straightforward and fairly short, but that no install would really reap substantial benefits from it today, so I don't expect to pursue it for some time.
  • I believe everything else is now complete, so I'll put this in production as soon as everything here lands and we can see what catches on fire.
Nov 22 2016, 2:58 PM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 2:55 PM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 2:07 PM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 12:36 PM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 2:08 AM · Clusters, Prioritized
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Restricted Differential Revision.
Nov 22 2016, 1:05 AM · Clusters, Prioritized

Nov 21 2016

epriestley closed T10759: Run PhabricatorDatabase/MySQLSetupCheck against all configured replicas, a subtask of T11044: Support partitioning application databases across multiple database hosts, as Resolved.
Nov 21 2016, 11:56 PM · Clusters, Prioritized
epriestley added a subtask for T11044: Support partitioning application databases across multiple database hosts: T11672: Evaluate persistent connections from HTTP contexts.
Nov 21 2016, 4:01 PM · Clusters, Prioritized
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

We have at least one tricky issue remaining: when applying storage upgrades, we currently apply them like this:

Nov 21 2016, 3:48 PM · Clusters, Prioritized

Nov 19 2016

epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

General state of the world here:

Nov 19 2016, 10:11 PM · Clusters, Prioritized

Nov 18 2016

epriestley added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

Yeah, I tend to agree. I don't really see any approaches we can take here that don't make things a little worse for experienced users who already know what they're looking for, and I'm not sure there's too much of a use case left for less-experienced users.

Nov 18 2016, 5:27 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
chad added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

The main concern for me would be typeaheads are for knowing what you're looking for already, and making that faster / easier. Browse interfaces are for discovery.

Nov 18 2016, 4:59 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

This is sort of what I was thinking about for the "Search Description" / "Usage Hint" (descriptions / projects are random examples):

Nov 18 2016, 4:39 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia

Nov 17 2016

epriestley added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

(You could write some similar rules if you want to try something in that vein, although it's a pretty big pain to fork JS/CSS right now.)

Nov 17 2016, 9:10 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
chad added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

I should make a Herald rule to auto-resign.

Nov 17 2016, 9:08 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

Okay well @chad resigned from D16893 as soon as he saw animation in a CSS file so I don't think that one's going anywhere.

Nov 17 2016, 9:07 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley added a revision to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier: Restricted Differential Revision.
Nov 17 2016, 9:03 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
aklapper added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

In https://phabricator.wikimedia.org/T76732#2803813 ksmith brings up that

In the search bar in the toolbar at the top of the screen, searching for "Maps" brings up Maps as the third option. Searching for "Discovery" brings up Discovery as the second option.
When editing a task and entering tags, autocomplete does bring up Maps and Discovery as the first items.

but that could be a separate (unprioritized) task I guess.

Nov 17 2016, 8:54 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a revision to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier: Restricted Differential Revision.
Nov 17 2016, 8:51 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
aklapper added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

Ah, thanks! I've asked downstream if anyone still runs into any issues, but so far I also believe this is now resolved.

Nov 17 2016, 8:45 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

Alright, I'm going to try this:

Nov 17 2016, 8:31 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

The most-likely-to-work secret magic for "Not in: Wikidata" is not(wikidata):

Nov 17 2016, 8:08 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
aklapper added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

phabricator.wikimedia.org is running a version including these changes plus D16886 on top. So far it looks like a great improvement!

Nov 17 2016, 8:03 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
aklapper added a comment to T11034: Provide more information about projects, etc. in tokenizer browse dialog to make distinguishing between similar projects easier.

+1 to what 20after4 wrote. The approach is definitely helpful, and to make this look less cluttered and inconsistent ("are projects with longer descriptions which take up more vertical space somehow more important?"), cutting off project descriptions seems like a good compromise.

Nov 17 2016, 7:46 PM · Restricted Project, Design, Typeahead, Prioritized, Wikimedia
epriestley added a revision to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches: Restricted Differential Revision.
Nov 17 2016, 1:05 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

I caught another case of this: here, "Phacility" should be first, but is not.

Nov 17 2016, 11:38 AM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search

Nov 16 2016

epriestley reopened T8588: One instance has a severe, global performance issue as "Open".
Nov 16 2016, 11:55 PM · Restricted Project
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

Ah. I believe I fixed a simpler version of that case earlier, but not the specific case you describe. D16881 should fix that case.

Nov 16 2016, 6:08 PM · Prioritized, Transactions
epriestley added a revision to T7643: Improve prose diffs (was: description changes don't generate usable diffs): Restricted Differential Revision.
Nov 16 2016, 6:05 PM · Prioritized, Transactions
dseifert added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

@epriestley One issue that is still open is adding a change to the beginning and to the end of a large prose text. The diff of that becomes unusable. This is still reproducable with the current stable branch )(fc71a7e92dc26e0d93ec33d709391411d2bcd827 (Mon, Nov 14)). For details / reproduction steps please see T11825.

Nov 16 2016, 5:34 PM · Prioritized, Transactions