Until now, strip was bootstrapped as an extension. This patch adds
debugstrip on core. For the users who are still used to strip
extension, we will preserve that name.
Details
- Reviewers
durin42 baymax - Group Reviewers
hg-reviewers
Diff Detail
- Repository
- rHG Mercurial
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Event Timeline
mercurial/commands.py | ||
---|---|---|
5646 ↗ | (On Diff #16856) | debugstrip please (and put it in debugcommands) and Gregory points out we need a strip extension that preserves the old strip name for users that are used to that |
mercurial/commands.py | ||
---|---|---|
5646 ↗ | (On Diff #16856) | i see. do i need to replace strip by debugstrip on every occurrence in tests? otherwise, i would have to revert the changes in the tests and keep the extension as it is. |
mercurial/commands.py | ||
---|---|---|
5646 ↗ | (On Diff #16856) | One or two is fine, you can leave the majority on the legacy extension flavor. We want some using each so we don't have to add a new test to test one of them. |
hgext/strip.py | ||
---|---|---|
3 ↗ | (On Diff #16914) | the documentation should be left as it is. |
There seems to have been no activities on this Diff for the past 3 Months.
By policy, we are automatically moving it out of the need-review state.
Please, move it back to need-review without hesitation if this diff should still be discussed.
:baymax:need-review-idle: