cerus / cerus/deploy-helper

Feature: Split Deployer class

Open
#5 3 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue hacktoberfest
Dominant language
Java
Stars
3
Forks
3
PR merge metrics
No merged PRs in 30d

Description

The `Deployer` class is a little big at the moment and it is expected to grow. This will eventually lead to messy code and thats what we are trying to avoid. A solution to this would be to split the class into multiple classes. This is also a great way to get a little more abstraction into the code base.

This is what I imagined:
- Create some sort of task interface with some sort of `run` method
- `execSsh` and `execCommands` will be moved into their respective classes which implement the task interface
- Create instances of the task classes in the `Deployer` class and call them when needed

I think this would greatly improve the code. If you have any other solution idea please let me know!

Please comment if you would like to work on this.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.