One more possibility is to document that people need to setup an ssh key on their account to be able to push their changes.

Using a https URL for the staging area means people have to type their Phabricator password each time they create a diff (if they don't have pushInsteadOf set up); I'm not too keen on that.

Disabling the staging area will not let us use the staging area (duh), and we should really work on using them (https://secure.phabricator.com/book/phabricator/article/harbormaster/#change-handoff recommends them) so it's not great.

--skip-staging will prevent tools using the staging area which is bad when we start actually using them.

After unrolling everything, I guess a sensible compromise would be

• use https for the staging area configuration (which will allow people to push it even if nothing is configured, by just typing their password)
• make sure people configure a ssh key and git's pushInsteadOf setting (so that they don't _have_ to type their password)

sounds like a plan!

use https for the staging area configuration (which will allow people to push it even if nothing is configured, by just typing their password)

That'd mean also to change that uri to be read/write, it's read-only for now.

In T1522#27987, @ardumont wrote:

use https for the staging area configuration (which will allow people to push it even if nothing is configured, by just typing their password)

That'd mean also to change that uri to be read/write, it's read-only for now.

That's done now.

I've moved the repositories that need it to have their staging area set to https.

As it turns out, there's *also* a hoop to go through if one wants to push via HTTP: Phabricator wants you to add a VCS password to your account, separate from your account password (which is, IMO, a good thing!).

So either way we need to document this in our contribution documentation. I guess I'll document both the VCS password approach and the ssh key + pushInsteadOf approach.

I've documented the extra hoops people need to jump through for git push to work on https://wiki.softwareheritage.org/index.php?title=Code_review_in_Phabricator&type=revision&diff=964&oldid=899

Thanks by the way!