I think this is done enough.

I think adding it in the Community Resources is a good move:

• It will let users use it right away
• You can collect feedback and fix stuff while it's there
• The wait time for upstreaming linters is very long (It's essentially a special integration, with all the costs of one). So getting it somewhere in the meanwhile makes sense.

Right, adding the "shell command" words would be nice.

Note that the comment when removing the alias has also this "issue":

Aliased 'arc foo' to 'foo'.

There's no description of what problems this is trying to solve. I don't want to just add random fields in the hope that some will be useful; see Contributing Feature Requests.

I'll need to at some point go and clean these up.

• I believe username association works correctly.
• Assigning an email is already mandatory when creating a bot:

I don't plan Almanac to ever autoscale. Autoscaling would be via a new, non-Almanac EC2 blueprint (or similar blueprints, which permit creation of new host resources). I'm just going to merge this into T5544.

To clarify: I certainly don't this dependency to be simply swapped for another dependency. Also having 2 server implementations isn't desirable either. I can't judge whether a PHP implementation can challenge the NodeJS implementation because I'm lacking the experience in PHP.

Perfect! This should be easy to fix.

I think I made some here: https://secure.phabricator.com/project/profile/1387/ by editing the Custom View Policy and trying to open the transaction in an Incognito window.

I can reproduce the bug on what I think is a relatively up-to-date production installation.

I can't examine any of those either, but I can reproduce the bug on what I think is a relatively up-to-date production installation.
If it helps, this URL was linked from https://phabricator.wikimedia.org/project/profile/1090/#17189 (edit policy entry)

You need to give us steps we can actually follow in a development environment to reproduce the issue. We can not examine database data, logs, etc, on the WMF install.

In T9869#146119, @chad wrote:

Please file bugs with more information, specifically steps to reproduce. See Contributing Bug Reports.

Here's what I get. Note:

I can't reproduce, those steps work fine for me on this install. I can click "Show Details" on the transaction above while logged out and get a dialog showing the details.

Here's my guess at how to reproduce:

This is very unlikely to happen.

I think it would be nice to have out-of-the-box support for the clang linter.

Please file bugs with more information, specifically steps to reproduce. See Contributing Bug Reports.

I would write it as a custom application. There are 40+ applications in Phabricator which can provide you examples. We don't provide support for building custom applications, so you're on your own and should expect it to task a few days (if you're familiar with Phabricator development). This isn't really a good task if you're new to Phabricator. If you need us to write such an application, we do have Consulting available.

I'd really like to see this happen. So far I haven't used Aphlict simply because I don't want to manage the extra dependency.

fyi I ended up implementing Protobuild.exe --build and a bunch of other functionality (like support for build scripts), which negated the need for having separate build plans for each platform

Hi, at first thanks a lot for your answer.

My bad, I didn't activate the option serving this repository over HTTP!
This wasn't a bug at all it's just I didn't ready the instruction carefully.
I missed this line:

Make sure you've added an SSH public key to your account. You can do this in Settings → SSH Public Keys.

Authentication Failure
This Phabricator install is not configured with any enabled authentication providers which can be used to log in. If you have accidentally locked yourself out by disabling all providers, you can use phabricator/bin/auth recover <username> to recover access to an administrative account.

Voilà ce que j'obtiens en faisant un clone.

Cloning into 'testrepo'...
* Couldn't find host phabricator.vonji.fr in the .netrc file; using defaults
* Hostname was NOT found in DNS cache
*   Trying 62.210.236.42...
* Connected to phabricator.vonji.fr (62.210.236.42) port 80 (#0)
> GET /diffusion/TR/testrepo.git/info/refs?service=git-upload-pack HTTP/1.1
User-Agent: git/1.9.1
Host: phabricator.vonji.fr
Accept: */*
Accept-Encoding: gzip
Pragma: no-cache

In T9848#145894, @OCram wrote:

Hmm strange, I do not use any md5 function. I simply copied the ArcanistTextLinter and changed it to work with files containing \r\n line ends. Thats the reason why I do not know where the md5 function all or the variables come from ^^

D14589 should fix the immediate issue. I've also cherry-picked it into stable.

@epriestley Thanks for the analysis, I'm happy it was easy to reproduce.

Context here is:

I think this is just junked up and I hadn't pushed this server when I claimed it works.

This is likely the next cause up the line:

I just saw this happen on this install (no metamta.differential.unified-comment-context) so my guess was probably incorrect.