Interactive platform selection for exfig init - #36
Conversation
UX improvement: Instead of failing when the required `--platform` argument is missing, `exfig init` now interactively prompts the user to select a platform if running in a TTY environment. - Make `platform` argument optional in `GenerateConfigFile` - Add `CaseIterable` conformance to `Platform` enum - Implement interactive prompt logic using `TTYDetector` - Gracefully handle non-interactive environments by throwing a validation error if the argument is missing
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
Improved the UX of the
exfig initcommand. Previously, omitting the--platformargument resulted in an error. Now, in interactive environments (TTY), the user is presented with a numbered list of available platforms to choose from. This guides the user through the setup process instead of halting it. In non-interactive environments (e.g., CI scripts), the command still fails safely if the argument is missing. This change required making theplatformargument optional and addingCaseIterableconformance to thePlatformenum.PR created automatically by Jules for task 15904252445429364441 started by @alexey1312