Skip to content
This repository was archived by the owner on Jul 21, 2025. It is now read-only.

Conversation

@akshay-online
Copy link
Collaborator

This pull request includes several changes to the src/ADOGenerator/Program.cs file to improve code readability and user interaction. The most important changes include restructuring conditional statements, enhancing user prompts, and modifying file path construction.

Code readability improvements:

  • src/ADOGenerator/Program.cs: Restructured the conditional statement in the TryGetTemplateDetails method to improve readability by using an else block.

User interaction enhancements:

  • src/ADOGenerator/Program.cs: Updated the user prompt messages in the ValidateExtensions method to provide clearer instructions, such as adding "press enter to confirm" to the prompts.

File path construction:

  • src/ADOGenerator/Program.cs: Modified the construction of the extensionsFilePath in the ValidateExtensions method to include the current directory and "Templates" folder for more accurate path resolution.
  • Refactored TryGetTemplateDetails to include an else block for template existence checks, improving logic and readability.
  • Updated extensionsFilePath construction in ValidateExtensions to ensure correct path by including current directory and "Templates" folder.
  • Enhanced user prompts in ValidateExtensions for clearer instructions and adjusted input handling to treat empty input as confirmation.

- Refactored `TryGetTemplateDetails` to include an `else` block for template existence checks, improving logic and readability.
- Updated `extensionsFilePath` construction in `ValidateExtensions` to ensure correct path by including current directory and "Templates" folder.
- Enhanced user prompts in `ValidateExtensions` for clearer instructions and adjusted input handling to treat empty input as confirmation.
@akshay-online akshay-online merged commit 682332c into main Jan 24, 2025
2 checks passed
@akshay-online akshay-online deleted the aksh/bug-fixes branch January 24, 2025 15:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants