Page MenuHomePhabricator

scripts/category.py: Enable "-always" option for other actions than clean
Open, MediumPublicFeature

Description

I'm running the category.py script without the -always argument, so the bot should show the diff and ask if it needs to be edited. But the bot doesn't this, always don't disable, bot making wrong edits.

There seems to be an error at line #1530:

elif option == 'always':
      options[option] = True

Here T291757#7378248 always has been set to always=True as default, so it is always True.

Perhaps this error remained in other scripts.

Event Timeline

Be aware: -always option is defined for clean action only. What is the command line you have used?

I use the remove command, examples are given here T318239.

How to disable -always for other bulk edit commands?

Xqt renamed this task from scripts/category.py: Bug of "-always" argument to scripts/category.py: Enable "-always" option for other actions than clean.Sep 21 2022, 4:34 PM
Xqt triaged this task as Medium priority.
Xqt changed the subtype of this task from "Task" to "Feature Request".

How to disable -always for other bulk edit commands?

Currently the only way is to simulate the run first using the global -simulate option. I think this your feature request is very important and should be implemented.