search

SmartBear-DevRel / explore-cli

Simple utility CLI for importing and exporting data to and from SwaggerHub Explore
https://www.nuget.org/packages/Explore.Cli
Apache License 2.0
5 stars 4 forks source link

Refactor `Program.cs` #15

Open Yousef-Majidi opened 1 year ago

Yousef-Majidi commented 1 year ago

I recently worked on adding some features to this project and identified a multiple refactor possibilities. For example, export-spaces and 'import-spaces' can be refactored into individual modules and their code can be further refactored into methods.

Yousef-Majidi commented 1 year ago

if you agree, I could work on this slowly over the next few weeks. This will also make it easier to develop some unit testing for the core functionalities.

frankkilcommins commented 1 year ago

It's a slight trade off at this point. I normally start from the outset with dependency injection and separation of concerns, but fir this project was weary of spending the additional time until ROI is more clear. Part of me is also enjoying the simple navigation.

That being said, if you're very eager and willing, feel free to start the refactor. You can also propose your changes before investing the time, if you would like feedback etc.