I found a method called setControlInstructions that adds helpful text above the field

• Refactored to use the setControlInstructions method on the control

How would this work if the task has more than one revision associated with it?

There is one way to sort of do this already, by using milestones for your columns, then you can use herald 'add projects' rules. This only works by sort of abusing the purpose of milestones to take advantage of the fact that milestone columns are mutually exclusive. Adding one milestone removes the other milestones from the same parent project.

I agree that the hover behavior could be improved. Showing only after a delay could help here, though that would make it slightly worse for the times you actually want to interact with the hover.

Remove the mailKey column from PhabricatorPaste

In D25900#24197, @valerio.bozzolan wrote:

You know better than me the context. Is there a possibility that an installation is already without that column?

You know better than me the context. Is there a possibility that an installation is already without that column?

Created D25900 for the deleted paste mailkey migration.

Ran ./bin/storage upgrade --namespace phabricator_quickstart_test2, and that didn't fail.

OK lol no, this has nothing to do with D25898. Need another task for the "Missing input descriptions"

In D25898#24165, @avivey wrote:

• There's already a "description" field on these, which I think was supposed to be displayed; We might have lost it at some point, which is concerning.

• There's already a "description" field on these, which I think was supposed to be displayed; We might have lost it at some point, which is concerning.
• Use pht() for all human-visible text, to allow translation.

Need to go over the relevant code for deciding if a commit has an approved Revision - I think it looks mostly on the commit message.

I updated this revision so the title will only show the monogram if the user is logged in

• The title will only show the monogram when the post is public

have an approved Revision", but this too can be tricked by a sufficiently competent user

Hackerman meme

Don't find a problem with this at all.

Ahem...

I created D25895. Reviews welcome.

clarify how to run the command (last change promise)

Aldo I may need extra help from @waldyrious asdlol

aaand I go to bed :3

OK my friend, I've worked a lot to make it shorter... but still more dense of useful info, reducing jokes at minimum.

This is much better. A few spelling errors remain, plus two suggestions to make sentences read a little less awkward.

Dear friend @mainframe98 - I suggest to click the button Hide "Done" inlines in the top-right corner to easily do another reading

See also https://secure.phabricator.com/T1751 and rARC3116d3656ad017783e75df7e863368053d753e7f - looks like we had arc backout at some point for this, but it's no longer there.

Yeah, that would work too.

You mean a shorthand for git revert + arc diff? Sounds reasonable

Yes, I mean the way it works in Meta's Phabricator: it runs the equivalent of git revert + git push behind the scenes.

Potentially related to Herald. Adding Herald to raise more attention.

try now

Tried. I Shall Not Pass :O

@valerio.bozzolan go ahead and give it a try? I think you have access to create rules.

I'd be very much in favor and I'd be happy to contribute changes for this. I've added type hints in my local fork to assist in development. Especially for core utility functions like idx and id which otherwise completely hide type information.

The code to select which tests to run by default is in ArcanistWorkflow::selectPathsForWorkflow().

There is no Edit Task option in the Add Action... dropdown. What makes you think so?

I can't see an Edit Task option in the dropdown. What actually enables this?

@avivey what do you think about? Maybe we can test with some powerful clicks

git rebase master; ./bin/celerity map; shows no output

Thanks for the quick review. I'll refrain from touching JS and CSS files.

I confirm that ./bin/celerity map does nothing after this change

Oh boy. D25893 first, I guess.

This makes me and probably other people mad lol

Aaand last 48 lines checked. Differential is crashing in my laptop lol

(Review 3 of ?)

P.S. maybe nice to introduce in AphrontView a method like getUserRequired() or something similar to implement exactly this logic - so we avoid that scattered logic everywhere

(epic review part 1 of ?)

I wonder if shouldAllowPublic() was just forgotten or if the idea behind was to exclude web crawlers (as querying those lists en-masse could be expensive).

Most of these can be done by adding the following to controller files related to query, list, and view files:

and reviews love you too! (lol)