Skip to content

Refactor Program.cs #15

Description

@Yousef-Majidi

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.

Activity

  1. Yousef-Majidi commented on Nov 16, 2023

    @Yousef-Majidi
    ContributorAuthor

    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.

  2. frankkilcommins commented on Nov 17, 2023

    @frankkilcommins
    Contributor

    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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions