This is an archive of the discontinued Mercurial Phabricator instance.

heptapod-ci: also run tests for chg on python 2
ClosedPublic

Authored by marmoute on Feb 28 2020, 8:25 AM.

Details

Summary

I am not aware of any regular effort to run test with chg. So let's at least do it
here.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

marmoute created this revision.Feb 28 2020, 8:25 AM
pulkit added a subscriber: pulkit.Feb 28 2020, 8:52 AM

Any reason why py3-chg is not included?

marmoute abandoned this revision.Feb 28 2020, 9:09 AM

how did this go on phabricator… (probably runing phabsend with the wrong hash)

abandonning this for now. I mostly wanted to see a run on heptapod (spoiler warning about 20 test are broken)

pulkit retitled this revision from heptapod-ci: also run some test for chg to heptapod-ci: also run tests for chg on python 2.Apr 2 2020, 11:57 AM
pulkit edited the summary of this revision. (Show Details)
pulkit updated this revision to Diff 20967.

Is this targetted for stable or default ? having it on stable would be nice.

Is this targetted for stable or default ? having it on stable would be nice.

Since all the fixes are on default branch, I am targeting it for default. Also default will be merged in stable after couple of weeks for release candidate, so we will have it in stable soon.

Can we get the fix grafted on the stable branches. There are people using chg in production that would need them.

pulkit added a comment.Apr 3 2020, 5:23 AM

Can we get the fix grafted on the stable branches. There are people using chg in production that would need them.

I don't have a strong opinion here because I feel fixes are minor.

Gentle review ping. Since, we are going to cut a RC this week, let's push this to default. Also for reviewers, the patch is still authored by @marmoute, I just rebased and updated after my chg fixes got in.

I wanted to make it accepted, but apparently I am flag as owner…

So this looks good to me. As followup we could have the same for pure and rust (probably starting with manual triggering to avoid CI load explosion).

What's the latest of having this for python3 ? it would be nice to have chg well covered there too.

Gentle ping, it would be nice to have this on stable.

Alphare accepted this revision.May 7 2020, 12:49 PM
pulkit accepted this revision.Jun 9 2020, 4:44 AM
This revision is now accepted and ready to land.Jun 9 2020, 4:44 AM
This revision was automatically updated to reflect the committed changes.