Skip to content

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

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

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

Conversation

@ivantodorovich
Copy link
Contributor

No description provided.

@sebalix
Copy link
Collaborator

sebalix commented Jul 28, 2025

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
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
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