Skip to content

Add support for the OPAMAUTOANSWER environment variable (for internal use only)#6709

Merged
kit-ty-kate merged 2 commits intoocaml:masterfrom
kit-ty-kate:auto-answer
Oct 6, 2025
Merged

Add support for the OPAMAUTOANSWER environment variable (for internal use only)#6709
kit-ty-kate merged 2 commits intoocaml:masterfrom
kit-ty-kate:auto-answer

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

Required for #6611

Labeling every single OpamConsole.confirm is a bit too large of a task at the moment but this can be done in the future incrementally as need arise (e.g. #6611 does add three labels).

For now i prefer advertising this feature as internal-use only to allow it to grow and evolve as needed, rather than inviting users to use it and expect a full feature support with menus and other interactive prompts. In time though i'm sure it'll become something usable more widely.

Copy link
Copy Markdown
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed in meeting, this PR is good to go to unlock #6611.
In #6611, we'll introduce what is strictly needed for the tests. In a following PR, we'll implement completely a first real support for this option.

@kit-ty-kate kit-ty-kate merged commit ce734b3 into ocaml:master Oct 6, 2025
57 checks passed
@kit-ty-kate kit-ty-kate deleted the auto-answer branch October 6, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants