Skip to content

Implement --yes flag to assume yes on interactive prompts#83

Open
ivantodorovich wants to merge 1 commit into
OCA:mainfrom
ivantodorovich:flag-yes
Open

Implement --yes flag to assume yes on interactive prompts#83
ivantodorovich wants to merge 1 commit into
OCA:mainfrom
ivantodorovich:flag-yes

Conversation

@ivantodorovich

Copy link
Copy Markdown
Contributor

No description provided.

@sebalix

sebalix commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator

A user would expect this flag to work also when porting commits/PRs (port_addon_pr.py), what do you think? Probably more dangerous there as some PRs should not be ported but probably blacklisted.

@ivantodorovich

Copy link
Copy Markdown
Contributor Author

Probably more dangerous there as some PRs should not be ported but probably blacklisted.

I considered but this is exactly why I hesitated and ended up not implementing it there. The port PR process is a lot more interactive. Prompts like "create a draft PR.. ?" should perhaps not be assumed yes, and have their own specific --create-pull-request option. WDYT?

@ivantodorovich

Copy link
Copy Markdown
Contributor Author

I also rebased this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants