Postfix the type aliases with "T"

Rebase

Refactor test detection in a function

Abandoned in favor of D5687

Cover "noop" project case

In D5659#143653, @vlorentz wrote:

Jenkins/Phabricator reports most of the new code in listed_origins and _get_pages_for_project is not covered by tests, could you check? (It's sometimes wrong, we don't understand why)

Fix incremental testing + incorporate suggestions

Mhh it would appear I don't have the rights to push this.

Rebase

Rebase

... push the right one

Rebase

Rebase

Rebase

Rebase

Actually send the right changes...

Actually incorporate the proposed changes + improve commit message

Integrate suggestion to spend less time sleeping

In D5628#142822, @vlorentz wrote:

LGTM, but let's wait for a second opinion

One small nit, but the logic (and test) looks good. Thanks!

Rebase

Simplify date tests (sorry for the update spam)

Record lastmod for origins

Update typing and add retry tests + check other fatal errors

It looks like there are projects outside of the /p/ namespace. Just looking at the very first sitemap, I got an /adobe/ namespace (https://sourceforge.net/rest/adobe/manjobi), which implies that we should also consider namespaces outside of /p/ when listing.

Rebase

Fix and test visit status, improve error message

Rebase

In D5146#130548, @vlorentz wrote:

meh

Fix flake8 errors

In D4688#130529, @ardumont wrote:

Prior to update the diff, you might want to either use tox directly, which does among other thing trigger that check ^

Rebase and address some comments

Rebase

Add suggested assertion and fix comment typo

Rebase

Rebase

Rebase

Improve heads handling + remove useless fixture call

In D5146#130314, @vlorentz wrote:

hmm actually, modifying this global may interact with other parts of the code in surprising way.

Could you only use the env argument of subprocess functions instead of changing os.environ?

Fix flake8

Fix broken test + rebase (taking over from Antoine's work)

Update copyright notices

Rebase

• Disable user customization in Mercurial tests
• Sort imports according to isort rules
• Filter hg subprocess environment to the minimum
• Make Mercurial's behavior predictable for scripts

• Disable user customization in Mercurial tests
• Sort imports according to isort rules
• Filter hg subprocess environment to the minimum
• Make Mercurial's behavior predictable for scripts

Possibly this should be scope="session"? Since this trickles down to all subprocesses. That's a minor optimization, but might as well not be wasteful for no reason.

In D5124#129604, @vlorentz wrote:

Shouldn't it also happen outside tests?

In D4688#129595, @ardumont wrote:

@Alphare will you continue on this work as well?

I have no issue with deprecating this module if there is a more canonical way of doing the same thing, the mercurial-specific README will have to be adjusted, I simply tried the first example and got a traceback. :)