Page MenuHomePhabricator

tests: rename test-clone-uncompressed.t
ClosedPublic

Authored by valentin.gatienbaron on Aug 1 2021, 10:59 AM.

Details

Summary

as clone --uncompressed is deprecated in favor of --stream

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit accepted this revision.Aug 1 2021, 3:02 PM
This revision is now accepted and ready to land.Aug 1 2021, 3:02 PM

This patch seems to be for default. I think we're supposed to be in code freeze (though lots of patches have still been queued). Should we mark patches (like this one) that are not for stable as "Changes Requested" so they're not accidentally queued during the freeze?

I have been assuming that changes can still be submitted during a change freeze, but they are just going to get ignored for a few days. None of my changes are particularly intended to go on 5.9.

I have been assuming that changes can still be submitted during a change freeze, but they are just going to get ignored for a few days. None of my changes are particularly intended to go on 5.9.

That's fine with me. I've just noticed that lots of patches get queued even though I also heard that we're supposed to have a code freeze this time. So you may consider my message just a reminder to reviewers :)

Even though it is quite simple, it can wait for the end of the freeze.
@valentin.gatienbaron We're supposed to have a real code freeze this cycle, so it's really the reviewers' job to ignore the patches that don't meet the stable criteria until the end of the freeze. As Martin said, it seems that some reviewers still queued patches.

This revision was automatically updated to reflect the committed changes.